diff options
author | Wai-Hong Tam <waihong@google.com> | 2020-11-09 16:57:58 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-11-10 06:31:51 +0000 |
commit | fdfcd39cfe0ea0fcf4ca64440480226c49cae50b (patch) | |
tree | f7282e41063de790522122765249e2e697a2afb1 | |
parent | fca5fa19da53c5cf598c0c6b9500668fc72ace25 (diff) | |
download | chrome-ec-fdfcd39cfe0ea0fcf4ca64440480226c49cae50b.tar.gz |
isl923x: Disable PSYS monitor on suspend
The PSYS monitor consumes power, 905uA (enabled) vs 24uA (disabled).
When AP is in suspend, which uses less power, the PSYS monitor takes
a significant portion. Should disable it to save power. The `psys`
command will return -1.
BRANCH=None
BUG=b:172701160
TEST=Triggered AP in suspend, used `amonbmon` command to check the
battery discharge current drop.
Change-Id: I6df18a65b040b73e626587d9775296df58d283c6
Signed-off-by: Wai-Hong Tam <waihong@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2528691
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
-rw-r--r-- | driver/charger/isl923x.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/driver/charger/isl923x.c b/driver/charger/isl923x.c index 714d73a93c..655a2890f7 100644 --- a/driver/charger/isl923x.c +++ b/driver/charger/isl923x.c @@ -813,7 +813,7 @@ static void charger_enable_psys(void) out: mutex_unlock(&control1_mutex); } -DECLARE_HOOK(HOOK_CHIPSET_STARTUP, charger_enable_psys, HOOK_PRIO_DEFAULT); +DECLARE_HOOK(HOOK_CHIPSET_RESUME, charger_enable_psys, HOOK_PRIO_DEFAULT); static void charger_disable_psys(void) { @@ -837,7 +837,7 @@ static void charger_disable_psys(void) out: mutex_unlock(&control1_mutex); } -DECLARE_HOOK(HOOK_CHIPSET_SHUTDOWN, charger_disable_psys, HOOK_PRIO_DEFAULT); +DECLARE_HOOK(HOOK_CHIPSET_SUSPEND, charger_disable_psys, HOOK_PRIO_DEFAULT); #ifdef CONFIG_CHARGER_PSYS_READ int charger_get_system_power(void) |