esp8266/espneopixel: Disable IRQs during eps.neopixel_write.

torwag · dpgeorge · commit 69768c97c060 · 2016-09-06T11:51:35.000+10:00
Interrupts during neopixel_write causes timing problems and therefore
wrong light patterns.  Switching off IRQs should help to keep the strict
timing schedule.
diff --git a/esp8266/espneopixel.c b/esp8266/espneopixel.c
@@ -41,6 +41,7 @@ void /*ICACHE_RAM_ATTR*/ esp_neopixel_write(uint8_t pin, uint8_t *pixels, uint32
   }
 #endif
 
+  uint32_t irq_state = mp_hal_quiet_timing_enter();
   for(t = time0;; t = time0) {
     if(pix & mask) t = time1;                             // Bit high duration
     while(((c = mp_hal_ticks_cpu()) - startTime) < period); // Wait for bit start
@@ -55,4 +56,5 @@ void /*ICACHE_RAM_ATTR*/ esp_neopixel_write(uint8_t pin, uint8_t *pixels, uint32
     }
   }
   while((mp_hal_ticks_cpu() - startTime) < period); // Wait for last bit
+  mp_hal_quiet_timing_exit(irq_state);
 }

Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,7 @@ void /ICACHE_RAM_ATTR/ esp_neopixel_write(uint8_t pin, uint8_t *pixels, uint32`
`41`	`41`	`}`
`42`	`42`	`#endif`
`43`	`43`
	`44`	`+ uint32_t irq_state = mp_hal_quiet_timing_enter();`
`44`	`45`	`for(t = time0;; t = time0) {`
`45`	`46`	`if(pix & mask) t = time1; // Bit high duration`
`46`	`47`	`while(((c = mp_hal_ticks_cpu()) - startTime) < period); // Wait for bit start`
`@@ -55,4 +56,5 @@ void /ICACHE_RAM_ATTR/ esp_neopixel_write(uint8_t pin, uint8_t *pixels, uint32`
`55`	`56`	`}`
`56`	`57`	`}`
`57`	`58`	`while((mp_hal_ticks_cpu() - startTime) < period); // Wait for last bit`
	`59`	`+ mp_hal_quiet_timing_exit(irq_state);`
`58`	`60`	`}`