ESP32 x8 Parallel with a RGBW doesn't output the W channel

[Makuna]

Describe the bug
WLED Issue 4380.
The W channel is always empty with i2s parallel, but is present with RMT.

Development environment (please complete the following information):
Pending: determining which NeoPixelBus version is being used as it was noted that previous versions it worked.

[DedeHai]

I tested this on WLED with NPB 2.8.0 as well as 2.8.3, both show the issue on ESP32.
However, testing it on ESP32 S2 with parallel I2S output with 5 outputs, SK6812, 10 LEDs per bus the white channel works.

[Makuna]

At first glance through the changes, I don't see any obvious issues.
The big change was to move to the 3Step Cadence to save memory. It still supports 4Step Cadence, but it requires a compile time flag (NPB_CONF_4STEP_CADENCE). Does WLED use 4 or 3 step?
The interesting thing is that S2 had a ton of changes here, as I discovered that the something in the HW abstraction layer i2s API changed the byte order of the DMA data (probably a bug fix in their abstraction layer for a flag to data layout that I needed to adjust too).

[blazoncek]

@DedeHai S2 code had an issue and was not actually using parallel I2S. Will push modifications so please retest when you can.

@Makuna I do not specifically set cadence, whatever is default.

[blazoncek]

@Makuna I've now finally managed to set up my test unit and can indeed confirm that White channel is missing on SK6812 strip when using parallel I2S (I2S1 on ESP32).

I am using mixed LEDs on the same I2S bus:

• 1st bus 200 WS2812
• 2nd bus 200 WS2812
• 3rd bus 25 SK6812

Buses are created using new and after last bus is created Begin() is called for each bus.
Something like:

bus1 = new NeoPixelBusLg<NeoGrbFeature, X8Ws2812xMethod, NeoGammaNullMethod>(len1, pin1);
bus2 = new NeoPixelBusLg<NeoGrbFeature, X8Ws2812xMethod, NeoGammaNullMethod>(len2, pin2);
bus3 = new NeoPixelBusLg<NeoGrbFeature, X8Sk6812Method, NeoGammaNullMethod>(len3, pin3);
bus1->Begin();
bus2->Begin();
bus3->Begin();

EDIT: Adding

#define NPB_CONF_4STEP_CADENCE
#include "NeoPixelBusLg.h"

has no effect.

[DedeHai]

@blazoncek I retested the S2: 5xSK6812 outputs, 10 LEDs each using your esp-now-search branch. White channel still works.

[blazoncek]

@DedeHai you need 6 or more outputs on S2 (5 or more on S3). You will be able to achieve that by toggling "User parallel I2S" in LED settings.
S2 and S3 will use RMT for first 4 outputs. S2 will use single mode I2S if parallel is not selected for 5th output. You should not be able to configure more than 5 outputs on S2 if you do not select "Use parallel I2S".

[Makuna]

I can't reproduce with Arduino IDE with ESP32 board support v3.1.1

There may be something specific missing in the sketch to trigger the issue. Can we start with this sketch below and modify to get the problem to show.

Sample Sketch:

#include <NeoPixelBusLg.h>

const uint16_t PixelCount = 10; 
const uint8_t DebugPin = 6; // used for logic anaylyser trigger capture 

typedef NeoRgbwFeature NeoFeature;
typedef NeoEsp32I2s0X8Ws2812xMethod NeoMethod;
#define NeoBus NeoPixelBusLg
//#define INCLUDE_ALL_EIGHT 

//typedef NeoEsp32I2s1X8DblWs2812xMethod NeoMethod;


// moved construction into Setup() so we can capture log output into Serial
//
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip0;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip1;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip2;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip3;

#if defined(INCLUDE_ALL_EIGHT)
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip4;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip5;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip6;
NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>* strip7;
#endif

void ResetPinMatrix()
{
  const uint8_t reset_pins[] = {2,4,5,12,13,14,15,16,17,18,19,21,22,23,25,26,27,32,33};
  const uint32_t MATRIX_DETACH_OUT_SIG = 0x100;
  for (uint8_t pin = 0; pin < (sizeof(reset_pins)/sizeof(reset_pins[0])); pin++)
  {
        gpio_matrix_out(reset_pins[pin], MATRIX_DETACH_OUT_SIG, false, false);
  }
}

void setup() 
{
//  ResetPinMatrix();
    Serial.begin(115200);
    while (!Serial); // wait for serial attach
    
    Serial.println();
    Serial.println("Initializing...");
    Serial.flush();

    strip0 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 5);
    strip1 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 4);
    strip2 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 3);
    strip3 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 2);
      
#if defined(INCLUDE_ALL_EIGHT)
    strip4 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 1);
    strip5 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 9);
    strip6 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 8);
    strip7 = new NeoBus<NeoFeature, NeoMethod, NeoGammaNullMethod>(PixelCount, 7);
#endif

    strip0->Begin();
    Serial.println(" 1");
    strip1->Begin();
    Serial.println(" 2");
    strip2->Begin();
    Serial.println(" 3");
    strip3->Begin();
    Serial.println(" 4");


#if defined(INCLUDE_ALL_EIGHT)
    strip4->Begin();
    Serial.println(" 5");
    strip5->Begin();
    Serial.println(" 6");
    strip6->Begin();
    Serial.println(" 7");
    strip7->Begin();
    Serial.println(" 8");
#endif

    pinMode(DebugPin, OUTPUT);
    digitalWrite(DebugPin, LOW);
  
    Serial.println();
    Serial.println("Running...");
}

const RgbwColor testColor = {'R', 'G', 'B', 'W'};

void loop() {
   delay(5000);

    Serial.println("Update ...");

    strip0->SetPixelColor(PixelCount - 1, testColor);
    strip0->SetPixelColor(0, testColor);
    strip1->SetPixelColor(0, testColor);
    strip2->SetPixelColor(0, testColor);
    strip3->SetPixelColor(0, testColor);

#if defined(INCLUDE_ALL_EIGHT)
    strip4->SetPixelColor(0, testColor);
    strip5->SetPixelColor(0, testColor);
    strip6->SetPixelColor(0, testColor);
    strip7->SetPixelColor(0, testColor);
#endif

    digitalWrite(DebugPin, HIGH);
    
    strip0->Show();
    strip1->Show();
    strip2->Show();
    strip3->Show();

#if defined(INCLUDE_ALL_EIGHT)
    strip4->Show();
    strip5->Show();
    strip6->Show();
    strip7->Show();
#endif

    digitalWrite(DebugPin, LOW);
}

results captured:
Front of stream (only showing first three busses in capture)

Back of stream

[blazoncek]

@Makuna I will try to debug more on WLED side and report back as I do not have logic analyser or oscilloscope.

[Makuna]

@Makuna I will try to debug more on WLED side and report back as I do not have logic analyser or oscilloscope.

What versions of the ESP32 core (Arduino board support?) are you using?

[blazoncek]

It looks like a very old one for ESP32.

platform = espressif32@3.5.0
platform_packages = framework-arduinoespressif32 @ https://github.com/Aircoookie/arduino-esp32.git#1.0.6.4

While S2, S3 & C3 are using:

platform = espressif32@ ~6.3.2
platform_packages = platformio/framework-arduinoespressif32 @ 3.20009.0    ;; select arduino-esp32 v2.0.9 (arduino-esp32 2.0.10 thru 2.0.14 are buggy so avoid them)

This is mostly due to image file size since when using newer core/framework we are running out of flash storage.

[Makuna]

My tests above where on a ESP32-S2. It was mentioned it was demonstrated on an S2 also.
Not sure how to get that information for the Arduino ESP32 board support 8-/

It might be IDF v5.3.