diff options
author | Logan_Liao <logan_Liao@compal.corp-partner.google.com> | 2022-02-22 08:12:05 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2022-02-23 14:30:25 +0000 |
commit | b88d5765c4f86003a317c9bb4ffb0a8991c0a885 (patch) | |
tree | 2069e5a6db6faf033a1a97502e13122609ac16a2 /board/banshee/usbc_config.c | |
parent | 10bf02bbae296df6029976a397f26526ba5b9e2a (diff) | |
download | chrome-ec-b88d5765c4f86003a317c9bb4ffb0a8991c0a885.tar.gz |
Banshee : Correct IOEX wrong calling, PD, Charger setting.stabilize-14532.B-main
This patch correct IOEX to use ioex_set_level function, modify PD and
Charger setting.
BUG=b:214871796
BRANCH=brya
TEST=bring up success on Proto board.
Change-Id: Icb1d96cc20faff96476785cfc5cb3cf1b20877bc
Signed-off-by: Logan_Liao <logan_Liao@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3478977
Reviewed-by: Logan Liao <logan_liao@compal.corp-partner.google.com>
Tested-by: Logan Liao <logan_liao@compal.corp-partner.google.com>
Reviewed-by: Elthan Huang <elthan_huang@compal.corp-partner.google.com>
Reviewed-by: Boris Mittelberg <bmbm@google.com>
Commit-Queue: Logan Liao <logan_liao@compal.corp-partner.google.com>
Diffstat (limited to 'board/banshee/usbc_config.c')
-rw-r--r-- | board/banshee/usbc_config.c | 25 |
1 files changed, 12 insertions, 13 deletions
diff --git a/board/banshee/usbc_config.c b/board/banshee/usbc_config.c index 77226b367a..b93c6fb2dc 100644 --- a/board/banshee/usbc_config.c +++ b/board/banshee/usbc_config.c @@ -304,23 +304,21 @@ void board_reset_pd_mcu(void) */ gpio_set_level(GPIO_USB_C0_C1_TCPC_RST_ODL, 0); gpio_set_level(GPIO_USB_C2_C3_TCPC_RST_ODL, 0); - gpio_set_level(IOEX_USB_C0_RT_RST_ODL, 0); - gpio_set_level(IOEX_USB_C1_RT_RST_ODL, 0); - gpio_set_level(IOEX_USB_C2_RT_RST_ODL, 0); - gpio_set_level(IOEX_USB_C3_RT_RST_ODL, 0); /* * delay for power-on to reset-off and min. assertion time */ - msleep(20); + msleep(NCT38XX_RESET_HOLD_DELAY_MS); gpio_set_level(GPIO_USB_C0_C1_TCPC_RST_ODL, 1); gpio_set_level(GPIO_USB_C2_C3_TCPC_RST_ODL, 1); - gpio_set_level(IOEX_USB_C0_RT_RST_ODL, 1); - gpio_set_level(IOEX_USB_C1_RT_RST_ODL, 1); - gpio_set_level(IOEX_USB_C2_RT_RST_ODL, 1); - gpio_set_level(IOEX_USB_C3_RT_RST_ODL, 1); + + nct38xx_reset_notify(USBC_PORT_C0); + nct38xx_reset_notify(USBC_PORT_C1); + nct38xx_reset_notify(USBC_PORT_C2); + nct38xx_reset_notify(USBC_PORT_C3); /* wait for chips to come up */ - msleep(50); + if (NCT3808_RESET_POST_DELAY_MS != 0) + msleep(NCT3808_RESET_POST_DELAY_MS); } static void board_tcpc_init(void) @@ -362,9 +360,10 @@ uint16_t tcpc_get_alert_status(void) uint16_t status = 0; if (gpio_get_level(GPIO_USB_C0_C1_TCPC_INT_ODL) == 0) - status |= PD_STATUS_TCPC_ALERT_0 | PD_STATUS_TCPC_ALERT_1; + status = PD_STATUS_TCPC_ALERT_0 | PD_STATUS_TCPC_ALERT_1; + if (gpio_get_level(GPIO_USB_C2_C3_TCPC_INT_ODL) == 0) - status |= PD_STATUS_TCPC_ALERT_2 | PD_STATUS_TCPC_ALERT_3; + status = PD_STATUS_TCPC_ALERT_2 | PD_STATUS_TCPC_ALERT_3; return status; } @@ -389,7 +388,7 @@ void tcpc_alert_event(enum gpio_signal signal) schedule_deferred_pd_interrupt(USBC_PORT_C0); break; case GPIO_USB_C2_C3_TCPC_INT_ODL: - schedule_deferred_pd_interrupt(USBC_PORT_C1); + schedule_deferred_pd_interrupt(USBC_PORT_C2); break; default: break; |