diff options
| author | Duncan Laurie <dlaurie@google.com> | 2018-05-07 09:34:05 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-09-07 02:24:15 +0000 |
| commit | 5f4f01f54794d4b85d7f6903ef68296d5bc93640 (patch) | |
| tree | b7b06bacf977a2dd87613d8d13b04b048457186c /board/eve/board.c | |
| parent | d55be87185ba432114da3f10c46f85d1669241d5 (diff) | |
| download | chrome-ec-5f4f01f54794d4b85d7f6903ef68296d5bc93640.tar.gz | |
Revert "eve: Use PCH ACOK signal to control Deep Sleep entry"
This reverts commit a49f19a1aa9ad3f4951490f96f0320fa646b572e.
Battery life in S3 (not Deep S3) is terrible, so reverting this
change until we determine if improvements can be made.
BUG=b:79346372
BRANCH=eve
TEST=none
Change-Id: I570c5ea1b2b74f17fb41cd46424bfe2a5749d49a
Reviewed-on: https://chromium-review.googlesource.com/1047845
Reviewed-by: Duncan Laurie <dlaurie@google.com>
Commit-Queue: Duncan Laurie <dlaurie@google.com>
Tested-by: Duncan Laurie <dlaurie@google.com>
Trybot-Ready: Duncan Laurie <dlaurie@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2320266
Commit-Queue: Patryk Duda <pdk@semihalf.com>
Tested-by: Patryk Duda <pdk@semihalf.com>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'board/eve/board.c')
| -rw-r--r-- | board/eve/board.c | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/board/eve/board.c b/board/eve/board.c index c8c5ff70ba..4665698375 100644 --- a/board/eve/board.c +++ b/board/eve/board.c @@ -480,19 +480,6 @@ int board_has_working_reset_flags(void) return 1; } -/* - * Update status of the ACPRESENT pin on the PCH. In order to prevent - * Deep S3 when USB is inserted this will indicate that AC is present - * if either port is supplying VBUS or there an external charger present. - */ -void board_update_ac_status(void) -{ - gpio_set_level(GPIO_PCH_ACOK, extpower_is_present() || - board_vbus_source_enabled(0) || - board_vbus_source_enabled(1)); -} -DECLARE_HOOK(HOOK_AC_CHANGE, board_update_ac_status, HOOK_PRIO_DEFAULT); - /* Initialize board. */ static void board_init(void) { @@ -508,8 +495,8 @@ static void board_init(void) /* Enable interrupts from BMI160 sensor. */ gpio_enable_interrupt(GPIO_ACCELGYRO3_INT_L); - /* Update AC status to the PCH */ - board_update_ac_status(); + /* Provide AC status to the PCH */ + gpio_set_level(GPIO_PCH_ACOK, extpower_is_present()); #ifndef TEST_BUILD if (board_get_version() == BOARD_VERSION_EVT) { @@ -527,6 +514,15 @@ static void board_init(void) } DECLARE_HOOK(HOOK_INIT, board_init, HOOK_PRIO_DEFAULT); +/** + * Buffer the AC present GPIO to the PCH. + */ +static void board_extpower(void) +{ + gpio_set_level(GPIO_PCH_ACOK, extpower_is_present()); +} +DECLARE_HOOK(HOOK_AC_CHANGE, board_extpower, HOOK_PRIO_DEFAULT); + int pd_snk_is_vbus_provided(int port) { if (port != 0 && port != 1) |