From a5f6726587edb2761caeb396e4a10d32a74ed09e Mon Sep 17 00:00:00 2001 From: Philip Chen Date: Tue, 29 May 2018 15:28:22 -0700 Subject: [PATCH] scarlet: Don't disable idle mode in S3 I heard we only want to disable idle mode in S5, when battery is full. BUG=b:78792296 BRANCH=scarlet TEST=manually test on scarlet, and confirm when battery is full, idle mode is disabled in S5 but not in S3. Change-Id: I5809da581dd3fc3d382f606168a88263740256c0 Signed-off-by: Philip Chen Reviewed-on: https://chromium-review.googlesource.com/1077496 Commit-Queue: Philip Chen Tested-by: Philip Chen Reviewed-by: David Schneider Reviewed-by: Alexandru M Stan (cherry picked from commit 8746200bb7c71bdee057580447c78ffb53520fae) Reviewed-on: https://chromium-review.googlesource.com/1079732 Commit-Ready: Philip Chen Reviewed-by: Philip Chen --- board/scarlet/battery.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/board/scarlet/battery.c b/board/scarlet/battery.c index 62a733e463..7c407fe6d1 100644 --- a/board/scarlet/battery.c +++ b/board/scarlet/battery.c @@ -257,8 +257,7 @@ int charger_profile_override(struct charge_state_data *curr) * This is a workaround for b:78792296. When AP is off and * charge termination is detected, we disable idle mode. */ - if (chipset_in_state(CHIPSET_STATE_ANY_OFF | - CHIPSET_STATE_ANY_SUSPEND)) + if (chipset_in_state(CHIPSET_STATE_ANY_OFF)) disable_idle(); else enable_idle();