fix: delayMicroseconds intervals

Updated delayMicroseconds calls to respect Arduino documentation
zfields · Jan 21, 2021 · cc145fc · cc145fc
1 parent 3b762ef
commit cc145fc
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/library.properties b/library.properties
@@ -1,5 +1,5 @@
 name=NesRob
-version=1.0.0
+version=1.0.2
 author=Zachary J. Fields
 maintainer=Zachary J. Fields <zachary_fields@yahoo.com>
 sentence=Manipulates LED to signal NES R.O.B. (Robot Operating Buddy)

diff --git a/src/NesRob_Arduino.cpp b/src/NesRob_Arduino.cpp
@@ -16,14 +16,25 @@ NesRob::blinkCommand (
   Command command_
 ) {
   // Send initialization sequence
-  ::delayMicroseconds(66666); // 0000 (required delay between commands)
+  // Max delay of 16383 guaranteed by Arduino reference
+  // https://www.arduino.cc/reference/en/language/functions/time/delaymicroseconds/
+  {
+    ::delayMicroseconds(16383);
+    ::delayMicroseconds(16383);
+    ::delayMicroseconds(16383);
+    ::delayMicroseconds(16383);
+    ::delayMicroseconds(1134);
+  } // 0000 (required delay between commands)
   {
     ::digitalWrite(_led_pin, HIGH);
     ::delayMicroseconds(1500);
     ::digitalWrite(_led_pin, LOW);
     ::delayMicroseconds(15166);
   } // 1
-  ::delayMicroseconds(16666); // 0
+  {
+    ::delayMicroseconds(16383);
+    ::delayMicroseconds(283);
+  } // 0
 
   // Flash command sequence
   for (int i = 7 ; i >= 0 ; --i) {