diff options
author | Eric Yilun Lin <yllin@chromium.org> | 2023-05-08 17:50:57 +0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-05-11 09:29:43 +0000 |
commit | 3aee7bb0a62128e3a7e657c8197925353198ac95 (patch) | |
tree | 62dc8fe7e8d28f9175b688a5b85d5e3524f4c452 /zephyr/program/corsola/src/npcx_usbc.c | |
parent | 0d6fd0109cc767c69aecff8617a58190685d3b5b (diff) | |
download | chrome-ec-3aee7bb0a62128e3a7e657c8197925353198ac95.tar.gz |
corsola: do not reset TCPC if no battery connected
The RT1718S controls the EN_SNK pin to the PPC. When the TCPC
is reset without battery connected, this might cause the brown-out of
the system due to the PPC stops sinking.
BUG=b:276661970
TEST=reset EC without battery, and it first boots due to the soft-reset,
and brown-out and then second boots with power-on flag.
Change-Id: I79453e3625c8510818a78552c674b8887505dcb3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4506109
Reviewed-by: Sung-Chi Li <lschyi@chromium.org>
Commit-Queue: Eric Yilun Lin <yllin@google.com>
Tested-by: Eric Yilun Lin <yllin@google.com>
Diffstat (limited to 'zephyr/program/corsola/src/npcx_usbc.c')
-rw-r--r-- | zephyr/program/corsola/src/npcx_usbc.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/zephyr/program/corsola/src/npcx_usbc.c b/zephyr/program/corsola/src/npcx_usbc.c index a03e659171..2e1d8d11c5 100644 --- a/zephyr/program/corsola/src/npcx_usbc.c +++ b/zephyr/program/corsola/src/npcx_usbc.c @@ -11,6 +11,7 @@ #endif #include "baseboard_usbc_config.h" +#include "battery.h" #include "console.h" #include "driver/ppc/nx20p348x.h" #include "driver/tcpm/anx7447.h" @@ -53,8 +54,10 @@ DECLARE_HOOK(HOOK_INIT, board_usb_mux_init, HOOK_PRIO_INIT_I2C + 1); void board_tcpc_init(void) { - /* Only reset TCPC if not sysjump */ - if (!system_jumped_late()) { + /* Reset TCPC if we only we have a battery connected, or the SINK + * gpio to the PPC might be reset and cause brown-out. + */ + if (!system_jumped_late() && battery_is_present() == BP_YES) { /* TODO(crosbug.com/p/61098): How long do we need to wait? */ board_reset_pd_mcu(); } @@ -84,7 +87,13 @@ __override int board_rt1718s_init(int port) { static bool gpio_initialized; - if (!system_jumped_late() && !gpio_initialized) { + /* Reset TCPC sink/source control when it's a power-on reset or has a + * battery. Do not alter the carried GPIO status or this might stop PPC + * sinking and brown-out the system when battery disconnected. + */ + if (!system_jumped_late() && !gpio_initialized && + (battery_is_present() == BP_YES || + (system_get_reset_flags() & EC_RESET_FLAG_POWER_ON))) { /* set GPIO 1~3 as push pull, as output, output low. */ rt1718s_gpio_set_flags(port, RT1718S_GPIO1, GPIO_OUT_LOW); rt1718s_gpio_set_flags(port, RT1718S_GPIO2, GPIO_OUT_LOW); |