diff options
| author | Duncan Laurie <dlaurie@google.com> | 2017-06-26 23:35:55 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-06-27 10:55:48 -0700 |
| commit | 3fdf85ece7c14d5fd813783d5fa67e2558ea9ecd (patch) | |
| tree | c888e514a345b52f93723cc77c59655d02da94bb | |
| parent | 61c8096e5c1e35ef134dec79b8736067c5b689dc (diff) | |
| download | chrome-ec-3fdf85ece7c14d5fd813783d5fa67e2558ea9ecd.tar.gz | |
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 <dlaurie@google.com>
Reviewed-on: https://chromium-review.googlesource.com/549740
Reviewed-by: Todd Broch <tbroch@chromium.org>
| -rw-r--r-- | board/eve/led.c | 4 |
1 files 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; } |