From a5d01642de0e16596ccff0a52d07431210190358 Mon Sep 17 00:00:00 2001 From: Keith Short Date: Wed, 4 Nov 2020 12:55:18 -0700 Subject: voema: remove board ID 0 checks Board ID 0 checks were only used on the volteer reference board. BUG=b:149858568 BRANCH=firmware-volteer-13521.B-master TEST=make buildall Signed-off-by: Keith Short Change-Id: Id55e4ca49aa584afe0ba74aec65cba0ef5c51ec9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2519964 Reviewed-by: Abe Levkoy Commit-Queue: Abe Levkoy --- board/voema/board.c | 16 ++-------------- board/voema/gpio.inc | 16 ++++------------ 2 files changed, 6 insertions(+), 26 deletions(-) diff --git a/board/voema/board.c b/board/voema/board.c index 1993b241fb..32d7eaadb8 100644 --- a/board/voema/board.c +++ b/board/voema/board.c @@ -358,16 +358,14 @@ const int usb_port_enable[USB_PORT_COUNT] = { GPIO_EN_PP5000_USBA, }; -static enum gpio_signal ps8xxx_rst_odl = GPIO_USB_C1_RT_RST_ODL; - static void ps8815_reset(void) { int val; - gpio_set_level(ps8xxx_rst_odl, 0); + gpio_set_level(GPIO_USB_C1_RT_RST_ODL, 0); msleep(GENERIC_MAX(PS8XXX_RESET_DELAY_MS, PS8815_PWR_H_RST_H_DELAY_MS)); - gpio_set_level(ps8xxx_rst_odl, 1); + gpio_set_level(GPIO_USB_C1_RT_RST_ODL, 1); msleep(PS8815_FW_INIT_DELAY_MS); /* @@ -458,16 +456,6 @@ __override void board_cbi_init(void) { enum ec_cfg_usb_db_type usb_db = ec_cfg_usb_db_type(); - /* Reconfigure Volteer GPIOs based on the board ID */ - if (get_board_id() == 0) { - CPRINTS("Configuring GPIOs for board ID 0"); - CPRINTS("VOLUME_UP button disabled"); - - /* Reassign USB_C1_RT_RST_ODL */ - bb_controls[USBC_PORT_C1].retimer_rst_gpio = - GPIO_USB_C1_RT_RST_ODL_BOARDID_0; - ps8xxx_rst_odl = GPIO_USB_C1_RT_RST_ODL_BOARDID_0; - } config_port_discrete_tcpc(0); switch (usb_db) { case DB_USB_ABSENT: diff --git a/board/voema/gpio.inc b/board/voema/gpio.inc index ab088e81c7..eb9cd12b30 100644 --- a/board/voema/gpio.inc +++ b/board/voema/gpio.inc @@ -84,19 +84,11 @@ GPIO(EC_PCH_INT_ODL, PIN(B, 0), GPIO_ODR_HIGH) /* USB and USBC Signals */ GPIO(EN_PP5000_USBA, PIN(C, 6), GPIO_OUT_LOW) /* Enable USB-A charging - all ports */ GPIO(USB_A_LOW_PWR_OD, PIN(6, 6), GPIO_ODR_LOW) /* Don't limit USB-A charging by default - all ports */ - -/* - * USB_C1 moved from GPIO32 to GPIO83 on boards with board ID >=1. - * GPIO83/EN_PP1800_A is DNS on board ID 0 and GPIO32 is N/C on board ID >=1 - * so it's safe to define GPIOs compatible with both designs. - * TODO (b/149858568): remove board ID=0 support. - */ -GPIO(USB_C1_RT_RST_ODL_BOARDID_0, PIN(3, 2), GPIO_ODR_LOW) /* USB_C1 Reset on boards without board ID */ -GPIO(USB_C1_RT_RST_ODL, PIN(8, 3), GPIO_ODR_LOW) /* USB_C1 Reset on boards board ID >=1 */ -GPIO(USB_C0_OC_ODL, PIN(B, 1), GPIO_ODR_HIGH) -GPIO(USB_C1_OC_ODL, PIN(5, 0), GPIO_ODR_HIGH) +GPIO(USB_C1_RT_RST_ODL, PIN(8, 3), GPIO_ODR_LOW) +GPIO(USB_C0_OC_ODL, PIN(B, 1), GPIO_ODR_HIGH) +GPIO(USB_C1_OC_ODL, PIN(5, 0), GPIO_ODR_HIGH) /* There is currently no need to service this interrupt. */ -GPIO(USB_C1_RT_INT_ODL, PIN(F, 3), GPIO_INPUT) +GPIO(USB_C1_RT_INT_ODL, PIN(F, 3), GPIO_INPUT) /* Don't have a load switch for retimer */ UNIMPLEMENTED(USB_C1_LS_EN) -- cgit v1.2.1