From 50188a82a6c315cb1851af8b5a98e9f89e97b83b Mon Sep 17 00:00:00 2001 From: Daisuke Nojiri Date: Mon, 16 Jul 2018 16:58:06 -0700 Subject: 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 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 Tested-by: Daisuke Nojiri Reviewed-by: Aaron Durbin --- board/nami/led.c | 18 +++++++++++++++--- 1 file 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 */ @@ -140,6 +138,18 @@ const static led_patterns battery_pattern_2 = { {{LED_WHITE, PULSE_NO}, {LED_WHITE, PULSE_NO}, {LED_WHITE, PULSE_NO}}, }; +/* + * 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 @@ -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; -- cgit v1.2.1