diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2018-07-16 16:58:06 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-07-17 21:52:53 -0700 |
commit | 50188a82a6c315cb1851af8b5a98e9f89e97b83b (patch) | |
tree | 2dca424fd1d3e971bc6d086d91b36fe000e8777e | |
parent | 8d6bbaf73d3271ee3ae74b22e2bf3bd5b8fba3e8 (diff) | |
download | chrome-ec-50188a82a6c315cb1851af8b5a98e9f89e97b83b.tar.gz |
Nami: Add power LED control to Sona
This patch makes EC control the power LED to show power status and
tweaks the battery LED behavior accordingly.
Power LED (single color):
S0: White on
S3: White on 1sec off 1sec
S5: Off
AC In/Battery LED (dual color):
AC attached & battery full White on (S0/S3/S5)
Charging Amber on (S0/S3/S5)
AC not connected & Discharge Off
Battery Error Amber on 0.5 sec, off 0.5 sec
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
BUG=b:74940319
BRANCH=none
TEST=Verify LEDs show expected patterns on Sona for S0/S3/S5 and
full/charge/discharge/error.
Change-Id: I8fc2df6c6437f045b5a48c4a0387c4594c41a9c8
Reviewed-on: https://chromium-review.googlesource.com/1139216
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r-- | board/nami/led.c | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/board/nami/led.c b/board/nami/led.c index 6812e655ee..7dca3c4de2 100644 --- a/board/nami/led.c +++ b/board/nami/led.c @@ -113,12 +113,10 @@ const static led_patterns battery_pattern_0 = { /* * Sona - Battery LED (dual color) - * All patterns are the same as Nami except in S3/S0ix, we do alternate pulsing - * (up-down-off-off). */ const static led_patterns battery_pattern_1 = { /* discharging: s0, s3, s5 */ - {{LED_OFF, PULSE_NO}, {LED_WHITE, ALTERNATE(PULSE(10))}, {LED_OFF, 0}}, + {{LED_OFF, PULSE_NO}, {LED_OFF, PULSE_NO}, {LED_OFF, PULSE_NO}}, /* charging: s0, s3, s5 */ {{LED_AMBER, PULSE_NO}, {LED_AMBER, PULSE_NO}, {LED_AMBER, PULSE_NO}}, /* full: s0, s3, s5 */ @@ -141,6 +139,18 @@ const static led_patterns battery_pattern_2 = { }; /* + * Sona - Power LED (single color) + */ +const static led_patterns power_pattern_1 = { + /* discharging: s0, s3, s5 */ + {{LED_WHITE, PULSE_NO}, {LED_WHITE, BLINK(10)}, {LED_OFF, PULSE_NO}}, + /* charging: s0, s3, s5 */ + {{LED_WHITE, PULSE_NO}, {LED_WHITE, BLINK(10)}, {LED_OFF, PULSE_NO}}, + /* full: s0, s3, s5 */ + {{LED_WHITE, PULSE_NO}, {LED_WHITE, BLINK(10)}, {LED_OFF, PULSE_NO}}, +}; + +/* * Pantheon - Power LED * S0: White on * S3/S0ix: White 1 second on, 3 second off @@ -195,6 +205,8 @@ static void led_init(void) break; case PROJECT_SONA: patterns[0] = &battery_pattern_1; + patterns[1] = &power_pattern_1; + battery_error.pulse = BLINK(5); break; case PROJECT_PANTHEON: patterns[0] = &battery_pattern_2; |