diff options
author | Diana Z <dzigterman@chromium.org> | 2021-04-09 15:27:39 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-04-13 06:36:17 +0000 |
commit | 76cd38a0b214bdbfd45ffa7c3633329a8eab65ab (patch) | |
tree | bb822a8d761ffd5494d66e6bc1e9d691794f7c92 /board/guybrush | |
parent | e764f94a625fb4aeac3ed92f42a2b5085fb19a14 (diff) | |
download | chrome-ec-76cd38a0b214bdbfd45ffa7c3633329a8eab65ab.tar.gz |
Guybrush: Add LED behavior
Use the common LED on/off states for LED behavior. Set it up for our
two colors: amber and white.
BRANCH=None
BUG=b:184845299
TEST=on guybrush, confirm charging shows Amber, discharging is white,
shutdown shows no LED on
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: Idc57de843fd2addaaa3006d4abd8566d76b2de6b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2818151
Reviewed-by: Rob Barnes <robbarnes@google.com>
Commit-Queue: Rob Barnes <robbarnes@google.com>
Diffstat (limited to 'board/guybrush')
-rw-r--r-- | board/guybrush/board.h | 4 | ||||
-rw-r--r-- | board/guybrush/build.mk | 2 | ||||
-rw-r--r-- | board/guybrush/led.c | 90 |
3 files changed, 95 insertions, 1 deletions
diff --git a/board/guybrush/board.h b/board/guybrush/board.h index 93b718e198..ae6d9fd1cd 100644 --- a/board/guybrush/board.h +++ b/board/guybrush/board.h @@ -37,6 +37,10 @@ /* Fan features */ +/* LED features */ +#define CONFIG_LED_COMMON +#define CONFIG_LED_ONOFF_STATES + #ifndef __ASSEMBLER__ #include "gpio_signal.h" diff --git a/board/guybrush/build.mk b/board/guybrush/build.mk index 1e79b1895e..4324e2b034 100644 --- a/board/guybrush/build.mk +++ b/board/guybrush/build.mk @@ -9,4 +9,4 @@ BASEBOARD:=guybrush board-y=board.o -board-y+=board_fw_config.o +board-y+=board_fw_config.o led.o diff --git a/board/guybrush/led.c b/board/guybrush/led.c new file mode 100644 index 0000000000..d82070cdaa --- /dev/null +++ b/board/guybrush/led.c @@ -0,0 +1,90 @@ +/* Copyright 2021 The Chromium OS Authors. All rights reserved. + * Use of this source code is governed by a BSD-style license that can be + * found in the LICENSE file. + * + * Guybrush specific PWM LED settings. + */ + +#include "common.h" +#include "led_onoff_states.h" +#include "led_common.h" +#include "gpio.h" +#include "pwm.h" + +/* Note PWM LEDs are active low */ +#define LED_OFF_LVL 1 +#define LED_ON_LVL 0 + +#define CPRINTS(format, args...) cprints(CC_PWM, format, ## args) + +const int led_charge_lvl_1 = 5; + +const int led_charge_lvl_2 = 97; + +struct led_descriptor led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = { + [STATE_CHARGING_LVL_1] = {{EC_LED_COLOR_AMBER, LED_INDEFINITE} }, + [STATE_CHARGING_LVL_2] = {{EC_LED_COLOR_AMBER, LED_INDEFINITE} }, + [STATE_CHARGING_FULL_CHARGE] = {{EC_LED_COLOR_WHITE, LED_INDEFINITE} }, + [STATE_DISCHARGE_S0] = {{EC_LED_COLOR_WHITE, LED_INDEFINITE} }, + [STATE_DISCHARGE_S3] = {{EC_LED_COLOR_WHITE, 1 * LED_ONE_SEC}, + {LED_OFF, 1 * LED_ONE_SEC} }, + [STATE_DISCHARGE_S5] = {{LED_OFF, LED_INDEFINITE} }, + [STATE_BATTERY_ERROR] = {{EC_LED_COLOR_AMBER, 1 * LED_ONE_SEC}, + {LED_OFF, 1 * LED_ONE_SEC} }, + [STATE_FACTORY_TEST] = {{EC_LED_COLOR_AMBER, 2 * LED_ONE_SEC}, + {EC_LED_COLOR_WHITE, 2 * LED_ONE_SEC} }, +}; + +const enum ec_led_id supported_led_ids[] = { + EC_LED_ID_BATTERY_LED, +}; + +const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); + +void led_set_color_battery(enum ec_led_colors color) +{ + switch (color) { + case EC_LED_COLOR_AMBER: + pwm_enable(PWM_CH_LED_CHRG, LED_ON_LVL); + pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL); + break; + case EC_LED_COLOR_WHITE: + pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL); + pwm_enable(PWM_CH_LED_FULL, LED_ON_LVL); + break; + case LED_OFF: + pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL); + pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL); + break; + default: /* Unsupported colors */ + CPRINTS("Unsupported LED color: %d", color); + pwm_enable(PWM_CH_LED_CHRG, LED_OFF_LVL); + pwm_enable(PWM_CH_LED_FULL, LED_OFF_LVL); + break; + } +} + +void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range) +{ + if (led_id == EC_LED_ID_BATTERY_LED) { + brightness_range[EC_LED_COLOR_AMBER] = 1; + brightness_range[EC_LED_COLOR_WHITE] = 1; + } +} + +int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) +{ + if (led_id == EC_LED_ID_BATTERY_LED) { + if (brightness[EC_LED_COLOR_WHITE] != 0) + led_set_color_battery(EC_LED_COLOR_WHITE); + else if (brightness[EC_LED_COLOR_AMBER] != 0) + led_set_color_battery(EC_LED_COLOR_AMBER); + else + led_set_color_battery(LED_OFF); + } else { + CPRINTS("Unsupported LED set: %d", led_id); + return EC_ERROR_INVAL; + } + + return EC_SUCCESS; +} |