[FR] Neopixel support (Mega S) #272

Closed
opened 2022-02-02 06:16:34 -06:00 by ignotus666 · 5 comments
ignotus666 commented 2022-02-02 06:16:34 -06:00 (Migrated from github.com)

First of all, thank you so much for all the work put into this firmware, it is much appreciated.

Bug Description

Enabling #define NEOPIXEL_LED in Configuration.h leads to compilation errors.

Steps to Reproduce

  1. Enable #define NEOPIXEL_LED in Configuration.h, set pin, No. of LEDs, etc.
  2. Build in Platformio

Expected behavior: I'm aware that there's no menu support for configuring Neopixels but I expected the default behaviour.

Actual behavior: Compilation fails.

Additional Information

I know it's not a bug but rather an intentionally disabled feature, but I'd love to be able to stick a few Neopixel LEDs around the hotend - the default behaviour is fine and I can edit colours, etc, in the firmware if needed. I just need some pointers as to why it won't compile...

First of all, thank you so much for all the work put into this firmware, it is much appreciated. ### Bug Description Enabling `#define NEOPIXEL_LED` in Configuration.h leads to compilation errors. ### Steps to Reproduce <!-- Please describe the steps needed to reproduce the issue --> 1. Enable `#define NEOPIXEL_LED` in Configuration.h, set pin, No. of LEDs, etc. 2. Build in Platformio **Expected behavior:** I'm aware that there's no menu support for configuring Neopixels but I expected the default behaviour. **Actual behavior:** Compilation fails. #### Additional Information I know it's not a bug but rather an intentionally disabled feature, but I'd love to be able to stick a few Neopixel LEDs around the hotend - the default behaviour is fine and I can edit colours, etc, in the firmware if needed. I just need some pointers as to why it won't compile...
ignotus666 commented 2022-02-02 09:50:36 -06:00 (Migrated from github.com)

Ok, made some progress after finding another similar issue here. If I comment out:

#define PRINTER_EVENT_LEDS in Configuration.h

and

-<src/feature/leds/leds.cpp>
-<src/feature/leds/neopixel.cpp>

in platformio.ini, it compiles, but I would really like the PRINTER_EVENT_LEDS to work... But then again I suppose I could get the same function using the M150 command in my start and end code, couldn't I?

Ok, made some progress after finding another similar issue here. If I comment out: `#define PRINTER_EVENT_LEDS` in Configuration.h and `-<src/feature/leds/leds.cpp>` `-<src/feature/leds/neopixel.cpp>` in platformio.ini, it compiles, but I would really like the `PRINTER_EVENT_LEDS` to work... But then again I suppose I could get the same function using the M150 command in my start and end code, couldn't I?
ignotus666 commented 2022-02-03 01:29:55 -06:00 (Migrated from github.com)

Nothing like a good night's sleep to come back and see the blindingly obvious... So, for anyone else running into this problem here's the summary:

Enable #define NEOPIXEL_LED and #define PRINTER_EVENT_LEDS, set controller pin, number of LEDs, etc. in Configuration.h.

In platformio.ini, comment out at lines ~ 174-178 (add a # at the start of the line):

-<src/feature/leds/leds.cpp>
-<src/feature/leds/neopixel.cpp>
-<src/feature/leds/printer_event_leds.cpp>

Now it should compile. I haven't actually tested it with Neopixels but hopefully they'll work.

Nothing like a good night's sleep to come back and see the blindingly obvious... So, for anyone else running into this problem here's the summary: Enable `#define NEOPIXEL_LED` and `#define PRINTER_EVENT_LEDS`, set controller pin, number of LEDs, etc. in Configuration.h. In platformio.ini, comment out at lines ~ 174-178 (add a # at the start of the line): `-<src/feature/leds/leds.cpp>` `-<src/feature/leds/neopixel.cpp>` `-<src/feature/leds/printer_event_leds.cpp>` Now it should compile. I haven't actually tested it with Neopixels but hopefully they'll work.
ignotus666 commented 2022-02-03 03:54:36 -06:00 (Migrated from github.com)

Just an update to confirm it works :) Used the servo header that's already there (not using BLtouch or anything) providing a digital pin (D11), gnd and 5v. I'll only be using 3 pixels so I don't think I'll fry anything.

Just an update to confirm it works :) Used the servo header that's already there (not using BLtouch or anything) providing a digital pin (D11), gnd and 5v. I'll only be using 3 pixels so I don't think I'll fry anything.
ignotus666 commented 2022-02-03 06:41:38 -06:00 (Migrated from github.com)

Sorry - last update ;)

I forgot to mention that -<src/gcode/feature/leds/M150.cpp> in ~ line 238 of platformio.ini should also be commented out.

Sorry - last update ;) I forgot to mention that `-<src/gcode/feature/leds/M150.cpp>` in ~ line 238 of platformio.ini should also be commented out.
github-actions[bot] commented 2022-04-04 09:03:50 -05:00 (Migrated from github.com)

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wp/Marlin-2-0-x-Anycubic-i3-MEGA-S#272
No description provided.