diff options
| author | Mike M Hsieh <mike.m.hsieh@intel.com> | 2015-10-08 18:33:53 +0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2015-11-30 02:18:45 -0800 |
| commit | 0c4408efdcd37a20c57e74b4b0778e5a477926ac (patch) | |
| tree | fac7d534c8cde203552f5bb7c95bfad5b148752c | |
| parent | 2a09bf95e52d767a6c25cb9270ba92bc5b5d6ca8 (diff) | |
| download | chrome-ec-0c4408efdcd37a20c57e74b4b0778e5a477926ac.tar.gz | |
kunimitsu: Modify battery LED colour
Use one gpio instead of two to show amber colour to indicate charging state
BUG=none
BRANCH=none
TEST=checked and verified LED colour while charging
Signed-off-by: Mike Hsieh <mike.m.hsieh@intel.com>
Change-Id: Id3897eea4213efeea96c3e261f9f43e1b96e8dd0
Reviewed-on: https://chromium-review.googlesource.com/304700
Commit-Ready: Mike M Hsieh <mike.m.hsieh@intel.com>
Tested-by: Mike M Hsieh <mike.m.hsieh@intel.com>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
| -rw-r--r-- | board/kunimitsu/led.c | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/board/kunimitsu/led.c b/board/kunimitsu/led.c index 757d9792e4..058d7e7594 100644 --- a/board/kunimitsu/led.c +++ b/board/kunimitsu/led.c @@ -34,7 +34,6 @@ const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); enum led_color { LED_OFF = 0, LED_BLUE, - LED_PURPLE, LED_AMBER, LED_COLOR_COUNT /* Number of colors, not a color itself */ }; @@ -50,10 +49,6 @@ static int bat_led_set_color(enum led_color color) gpio_set_level(GPIO_BAT_LED_BLUE, BAT_LED_ON); gpio_set_level(GPIO_BAT_LED_AMBER, BAT_LED_OFF); break; - case LED_PURPLE: - gpio_set_level(GPIO_BAT_LED_BLUE, BAT_LED_ON); - gpio_set_level(GPIO_BAT_LED_AMBER, BAT_LED_ON); - break; case LED_AMBER: gpio_set_level(GPIO_BAT_LED_BLUE, BAT_LED_OFF); gpio_set_level(GPIO_BAT_LED_AMBER, BAT_LED_ON); @@ -92,10 +87,7 @@ static int kunimitsu_led_set_color(enum ec_led_id led_id, enum led_color color) int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) { - if (brightness[EC_LED_COLOR_BLUE] != 0 && - brightness[EC_LED_COLOR_AMBER] != 0) - kunimitsu_led_set_color(led_id, LED_PURPLE); - else if (brightness[EC_LED_COLOR_BLUE] != 0) + if (brightness[EC_LED_COLOR_BLUE] != 0) kunimitsu_led_set_color(led_id, LED_BLUE); else if (brightness[EC_LED_COLOR_AMBER] != 0) kunimitsu_led_set_color(led_id, LED_AMBER); @@ -118,19 +110,19 @@ static void kunimitsu_led_set_battery(void) */ switch (charge_get_state()) { case PWR_STATE_CHARGE: - kunimitsu_led_set_color_battery(LED_PURPLE); + kunimitsu_led_set_color_battery(LED_AMBER); break; case PWR_STATE_DISCHARGE: /* Less than 3%, blink one second every two second */ if (charge_get_percent() < CRITICAL_LOW_BATTERY_PERCENTAGE) kunimitsu_led_set_color_battery( (battery_ticks % LED_TOTAL_2SECS_TICKS < - LED_ON_1SEC_TICKS) ? LED_PURPLE : LED_OFF); + LED_ON_1SEC_TICKS) ? LED_AMBER : LED_OFF); /* Less than 10%, blink one second every four seconds */ else if (charge_get_percent() < LOW_BATTERY_PERCENTAGE) kunimitsu_led_set_color_battery( (battery_ticks % LED_TOTAL_4SECS_TICKS < - LED_ON_1SEC_TICKS) ? LED_PURPLE : LED_OFF); + LED_ON_1SEC_TICKS) ? LED_AMBER : LED_OFF); else kunimitsu_led_set_color_battery(LED_OFF); break; |