failed test of NeoPXL8

platformio.ini

@@ -13,7 +13,7 @@ platform = https://github.com/maxgerhardt/platform-raspberrypi.git
 board = pico
 framework = arduino
 platform_packages = 
-	framework-arduinopico@https://github.com/earlephilhower/arduino-pico.git#lw
+	framework-arduinopico@https://github.com/earlephilhower/arduino-pico.git#master
 board_build.core = earlephilhower
 upload_port = /run/media/amelia/RPI-RP2/
 debug_tool = cmsis-dap
@@ -21,7 +21,6 @@ upload_protocol = cmsis-dap
 monitor_speed = 115200
 board_build.filesystem_size = 1m
 board_build.f_cpu = 133000000L
-; build_flags = -O3
-; board_flags = -DWIFICC=CYW43_COUNTRY_USA
 lib_deps = 
-	https://github.com/FastLED/FastLED#master
+	adafruit/Adafruit NeoPixel@^1.12.0
+	adafruit/Adafruit NeoPXL8@^1.2.6

src/FastLED_RGBW.h

@@ -0,0 +1,52 @@
+/* FastLED_RGBW
+ * 
+ * Hack to enable SK6812 RGBW strips to work with FastLED.
+ *
+ * Original code by Jim Bumgardner (http://krazydad.com).
+ * Modified by David Madison (http://partsnotincluded.com).
+ * 
+*/
+#ifndef FastLED_RGBW_h
+#define FastLED_RGBW_h
+struct CRGBW  {
+	union {
+		struct {
+			union {
+				uint8_t g;
+				uint8_t green;
+			};
+			union {
+				uint8_t r;
+				uint8_t red;
+			};
+			union {
+				uint8_t b;
+				uint8_t blue;
+			};
+			union {
+				uint8_t w;
+				uint8_t white;
+			};
+		};
+		uint8_t raw[4];
+	};
+	CRGBW(){}
+	CRGBW(uint8_t rd, uint8_t grn, uint8_t blu, uint8_t wht){
+		r = rd;
+		g = grn;
+		b = blu;
+		w = wht;
+	}
+	inline void operator = (const CRGB c) __attribute__((always_inline)){ 
+		this->r = c.r;
+		this->g = c.g;
+		this->b = c.b;
+		this->white = 0;
+	}
+};
+inline uint16_t getRGBWsize(uint16_t nleds){
+	uint16_t nbytes = nleds * 4;
+	if(nbytes % 3 > 0) return nbytes / 3 + 1;
+	else return nbytes / 3;
+}
+#endif

src/config.h

@@ -3,19 +3,26 @@
 
 #define DEBUG
 
-// Amount of color channels per pixel - i.e. RGB = 3, RGBW = 4
+#define LIGHTTEST
-#define PIXEL_SIZE 3
+
+// Total LED count PER CHANNEL
+// set to 128 for 4 channel, 170 for 3 channel
+#define MAX_LEDS 170
 
-// Total LED count
+//#define RGBW_MODE
-// set to 128*8 for 4 channel, 170*8 for 3 channel
+// Amount of color channels per pixel - i.e. RGB = 3, RGBW = 4
-#define MAX_LEDS 170*8
+#ifdef RGBW_MODE    
+    #define PIXEL_SIZE 4
+#else
+    #define PIXEL_SIZE 3    
+#endif
 
 // LED driver chip model - depends on strip
 #define LED_TYPE WS2812
 
-// RGB pin ordering - RGB, BGR, GBR, etc
+// RGB pin ordering - NEO_RGB, NEO_BGR, NEO_GBR, etc
-//#define RGB_ORDER GBR
+//#define RGB_ORDER NEO_GBR
-#define RGB_ORDER GRB
+#define RGB_ORDER NEO_GRB
 
 // Max number of LED strips connected
 // 8 for ARGB Controller PCB

src/main.cpp

@@ -6,15 +6,18 @@
 #include <HTTPUpdateServer.h>
 #include <WebServer.h>
 #include "e131.h"
-#include <FastLED.h>
+//#include <Adafruit_NeoPixel.h>
+#include <Adafruit_NeoPXL8.h>
+//#include <FastLED.h>
+//#include "FastLED_RGBW.h"
 #include "config.h"
 #include <EEPROM.h>
 #include <pico/stdlib.h>
 #include <hardware/vreg.h>
 
-int strips[LED_STRIPS] = {170, 170, 170, 170, 170, 170, 170, 170};
+
 // Begin code
-bool core1_separate_stack =  true;
+//bool core1_separate_stack =  true;
 
 #ifdef DEBUG
   #define PRINTFUNC print
@@ -26,8 +29,24 @@ bool core1_separate_stack =  true;
 
 int calculate[LED_STRIPS * 4];
 int universes[LED_STRIPS * 4];
-CRGB ledstrip[MAX_LEDS];
+
-int pins[8];
+int strips[LED_STRIPS] = {MAX_LEDS, MAX_LEDS, MAX_LEDS, MAX_LEDS, MAX_LEDS, MAX_LEDS, MAX_LEDS, MAX_LEDS}; // for compatibility
+int8_t pins[LED_STRIPS] = {STRIP1, STRIP2, STRIP3, STRIP4, STRIP5, STRIP6, STRIP7, STRIP8};
+//const int strips[LED_STRIPS] = {170, 170, 170, 170, 170, 170, 170, 170};
+//int offsets[(LED_STRIPS+1)];
+Adafruit_NeoPXL8 leds(170, pins, RGB_ORDER);
+
+
+// #ifdef RGBW_MODE
+//   // EVIL! hack to support RGBW ICs
+//   CRGBW leds[MAX_LEDS];
+//   CRGB *ledstrip = (CRGB *) &leds[0]; // yes, we just casted a 4-byte value array to a pseudo 3-byte value array
+//   int strips[LED_STRIPS] = {getRGBWsize(128), getRGBWsize(128), getRGBWsize(128), getRGBWsize(128), getRGBWsize(128), getRGBWsize(128), getRGBWsize(128), getRGBWsize(128)};
+// #else
+//   int strips[LED_STRIPS] = {170, 170, 170, 170, 170, 170, 170, 170};
+//   CRGB ledstrip[MAX_LEDS];
+// #endif
+
 
 uint8_t * livedata;
 // Networking
@@ -68,15 +87,27 @@ float airtemparray[TEMP_SAMPLES];
 float cputemp;
 float airtemp;
 int datapos = 0;
-
+#ifdef RGBW_MODE
-
+inline void setpixelrgb(int idx, byte r, byte g, byte b, byte w) {
+    leds.setPixelColor(idx, r, g, b, w);
+}
+#else
+inline void setpixelrgb(int idx, byte r, byte g, byte b) {
+    leds.setPixelColor(idx, r, g, b);
+}
+#endif
+inline void showpixels() {
+  leds.show();
+}
 template <class T> T print(T in) {
+  if(Serial)
     Serial.print(String(millis()/1000.0) + ": " + String(in));
   if(printer) clientbuffer += String(in);
   return (T)true;
 }
 
 template <class T> T println(T in) {
+  if(Serial)
     Serial.println(String(millis()/1000.0) + ": " + String(in));
   if(printer) {
     clientbuffer += String(in);
@@ -183,9 +214,9 @@ void handleForm() {
     if(reboot) {
       EEPROM.commit();
       for (int i = 0; i < MAX_LEDS; i++) {
-        ledstrip[i] = CRGB(0, 0, 0);
+        setpixelrgb(i, 0, 0, 0);
       }
-      FastLED.show();
+      showpixels();
       ready = 3; // trigger core 1 to stop
       delay(250);
       rp2040.reboot();
@@ -215,28 +246,31 @@ void handleNotFound() {
 
 void write_universe(long universe, uint8_t data[], long size) {
   // universe starts at 0
-  /*print("Universe: ");
+  //print("Universe: ");
-  Serial.println(universe);
+  //Serial.println(universe);
-  print("Calculate size: ");
+  //print("Calculate size: ");
-  Serial.println(sizeof(calculate));*/
+  //Serial.println(sizeof(calculate));
   int offset = calculate[universe];
-  /*print("Offset: ");
+  //print("Offset: ");
-  Serial.println(offset);
+  //Serial.println(offset);
-  print("Universes size: ");
+  //print("Universes size: ");
-  Serial.println(sizeof(universes));*/
+  //Serial.println(sizeof(universes));
   int write_size = universes[universe];
   /*print("Length: ");
   Serial.println(write_size * PIXEL_SIZE + (CHANNEL_START - 1) + 2);
   print("Data: ");
   Serial.println(size);*/
-  if (write_size * PIXEL_SIZE + (CHANNEL_START - 1) + 2 > size) {
+  if (write_size * PIXEL_SIZE + (CHANNEL_START - 1) > size) {
     println("Write size too big!!");
+    println(String(write_size * PIXEL_SIZE + (CHANNEL_START - 1)) + " with data size " + String(size));
     return;
   }
-  if(offset + write_size > sizeof(ledstrip)) {
+  /*if(offset + write_size > sizeof(ledstrip)) {
     println("Write size too big!!");
+    println(String(offset + write_size) + " with strip size " + sizeof(ledstrip));
+    
     return;
-  }
+  }*/
   //status = 0;
   for (int i = 0; i < write_size; i++) {
     int j = i * PIXEL_SIZE + (CHANNEL_START - 1);
@@ -248,7 +282,11 @@ void write_universe(long universe, uint8_t data[], long size) {
       Serial.print(data[j+2]);
       Serial.print("  ");
     }*/
-    ledstrip[offset + i] = CRGB(data[j], data[j+1], data[j+2]);
+    #ifdef RGBW_MODE
+      setpixelrgb(offset + i, data[j], data[j+1], data[j+2]);
+    #else
+      setpixelrgb(offset + i, data[j], data[j+1], data[j+2]);
+    #endif
     //ledstrip[strip].setPixelColor(i + offset, data[j], data[j+1], data[j+2]);
   }
   //FastLED.show();
@@ -260,16 +298,19 @@ void write_universe(long universe, uint8_t data[], long size) {
 }
 
 void setup() {
-  //vreg_voltage v = VREG_VOLTAGE_1_20;
+  vreg_voltage v = VREG_VOLTAGE_1_20;
-  //vreg_set_voltage(v);
+  vreg_set_voltage(v);
-  //set_sys_clock_khz(252000, false);
+  set_sys_clock_khz(252000, false);
+  pinMode(23, OUTPUT);
+  pinMode(23, HIGH);
   Serial.begin(115200);
   //rp2040.wdt_begin(8000);
   pinMode(24, INPUT); // VBUS detect - check for USB connection
   if (digitalRead(24)) {
     delay(3000); // Wait for serial
   }
-  pinMode(21, INPUT);
+  
+  pinMode(21, INPUT); // interrupt for W500
   Serial.println("");
   println("Starting RGB Controller...");
   pinMode(20, OUTPUT);
@@ -323,6 +364,7 @@ void setup() {
     EEPROM.commit();
   }
   IP_ADDR = IPAddress(EEPROM.read(0),EEPROM.read(1),EEPROM.read(2),EEPROM.read(3));
+  //IP_ADDR = IPAddress(192,168,5,5);
   if (!IP_ADDR.isSet())
     ETH_MODE = "dhcp";
   else
@@ -473,35 +515,16 @@ void setup1() {
   digitalWrite(LED_BUILTIN, HIGH);
   println("Initializing LED outputs and universe mappings...");
 
-  #ifdef STRIP1
+  
-    pins[0] = STRIP1;
-  #endif
-  #ifdef STRIP2
-    pins[1] = STRIP2;
-  #endif
-  #ifdef STRIP3
-    pins[2] = STRIP3;
-  #endif
-  #ifdef STRIP4
-    pins[3] = STRIP4;
-  #endif
-  #ifdef STRIP5
-    pins[4] = STRIP5;
-  #endif
-  #ifdef STRIP6
-    pins[5] = STRIP6;
-  #endif
-  #ifdef STRIP7
-    pins[6] = STRIP7;
-  #endif
-  #ifdef STRIP8
-    pins[7] = STRIP8;
-  #endif
   // Populate universes and offsets
   int offsetcount = 0;
   int currentsize = 0;
   for (int i = 0; i < LED_STRIPS; i++) {
+    #ifdef RGBW_MODE
+      int tmp = strips[i] * 3 / 4;
+    #else
       int tmp = strips[i];
+    #endif
     
     print("Strip ");
     Serial.print(i);
@@ -519,7 +542,9 @@ void setup1() {
       Serial.print(", Light count ");
       Serial.print(MAX_PIXELS_PER_UNIVERSE);
       Serial.print(", Size ");
-      Serial.println(MAX_PIXELS_PER_UNIVERSE * PIXEL_SIZE);
+      Serial.print(MAX_PIXELS_PER_UNIVERSE * PIXEL_SIZE);
+      Serial.print(", Offset ");
+      Serial.println(calculate[currentsize]);
       offsetcount += MAX_PIXELS_PER_UNIVERSE;
       currentsize += 1;
       tmp -= MAX_PIXELS_PER_UNIVERSE;
@@ -531,49 +556,57 @@ void setup1() {
     Serial.print(", Light count ");
     Serial.print(tmp);
     Serial.print(", Size ");
-    Serial.println(tmp * PIXEL_SIZE);
+    Serial.print(tmp * PIXEL_SIZE);
+    Serial.print(", Offset ");
+    Serial.println(calculate[currentsize]);
     offsetcount += tmp;
     currentsize += 1;
   }       
+  if (!leds.begin(true)) {
+    println("Failure to initialize LEDs!");
+    delay(1000);
+    //rp2040.reboot();
+  } else {
+    println("LED driver initialized.");
+  }
 
-  #ifdef STRIP1
+  leds.setLatchTime(500);
-    FastLED.addLeds<LED_TYPE, STRIP1, RGB_ORDER>(ledstrip, calculate[0], strips[0]);
+
-  #endif
+  for (uint32_t color = 0x440000; color > 0; color >>= 8) {
-  #ifdef STRIP2
+    leds.fill(color);
-    FastLED.addLeds<LED_TYPE, STRIP2, RGB_ORDER>(ledstrip, calculate[1], strips[1]);
+    leds.show();
-  #endif
+    delay(500);
-  #ifdef STRIP3
+  }
-    FastLED.addLeds<LED_TYPE, STRIP3, RGB_ORDER>(ledstrip, calculate[2], strips[2]);
+  
-  #endif
+  for (int i=0; i<8; i++) {
-  #ifdef STRIP4
+    if (pins && (pins[i] < 0)) {
-    FastLED.addLeds<LED_TYPE, STRIP4, RGB_ORDER>(ledstrip, calculate[3], strips[3]);
+      print("No leds on pin ");
-  #endif
+      Serial.println(pins[i]);
-  #ifdef STRIP5
+      continue; // No pixels on this pin
-    FastLED.addLeds<LED_TYPE, STRIP5, RGB_ORDER>(ledstrip, calculate[4], strips[4]);
+    }
-  #endif
+    leds.fill(0);
-  #ifdef STRIP6
+    uint32_t color = 0x0000aa;
-    FastLED.addLeds<LED_TYPE, STRIP6, RGB_ORDER>(ledstrip, calculate[5], strips[5]);
+    leds.fill(color, i * MAX_LEDS, MAX_LEDS);
-  #endif
+    leds.show();
-  #ifdef STRIP7
+    delay(300);
-    FastLED.addLeds<LED_TYPE, STRIP7, RGB_ORDER>(ledstrip, calculate[6], strips[6]);
+  }
-  #endif
-  #ifdef STRIP8
-    FastLED.addLeds<LED_TYPE, STRIP8, RGB_ORDER>(ledstrip, calculate[7], strips[7]);
-  #endif
 
 
   for (int i = 0; i < MAX_LEDS; i++) {
-    ledstrip[i] = CRGB(0, 0, 0);
+    setpixelrgb(i, 0, 0, 0);
   }
-  FastLED.show();
+  showpixels();
+
   // Test all lights
-  /*for (int i = 0; i < MAX_LEDS; i++) {
+  #ifdef LIGHTTEST
-    ledstrip[i] = CRGB(0, 0, 50);
+    for (int i = 0; i < MAX_LEDS; i++) {
-    FastLED.show();
+      setpixelrgb(i, 0, 0, 50);
-    //delay(1);
+      showpixels();
-    ledstrip[i] = CRGB(0, 0, 0);
+      delay(1);
+      setpixelrgb(i, 0, 0, 0);
     }
-  FastLED.show();*/
+    showpixels();
+  #endif
   //delay(3000);
   
   ready += 1;
@@ -586,9 +619,13 @@ void setup1() {
 }
 
 
+
 void loop() {
   /* Parse a packet */
   //println("Start loop");
+  if (millis() % 100 > 50) { // reset LED
+    digitalWrite(LED_BUILTIN, HIGH);
+  }
   
   if(channels = e131.parsePacket()) {
     // Offset by start universe
@@ -625,7 +662,11 @@ void loop() {
 }
 
 void loop1() {
-  rp2040.wdt_reset();
+  //rp2040.wdt_reset();
+  if (millis() % 100 < 50) {
+    //status = 0;
+    digitalWrite(LED_BUILTIN, LOW);
+  }
   if(BOOTSEL) {
     bootsel_count++;
     delay(50);
@@ -654,21 +695,15 @@ void loop1() {
   }
   status2 = 1;
   if(status == 1 && e131.universe > START_UNIVERSE - 1 && channels > 0) {
-    write_universe(e131.universe - START_UNIVERSE, livedata, channels);
+    write_universe(e131.universe - START_UNIVERSE - 1, livedata, channels);
-    FastLED.show();
+    showpixels();
     //println("Done Writing");
     status = 0;
   }
   
   status2 = 0;
   
-  if (millis() % 100 > 50) { // reset LED
+  
-    digitalWrite(LED_BUILTIN, HIGH);
-  }
-  else if (millis() % 100 < 50 && status == 1) {
-    //status = 0;
-    digitalWrite(LED_BUILTIN, LOW);
-  }
   //status = 0;
   //delay(50);
   float cputemp2 = analogReadTemp();