From 70816eeb52fd09c6a6fbcfd3ef5a4f1d6f0d624e Mon Sep 17 00:00:00 2001
From: Makuna <Makuna@hotmail.com>
Date: Fri, 1 May 2015 12:12:53 -0700
Subject: [PATCH] Reasonable with Glitches

---
 NeoPixelBus.cpp | 77 ++++++++++++++++++++++++++++++++++---------------
 NeoPixelBus.h   | 11 +------
 RgbColor.h      | 11 +------
 3 files changed, 56 insertions(+), 43 deletions(-)

diff --git a/NeoPixelBus.cpp b/NeoPixelBus.cpp
index cd67645..6ab3c5e 100644
--- a/NeoPixelBus.cpp
+++ b/NeoPixelBus.cpp
@@ -76,16 +76,57 @@ void NeoPixelBus::Begin(void)
 #if defined(ESP8266)
 #pragma optimize( "", off )
 
-void send_ws_0_800(uint8_t gpio) {
+void ICACHE_FLASH_ATTR send_ws_0_800(uint32_t pinRegister) {
     uint8_t i;
-    i = 4; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TS_ADDRESS, 1 << gpio);
-    i = 6; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TC_ADDRESS, 1 << gpio);
+
+    i = 4; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TS_ADDRESS, pinRegister);
+    i = 6; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TC_ADDRESS, pinRegister);
 }
 
-void send_ws_1_800(uint8_t gpio) {
+void ICACHE_FLASH_ATTR send_ws_1_800(uint32_t pinRegister) {
     uint8_t i;
-    i = 9; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TS_ADDRESS, 1 << gpio);
-    i = 2; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TC_ADDRESS, 1 << gpio);
+
+    i = 8; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TS_ADDRESS, pinRegister);
+    i = 1; while (i--) GPIO_REG_WRITE(GPIO_OUT_W1TC_ADDRESS, pinRegister);
+}
+
+//void ICACHE_FLASH_ATTR send_subpix_800(uint8_t subpix, uint8_t pin) 
+//{
+//    uint8_t mask;
+//    for (mask = 0x80; mask; mask >>= 1)
+//    {
+//        if (subpix & mask)
+//        {
+//            send_ws_1_800(pin);
+//        }
+//        else
+//        {
+//            send_ws_0_800(pin);
+//        }
+//    }
+//}
+
+void ICACHE_FLASH_ATTR send_pixels_800(uint8_t* pixels, uint8_t* end, uint8_t pin)
+{
+    uint8_t mask;
+    uint8_t subpix;
+    uint32_t pinRegister = _BV(pin);
+
+    while (pixels < end)
+    {
+        subpix = *pixels++;
+        for (mask = 0x80; mask; mask >>= 1)
+        {
+            if (subpix & mask)
+            {
+                send_ws_1_800(pinRegister);
+            }
+            else
+            {
+                send_ws_0_800(pinRegister);
+            }
+        }
+    }
 }
 
 void send_ws_0_400(uint8_t gpio) {
@@ -747,21 +788,11 @@ void NeoPixelBus::Show(void)
     {
 #endif
         // 800 KHz bitstream
-        while (p < end)
-        {
-            uint8_t subpix = *p++;
-            for (uint8_t mask = 0x80; mask; mask >>= 1)
-            {
-                if (subpix & mask)
-                {
-                    send_ws_1_800(_pin);
-                }
-                else
-                {
-                    send_ws_0_800(_pin);
-                }
-            }
-        }
+        send_pixels_800(p, end, _pin);
+        //while (p < end)
+        //{
+        //    send_subpix_800(*p++, _pin);
+        //}
 #ifdef INCLUDE_NEO_KHZ400_SUPPORT
     }
     else
@@ -769,8 +800,8 @@ void NeoPixelBus::Show(void)
         // 400 kHz bitstream
         while (p < end)
         {
-            uint8_t subpix = *p++;
-            for (uint8_t mask = 0x80; mask; mask >>= 1)
+            subpix = *p++;
+            for (mask = 0x80; mask; mask >>= 1)
             {
                 if (subpix & mask)
                 {
diff --git a/NeoPixelBus.h b/NeoPixelBus.h
index 8a75af5..cf61d13 100644
--- a/NeoPixelBus.h
+++ b/NeoPixelBus.h
@@ -15,17 +15,9 @@ You should have received a copy of the GNU Lesser General Public
 License along with NeoPixel.  If not, see
 <http://www.gnu.org/licenses/>.
 --------------------------------------------------------------------*/
+#pragma once
 
-#ifndef NEOPIXELBUS_H
-#define NEOPIXELBUS_H
-
-#if (ARDUINO >= 100)
 #include <Arduino.h>
-#else
-#include <WProgram.h>
-#include <pins_arduino.h>
-#endif
-
 #include "RgbColor.h"
 
 // '_flagsPixels' flags for LED _pixels (third parameter to constructor):
@@ -125,4 +117,3 @@ private:
 
 };
 
-#endif // NEOPIXELBUS_H
diff --git a/RgbColor.h b/RgbColor.h
index 48aa81c..fc3ca1f 100644
--- a/RgbColor.h
+++ b/RgbColor.h
@@ -13,16 +13,9 @@ You should have received a copy of the GNU Lesser General Public
 License along with NeoPixel.  If not, see
 <http://www.gnu.org/licenses/>.
 --------------------------------------------------------------------*/
+#pragma once
 
-#ifndef RGBCOLOR_H
-#define RGBCOLOR_H
-
-#if (ARDUINO >= 100)
 #include <Arduino.h>
-#else
-#include <WProgram.h>
-#include <pins_arduino.h>
-#endif
 
 // ------------------------------------------------------------------------
 // RgbColor represents a color object that is represented by Red, Green, Blue
@@ -95,5 +88,3 @@ struct RgbColor
 	uint8_t B;
 };
 
-
-#endif // RGBCOLOR_H
\ No newline at end of file