summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorScott Collyer <scollyer@google.com>2017-11-01 09:04:04 -0700
committerChromeOS Commit Bot <chromeos-commit-bot@chromium.org>2017-11-02 20:17:42 +0000
commit4cf6e7dd923ea8cd2ae6658bcee179e720341565 (patch)
treea055bd2c8395e58ef82bfdc9560a9810ef155a2c
parent58747efef0aa33cd829b0244b90db4ad2c4c6e01 (diff)
downloadchrome-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.c2
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} },