diff options
| author | David Huang <david.huang@quanta.corp-partner.google.com> | 2020-08-13 14:24:33 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-08-17 20:48:34 +0000 |
| commit | 504111961c412290b2ca78409a66a670cd335bbc (patch) | |
| tree | ba0470af8ec331a89e5c6b367adc999cfe88ae53 | |
| parent | 14c16ab26538d3e95b2b734ffaf15b49fd9caca8 (diff) | |
| download | chrome-ec-504111961c412290b2ca78409a66a670cd335bbc.tar.gz | |
terrador: Remove LED_SIDESEL_4_L function and config.
Remove LED_SIDESEL_4_L function and config.
BUG=b:163167407
BRANCH=master
TEST=make buildall success.
Signed-off-by: David Huang <david.huang@quanta.corp-partner.google.com>
Change-Id: I91fa57f08b2d6033d4153ed6867bb95fb3811173
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2352957
Reviewed-by: Keith Short <keithshort@chromium.org>
Commit-Queue: Keith Short <keithshort@chromium.org>
| -rw-r--r-- | board/terrador/board.c | 12 | ||||
| -rw-r--r-- | board/terrador/board.h | 1 | ||||
| -rw-r--r-- | board/terrador/led.c | 24 |
3 files changed, 1 insertions, 36 deletions
diff --git a/board/terrador/board.c b/board/terrador/board.c index dcd0831409..92dd58187f 100644 --- a/board/terrador/board.c +++ b/board/terrador/board.c @@ -48,9 +48,7 @@ union volteer_cbi_fw_config fw_config_defaults = { static void board_init(void) { - /* Illuminate motherboard and daughter board LEDs equally to start. */ - pwm_enable(PWM_CH_LED4_SIDESEL, 1); - pwm_set_duty(PWM_CH_LED4_SIDESEL, 50); + } DECLARE_HOOK(HOOK_INIT, board_init, HOOK_PRIO_DEFAULT); @@ -127,14 +125,6 @@ const struct pwm_t pwm_channels[] = { .flags = PWM_CONFIG_ACTIVE_LOW | PWM_CONFIG_DSLEEP, .freq = 2400, }, - [PWM_CH_LED4_SIDESEL] = { - .channel = 7, - .flags = PWM_CONFIG_ACTIVE_LOW | PWM_CONFIG_DSLEEP, - /* Run at a higher frequency than the color PWM signals to avoid - * timing-based color shifts. - */ - .freq = 4800, - }, [PWM_CH_KBLIGHT] = { .channel = 3, .flags = 0, diff --git a/board/terrador/board.h b/board/terrador/board.h index 051630386d..85b97a0b11 100644 --- a/board/terrador/board.h +++ b/board/terrador/board.h @@ -161,7 +161,6 @@ enum pwm_channel { PWM_CH_LED1_BLUE = 0, PWM_CH_LED2_GREEN, PWM_CH_LED3_RED, - PWM_CH_LED4_SIDESEL, PWM_CH_KBLIGHT, PWM_CH_COUNT }; diff --git a/board/terrador/led.c b/board/terrador/led.c index 735df935d2..1608219f28 100644 --- a/board/terrador/led.c +++ b/board/terrador/led.c @@ -77,27 +77,3 @@ int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) return EC_SUCCESS; } - -/* Illuminates the LED on the side of the active charging port. If not charging, - * illuminates both LEDs. - */ -static void led_set_charge_port_tick(void) -{ - int port; - int side_select_duty; - - port = charge_manager_get_active_charge_port(); - switch (port) { - case 0: - side_select_duty = 100; - break; - case 1: - side_select_duty = 0; - break; - default: - side_select_duty = 50; - } - - pwm_set_duty(PWM_CH_LED4_SIDESEL, side_select_duty); -} -DECLARE_HOOK(HOOK_TICK, led_set_charge_port_tick, HOOK_PRIO_DEFAULT); |