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 <philipchen@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1077496
Commit-Queue: Philip Chen <philipchen@chromium.org>
Tested-by: Philip Chen <philipchen@chromium.org>
Reviewed-by: David Schneider <dnschneid@chromium.org>
Reviewed-by: Alexandru M Stan <amstan@chromium.org>
(cherry picked from commit 8746200bb7c71bdee057580447c78ffb53520fae)
Reviewed-on: https://chromium-review.googlesource.com/1079732
Commit-Ready: Philip Chen <philipchen@chromium.org>
Reviewed-by: Philip Chen <philipchen@chromium.org>
This commit is contained in:
Philip Chen
2018-05-29 15:28:22 -07:00
committed by chrome-bot
parent 3dff02fa73
commit a5f6726587

View File

@@ -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();