diff options
author | Scott Collyer <scollyer@google.com> | 2017-11-01 09:04:04 -0700 |
---|---|---|
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2017-11-02 20:17:42 +0000 |
commit | 4cf6e7dd923ea8cd2ae6658bcee179e720341565 (patch) | |
tree | a055bd2c8395e58ef82bfdc9560a9810ef155a2c | |
parent | 58747efef0aa33cd829b0244b90db4ad2c4c6e01 (diff) | |
download | chrome-ec-4cf6e7dd923ea8cd2ae6658bcee179e720341565.tar.gz |
coral: Fix charging LED color for Robo
Robo's LED spec is Red <= 5%, Orange 6 < SOC < 97, Green >=
97%. However, the table had Orange and Green flipped. This CL corrects
that error.
BUG=b:64192049
BRANCH=coral
TEST=Used EC console battfake command to verify that charge LED color
is red until 5%, then orange until 94%, and green after that. Note the
94% limit is due to the define CONFIG_BATTERY_LEVEL_NEAR_FULL which is
set to 94 as that's when the battery will want charging again after
reaching 100%.
Change-Id: Ia8395d6ca28ab000e12fb7a43f13721c7959e35d
Signed-off-by: Scott Collyer <scollyer@google.com>
Reviewed-on: https://chromium-review.googlesource.com/748971
Commit-Ready: Scott Collyer <scollyer@chromium.org>
Tested-by: Scott Collyer <scollyer@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
(cherry picked from commit 030e44309461552a18427ef1924e193bfc660106)
Reviewed-on: https://chromium-review.googlesource.com/751701
Commit-Queue: Scott Collyer <scollyer@chromium.org>
-rw-r--r-- | board/coral/led.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/coral/led.c b/board/coral/led.c index e196bd8334..8ad20613a5 100644 --- a/board/coral/led.c +++ b/board/coral/led.c @@ -98,8 +98,8 @@ static const struct led_descriptor led_default_state_table[][LED_NUM_PHASES] = { /* COLOR_1 = Green, COLOR_2 = Red */ static const struct led_descriptor led_robo_state_table[][LED_NUM_PHASES] = { { {LED_COLOR_2, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, - { {LED_COLOR_1, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, { {LED_COLOR_BOTH, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, + { {LED_COLOR_1, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, { {LED_OFF, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, { {LED_OFF, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, { {LED_OFF, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, |