Skip to content
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

Impossible to set correct HUE for leds #7

Open
Fishwaldo opened this issue Jan 4, 2025 · 1 comment
Open

Impossible to set correct HUE for leds #7

Fishwaldo opened this issue Jan 4, 2025 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@Fishwaldo
Copy link

Due to this line (and the case statement for the LOW byte) its impossible to set up the correct HUE for the leds. We are OR'ing the incoming value with the existing value, hence you can't set a new HUE value

KEY_WS2812B_Hue = (((uint16_t)(IIC_Slave_Receive_Struct.Data[0] & B00000001)) << 8) | KEY_WS2812B_Hue;

(I'm assuming this is the factory firmware thats loaded on the STM)

Due to the non-standard fine pitched JST headers for SWD programing, I can't test a fix.

Llgok added a commit that referenced this issue Jan 23, 2025
@Llgok
Copy link
Contributor

Llgok commented Jan 23, 2025

@Fishwaldo
Thank you very much for your testing! After investigating, I found that the "KEY_WS2812B_Hue" value indeed could not be modified. I have fixed this issue and included the updated firmware in the firmware package. You may need to disassemble the device and use a CH340 serial port converter or an STLink programmer to flash the latest updated files. I sincerely apologize for this error!

@Llgok Llgok self-assigned this Jan 23, 2025
@Llgok Llgok added the bug Something isn't working label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants