From b979c8ade3c46d67ef739bfe22361b8fc5db6b34 Mon Sep 17 00:00:00 2001 From: lennon chen Date: Tue, 31 Oct 2017 15:46:13 +0800 Subject: [PATCH] Nasher: Modify power led to meet project specification 1.When discharge in S3/ S0ix status, white on for 1 sec off for 1 sec. 2.When battery error, amber on 1sec and off 1sec BUG=b:67923021 BRANCH=none TEST=Check DUT 1.discharge in S3/ S0ix status, power led white on 1 sec and off 1 sec. 2.battery error status, amber on 1sec and off 1sec. Change-Id: I97326fb3fbc6a46ae21436fb62564b0e3d9425cb Signed-off-by: lennon chen Reviewed-on: https://chromium-review.googlesource.com/746506 Commit-Ready: Sean FS Chiang Tested-by: Sean FS Chiang Reviewed-by: Scott Collyer Reviewed-by: Shawn N --- board/coral/led.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/board/coral/led.c b/board/coral/led.c index e196bd8334..91100a3f44 100644 --- a/board/coral/led.c +++ b/board/coral/led.c @@ -107,6 +107,17 @@ static const struct led_descriptor led_robo_state_table[][LED_NUM_PHASES] = { { {LED_COLOR_2, 2 * LED_ONE_SEC}, {LED_COLOR_1, 2 * LED_ONE_SEC} }, }; +static const struct led_descriptor led_nasher_state_table[][LED_NUM_PHASES] = { + { {LED_COLOR_1, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, + { {LED_COLOR_2, LED_INDEFINITE}, {LED_COLOR_1, LED_INDEFINITE} }, + { {LED_COLOR_2, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, + { {LED_COLOR_2, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, + { {LED_COLOR_2, 1 * LED_ONE_SEC}, {LED_OFF, 1 * LED_ONE_SEC} }, + { {LED_OFF, LED_INDEFINITE}, {LED_OFF, LED_INDEFINITE} }, + { {LED_COLOR_1, 1 * LED_ONE_SEC}, {LED_OFF, 1 * LED_ONE_SEC} }, + { {LED_COLOR_1, 2 * LED_ONE_SEC}, {LED_COLOR_2, 2 * LED_ONE_SEC} }, +}; + static struct led_info led; static int led_set_color_battery(enum led_color color) @@ -301,6 +312,10 @@ static void led_init(void) led.charge_lvl_1 = LED_CHARGE_LEVEL_1_ROBO; led.state_table = led_robo_state_table; led.update_power = led_robo_update_power; + } else if (sku >= 160 && sku <= 166) { + led.charge_lvl_1 = LED_CHARGE_LEVEL_1_DEFAULT; + led.state_table = led_nasher_state_table; + led.update_power = NULL; } else { led.charge_lvl_1 = LED_CHARGE_LEVEL_1_DEFAULT; led.state_table = led_default_state_table;