-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WS2812b LEDs with unusal GRB mapping #7
Comments
As a test change the colortype in xdrv_ws2812.ino from
into
and see what happens. |
What a pain. I changed line 34 - and tried that - no change - but then I'm not using pin3 I cannot change 36 because (I think) that feature is not compatible with the bitbang lib (I've hit that before on my code) /home/prh/Sonoff-Tasmota/sonoff/xdrv_ws2812.ino: In function 'void ws2812_init()': I'll try using DMA (pin 3/rx) |
You'll have to change lines 34, 36 , 460 and 462! Compiling works for me... |
Tried on GPIO03 - and still RG are swapped: FTR I also set in user_config.h: 129 #define USE_WS2812 // WS2812 Led string support (+8k code, +1k mem) Thanks Theo. |
OK I'm an idiot - changed all of those occurrences and it does work just fine with RG swapped 👍 |
Fixed in version 3.9.10 just released |
You're amazing - Thanks for such a quick response. Would it be possible to configure on a running system? I have a mix of 2812's and of course most are GRB - just a few RGB (the traditional LED shape ones) and it would be nice to have one firmware image for them all. If that is not possible I totally understand :) |
Looked at it and it's just too much overhead as a new object needs to be defined Makuna/NeoPixelBus#103 |
Oh thats a shame - thanks again. |
Revisit Timers off/on
Is there an easy way to configure an LED string that is mapped weirdly - where Red and Green are swapped?
I was wondering about adding another LedTable.
Apologies if I missed the obvious. I'm on 3.9.7 IIRC.
And again many thanks for an awesome fw!
The text was updated successfully, but these errors were encountered: