diff options
author | Diana Z <dzigterman@chromium.org> | 2021-04-16 15:06:16 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-04-22 22:30:34 +0000 |
commit | 6ada099de9c62d6cc290be071fee4529ed86566c (patch) | |
tree | 52483243492845b3bb93e444060159a593692a67 /board/dalboz | |
parent | 4746791009b75da679d44d45e8487c57c9191329 (diff) | |
download | chrome-ec-6ada099de9c62d6cc290be071fee4529ed86566c.tar.gz |
LED On/Off: Make battery LED optional
Currently, all boards using the LED On/Off module have battery LEDs.
However, if we'd like to expand support to Chromeboxes then the battery
LED must become optional.
BRANCH=None
BUG=b:185508707
TEST=make -j buildall
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: Ieae098829ebe6c8b103f23d5abdbf70e7bcbdf2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2832692
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'board/dalboz')
-rw-r--r-- | board/dalboz/led.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/board/dalboz/led.c b/board/dalboz/led.c index 84630de3ba..53e94e84d6 100644 --- a/board/dalboz/led.c +++ b/board/dalboz/led.c @@ -11,10 +11,11 @@ #define LED_OFF_LVL 1 #define LED_ON_LVL 0 -const int led_charge_lvl_1; -const int led_charge_lvl_2 = 100; +__override const int led_charge_lvl_1; +__override const int led_charge_lvl_2 = 100; -struct led_descriptor led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = { +__override struct led_descriptor + led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = { [STATE_CHARGING_LVL_1] = {{EC_LED_COLOR_GREEN, 2 * LED_ONE_SEC}, {EC_LED_COLOR_RED, 2 * LED_ONE_SEC} }, [STATE_CHARGING_LVL_2] = {{EC_LED_COLOR_RED, LED_INDEFINITE} }, @@ -57,7 +58,7 @@ __override void led_set_color_power(enum ec_led_colors color) gpio_set_level(GPIO_LED3_PWM, LED_OFF_LVL); } -void led_set_color_battery(enum ec_led_colors color) +__override void led_set_color_battery(enum ec_led_colors color) { switch (color) { case EC_LED_COLOR_GREEN: |