diff options
author | Simon Glass <sjg@chromium.org> | 2023-05-04 15:25:19 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-05-08 20:16:03 +0000 |
commit | 8ee0ea15c200f35695492581de44063e5cb42bab (patch) | |
tree | cc909cd256711c67dacd618f5a3dbd0006f22736 | |
parent | 6fec89af8998b606489ff5b93c3cc5873a21d9c4 (diff) | |
download | chrome-ec-8ee0ea15c200f35695492581de44063e5cb42bab.tar.gz |
charger: Move handling of external power to a function
Put this code in a separate function, since it is logically related.
This makes no functional change.
BUG=b:218332694
TEST=zmake build dev-posix
Change-Id: I1ebab33e72d5927bbc3fcd1e0562b66102c5c689
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4508346
Reviewed-by: Tristan Honscheid <honscheid@google.com>
Commit-Queue: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
-rw-r--r-- | common/charge_state_v2.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/common/charge_state_v2.c b/common/charge_state_v2.c index 2c92a90378..f9a1e190a8 100644 --- a/common/charge_state_v2.c +++ b/common/charge_state_v2.c @@ -2038,6 +2038,17 @@ int calculate_sleep_dur(int battery_critical, int sleep_usec) return sleep_usec; } +/* check external power and handle any changes */ +static void check_extpower(int chgnum) +{ + curr.ac = extpower_is_present(); + if (IS_ENABLED(CONFIG_EC_EC_COMM_BATTERY_CLIENT)) + base_check_extpower(); + + if (curr.ac != prev_ac) + process_ac_change(chgnum); +} + /* Main loop */ void charger_task(void *u) { @@ -2056,12 +2067,8 @@ void charger_task(void *u) sleep_usec = 0; problems_exist = 0; battery_critical = 0; - curr.ac = extpower_is_present(); - if (IS_ENABLED(CONFIG_EC_EC_COMM_BATTERY_CLIENT)) - base_check_extpower(); - if (curr.ac != prev_ac) - process_ac_change(chgnum); + check_extpower(chgnum); if (IS_ENABLED(CONFIG_EC_EC_COMM_BATTERY_CLIENT)) base_update_battery_info(); |