From a8eea870f54d8e541b3d10eb635a51f59d69a604 Mon Sep 17 00:00:00 2001 From: Benjamin Tissoires Date: Wed, 24 May 2017 06:58:26 +0200 Subject: [PATCH] hidpp20: make sure we check all modes in the switch statement Signed-off-by: Benjamin Tissoires --- src/hidpp20.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/hidpp20.c b/src/hidpp20.c index c3ff14b2..b5460a93 100644 --- a/src/hidpp20.c +++ b/src/hidpp20.c @@ -1899,7 +1899,9 @@ hidpp20_onboard_profile_read_led(struct hidpp20_led *led, uint16_t rate = 0; uint8_t brightness = 0; - switch (internal_led.mode) { + led->mode = (enum hidpp20_led_mode)internal_led.mode; + + switch (led->mode) { case HIDPP20_LED_CYCLE: rate = internal_led.effect.cycle.rate; brightness = internal_led.effect.cycle.brightness; @@ -1908,9 +1910,12 @@ hidpp20_onboard_profile_read_led(struct hidpp20_led *led, rate = internal_led.effect.breath.rate; brightness = internal_led.effect.breath.brightness; break; + case HIDPP20_LED_ON: + break; + case HIDPP20_LED_OFF: + break; } - led->mode = (enum hidpp20_led_mode)internal_led.mode; led->color = internal_led.color; led->rate = rate; led->brightness = brightness;