separate led structures

[ellensp]

Description

Some combinations of LED strips broke compiling the code

In particular

  #define NEOPIXEL_TYPE          NEO_GRB
  #define NEOPIXEL2_TYPE          NEO_GRBW
  #define NEOPIXEL2_SEPARATE

Currently errors with

Marlin\src\feature\leds\leds.cpp:283:97: error: 'const LEDColor' {aka 'const struct LEDColor'} has no member named 'w'
  283 |                             : neo2.Color(incol.r, incol.g, incol.b OPTARG(HAS_WHITE_LED2, incol.w));

The cause is that both led strings use a common struct LEDColor
But LEDColor dynamically changes at compile time to include white support or not. It also only looked at NEOPIXEL_TYPE for a white led.

I separated the two led strings to use their own unique LEDColor that allows either to have a white led.

Requirements

#define NEOPIXEL_TYPE NEO_GRB
#define NEOPIXEL2_TYPE NEO_GRBW
#define NEOPIXEL2_SEPARATE

Benefits

Builds and works as expected

Configurations

Example config for a RAMPS with two LED strips

  #define NEOPIXEL_TYPE          NEO_GRB
  #define NEOPIXEL_PIN                4 // LED driving pin
  #define NEOPIXEL2_TYPE          NEO_GRBW
  #define NEOPIXEL2_PIN               5
  #define NEOPIXEL2_SEPARATE

Example Configuration.zip

Related Issues

[BUG] Cannot successfully build when using separate NEO_RGB and NEO_RGBW Neopixel LEDs #27810

[ellensp]

Although I have tested this lots, there are so many LED options it is possible I missed or broke something.

Check thoroughly