diff options
| author | Duncan Laurie <dlaurie@chromium.org> | 2013-08-22 08:25:22 -0700 |
|---|---|---|
| committer | ChromeBot <chrome-bot@google.com> | 2013-08-22 09:09:20 -0700 |
| commit | db2ac4354228c8f908fa28e8a58092d0af1a0e21 (patch) | |
| tree | 22a4993b50dd3276fac2c87b452d8593550f668f | |
| parent | 8b1222a39d80dfa847757789367a64f6ce64580b (diff) | |
| download | chrome-ec-db2ac4354228c8f908fa28e8a58092d0af1a0e21.tar.gz | |
Change peppy battery led error blink timing
(led turn on 1 sec and turn off 3 sec)
BRANCH=peppy
BUG=none
TEST=manual
Disconnect internal battery to check battery led blink timing.
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Conflicts:
common/led_peppy.c
Original-Change-Id: I1417073448505aa5dca2e0e999185b5a9b53410e
Signed-off-by: David Huang <David.Huang@quantatw.com>
Reviewed-on: https://gerrit.chromium.org/gerrit/63891
Reviewed-by: Dave Parker <dparker@chromium.org>
Reviewed-by: Hsu Henry <Henry.Hsu@quantatw.com>
Tested-by: Dave Parker <dparker@chromium.org>
Commit-Queue: Dave Parker <dparker@chromium.org>
(cherry picked from commit 3f1380c624a4ece2f5ea131ca63be39d6559b339)
Change-Id: I21c73cc20482583b3eca1440e1806276aa0f8546
Reviewed-on: https://gerrit.chromium.org/gerrit/66650
Tested-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Commit-Queue: Duncan Laurie <dlaurie@chromium.org>
| -rw-r--r-- | common/led_peppy.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/common/led_peppy.c b/common/led_peppy.c index 50a851b8ee..88cdf7c237 100644 --- a/common/led_peppy.c +++ b/common/led_peppy.c @@ -138,7 +138,8 @@ static void peppy_led_set_battery(int ticks) break; case PWR_STATE_ERROR: peppy_led_set_color_battery( - (ticks & 0x2) ? LED_AMBER : LED_OFF); + (ticks % LED_TOTAL_TICKS < LED_ON_TICKS) ? + LED_AMBER : LED_OFF); break; case PWR_STATE_IDLE: /* External power connected in IDLE. */ if (chflags & CHARGE_FLAG_FORCE_IDLE) |