From 0095044dd4af6e2fb3be5e60ffe648c4cb9b94d5 Mon Sep 17 00:00:00 2001 From: Gergely Nagy Date: Sun, 20 Jan 2019 17:14:07 +0100 Subject: Fix turning LEDs off on suspend 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 --- Model01-Firmware.ino | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Model01-Firmware.ino b/Model01-Firmware.ino index 58b3ebe..0178726 100644 --- a/Model01-Firmware.ino +++ b/Model01-Firmware.ino @@ -355,9 +355,9 @@ static kaleidoscope::plugin::LEDSolidColor solidViolet(130, 0, 120); void toggleLedsOnSuspendResume(kaleidoscope::plugin::HostPowerManagement::Event event) { switch (event) { case kaleidoscope::plugin::HostPowerManagement::Suspend: - LEDControl.paused = true; LEDControl.set_all_leds_to({0, 0, 0}); LEDControl.syncLeds(); + LEDControl.paused = true; break; case kaleidoscope::plugin::HostPowerManagement::Resume: LEDControl.paused = false; -- cgit v1.2.3