wxcafe/Model01-Firmware
1
0
Fork 0
Code Releases Activity

Fix turning LEDs off on suspend

Browse Source 
When suspending, we want to set all LEDs off & sync first, and only then pause
LED operations. When the code was originally written, `LEDControl.syncLeds()`
synced LEDs even when `LEDControl.paused` was set - it no longer does so, so we
need to pause after we synced.

This addresses keyboardio/Kaleidoscope#516, at least partially.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
This commit is contained in:
Gergely Nagy 2019-01-20 17:14:07 +01:00
parent cf6234b77e
commit 0095044dd4
No known key found for this signature in database
GPG Key ID: AC1E90BAC433F68F
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Model01-Firmware.ino
View File

@ -355,9 +355,9 @@ static kaleidoscope::plugin::LEDSolidColor solidViolet(130, 0, 120);
void toggleLedsOnSuspendResume(kaleidoscope::plugin::HostPowerManagement::Event event) { void toggleLedsOnSuspendResume(kaleidoscope::plugin::HostPowerManagement::Event event) {
switch (event) { switch (event) {
case kaleidoscope::plugin::HostPowerManagement::Suspend: case kaleidoscope::plugin::HostPowerManagement::Suspend:
LEDControl.paused = true;
LEDControl.set_all_leds_to({0, 0, 0}); LEDControl.set_all_leds_to({0, 0, 0});
LEDControl.syncLeds(); LEDControl.syncLeds();
LEDControl.paused = true;
break; break;
case kaleidoscope::plugin::HostPowerManagement::Resume: case kaleidoscope::plugin::HostPowerManagement::Resume:
LEDControl.paused = false; LEDControl.paused = false;