From 504111961c412290b2ca78409a66a670cd335bbc Mon Sep 17 00:00:00 2001 From: David Huang Date: Thu, 13 Aug 2020 14:24:33 +0800 Subject: 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 Change-Id: I91fa57f08b2d6033d4153ed6867bb95fb3811173 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2352957 Reviewed-by: Keith Short Commit-Queue: Keith Short --- board/terrador/board.c | 12 +----------- board/terrador/board.h | 1 - board/terrador/led.c | 24 ------------------------ 3 files changed, 1 insertion(+), 36 deletions(-) (limited to 'board/terrador') 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); -- cgit v1.2.1