From 3fdf85ece7c14d5fd813783d5fa67e2558ea9ecd Mon Sep 17 00:00:00 2001 From: Duncan Laurie Date: Mon, 26 Jun 2017 23:35:55 -0700 Subject: [PATCH] eve: Check for left/right LED auto control The Eve board defines left and right LEDs, but the LED control loop is checking for auto control to be disabled on the power/battery LED IDs. This is breaking factory testing for LED colors when the battery level is low because it is flashing RED and ignoring the colors that the factory software attempts to set. BUG=b:63049012 BRANCH=eve TEST=manual testing on Eve with low battery Change-Id: I65b822111b3054a5994d71ec4e2a089c487acf97 Signed-off-by: Duncan Laurie Reviewed-on: https://chromium-review.googlesource.com/549740 Reviewed-by: Todd Broch --- board/eve/led.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/board/eve/led.c b/board/eve/led.c index 0687d61da6..f6bc548310 100644 --- a/board/eve/led.c +++ b/board/eve/led.c @@ -348,8 +348,8 @@ static void led_tick(void) if (led_debug == 1) return; - if (led_auto_control_is_enabled(EC_LED_ID_POWER_LED) && - led_auto_control_is_enabled(EC_LED_ID_BATTERY_LED)) { + if (led_auto_control_is_enabled(EC_LED_ID_LEFT_LED) && + led_auto_control_is_enabled(EC_LED_ID_RIGHT_LED)) { eve_led_set_power_battery(); return; }