diff options
author | Lu Zhang <lu.zhang@bitland.corp-partner.google.com> | 2020-04-14 15:26:07 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-04-15 07:12:14 +0000 |
commit | c9d9c37e3e29b1833ed6e67d74d216eaaecfd2e4 (patch) | |
tree | 09f33c92a17a26eaa3e6a72498fded62bf5947d8 /board | |
parent | db664eaccf085e83e8495063655ac9cf871d8e29 (diff) | |
download | chrome-ec-c9d9c37e3e29b1833ed6e67d74d216eaaecfd2e4.tar.gz |
dalboz: add power led support
dalboz has a power led.
BRANCH=none
BUG=none
TEST=S0: on
S3: blink
S5: off
Signed-off-by: Lu Zhang <lu.zhang@bitland.corp-partner.google.com>
Change-Id: Ic63f18cdf3ce697d2a1b612629c0c6067932e1d9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2147767
Reviewed-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'board')
-rw-r--r-- | board/dalboz/board.h | 3 | ||||
-rw-r--r-- | board/dalboz/gpio.inc | 2 | ||||
-rw-r--r-- | board/dalboz/led.c | 53 |
3 files changed, 48 insertions, 10 deletions
diff --git a/board/dalboz/board.h b/board/dalboz/board.h index 38d409fc5b..2a9db57ccc 100644 --- a/board/dalboz/board.h +++ b/board/dalboz/board.h @@ -22,6 +22,9 @@ #define CONFIG_MKBP_USE_GPIO +/* Power LEDs */ +#define CONFIG_LED_POWER_LED + /* Motion sensing drivers */ #define CONFIG_ACCELGYRO_LSM6DSM #define CONFIG_ACCEL_LSM6DSM_INT_EVENT \ diff --git a/board/dalboz/gpio.inc b/board/dalboz/gpio.inc index d10d151f74..f55565866e 100644 --- a/board/dalboz/gpio.inc +++ b/board/dalboz/gpio.inc @@ -52,6 +52,7 @@ UNIMPLEMENTED(PCH_SMI_L) GPIO(LED_FULL_L, PIN(6, 0), GPIO_OUT_HIGH) GPIO(LED_CHRG_L, PIN(C, 0), GPIO_OUT_HIGH) +GPIO(LED3_PWM, PIN(C, 3), GPIO_OUT_HIGH) /* * Dalboz has 2 DB options, with different IO expanders. IOEX_C1_NCT3807 is the @@ -120,7 +121,6 @@ ALTERNATE(PIN_MASK(B, BIT(2) | BIT(3)), 0, MODULE_I2C, 0) /* I2C7 */ ALTERNATE(PIN_MASK(4, BIT(2) | BIT(3)), 0, MODULE_ADC, 0) /* ADC2, ADC3 Temp Sensors */ -ALTERNATE(PIN_MASK(C, BIT(3)), 0, MODULE_PWM, 0) /* PWM0 LED */ ALTERNATE(PIN_MASK(8, BIT(0)), 0, MODULE_PWM, 0) /* PWM3 KB Backlight */ ALTERNATE(PIN_MASK(3, 0x03), 0, MODULE_KEYBOARD_SCAN, GPIO_INPUT) /* KSI_00-01 */ diff --git a/board/dalboz/led.c b/board/dalboz/led.c index 9f3f7492c0..6ef9c26e8c 100644 --- a/board/dalboz/led.c +++ b/board/dalboz/led.c @@ -27,12 +27,36 @@ struct led_descriptor led_bat_state_table[LED_NUM_STATES][LED_NUM_PHASES] = { {EC_LED_COLOR_RED, 2 * LED_ONE_SEC} }, [STATE_FACTORY_TEST] = {{EC_LED_COLOR_GREEN, LED_INDEFINITE} }, }; + BUILD_ASSERT(ARRAY_SIZE(led_bat_state_table) == LED_NUM_STATES); -const enum ec_led_id supported_led_ids[] = { EC_LED_ID_BATTERY_LED }; +const struct led_descriptor + led_pwr_state_table[PWR_LED_NUM_STATES][LED_NUM_PHASES] = { + [PWR_LED_STATE_ON] = {{EC_LED_COLOR_WHITE, LED_INDEFINITE} }, + [PWR_LED_STATE_SUSPEND_AC] = {{EC_LED_COLOR_WHITE, 3 * LED_ONE_SEC}, + {LED_OFF, 0.5 * LED_ONE_SEC} }, + [PWR_LED_STATE_SUSPEND_NO_AC] = {{LED_OFF, LED_INDEFINITE} }, + [PWR_LED_STATE_OFF] = {{LED_OFF, LED_INDEFINITE} }, +}; + +BUILD_ASSERT(ARRAY_SIZE(led_pwr_state_table) == PWR_LED_NUM_STATES); + +const enum ec_led_id supported_led_ids[] = { + EC_LED_ID_BATTERY_LED, + EC_LED_ID_POWER_LED +}; const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); +void led_set_color_power(enum ec_led_colors color) +{ + if (color == EC_LED_COLOR_WHITE) + gpio_set_level(GPIO_LED3_PWM, LED_ON_LVL); + else + /* LED_OFF and unsupported colors */ + gpio_set_level(GPIO_LED3_PWM, LED_OFF_LVL); +} + void led_set_color_battery(enum ec_led_colors color) { switch (color) { @@ -53,18 +77,29 @@ void led_set_color_battery(enum ec_led_colors color) void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range) { - brightness_range[EC_LED_COLOR_GREEN] = 1; - brightness_range[EC_LED_COLOR_RED] = 1; + if (led_id == EC_LED_ID_BATTERY_LED) { + brightness_range[EC_LED_COLOR_GREEN] = 1; + brightness_range[EC_LED_COLOR_RED] = 1; + } else if (led_id == EC_LED_ID_POWER_LED) { + brightness_range[EC_LED_COLOR_WHITE] = 1; + } } int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) { - if (brightness[EC_LED_COLOR_GREEN] != 0) - led_set_color_battery(EC_LED_COLOR_GREEN); - else if (brightness[EC_LED_COLOR_RED] != 0) - led_set_color_battery(EC_LED_COLOR_RED); - else - led_set_color_battery(LED_OFF); + if (led_id == EC_LED_ID_BATTERY_LED) { + if (brightness[EC_LED_COLOR_GREEN] != 0) + led_set_color_battery(EC_LED_COLOR_GREEN); + else if (brightness[EC_LED_COLOR_RED] != 0) + led_set_color_battery(EC_LED_COLOR_RED); + else + led_set_color_battery(LED_OFF); + } else if (led_id == EC_LED_ID_POWER_LED) { + if (brightness[EC_LED_COLOR_WHITE] != 0) + led_set_color_power(EC_LED_COLOR_WHITE); + else + led_set_color_power(LED_OFF); + } return EC_SUCCESS; } |