From 8ee0ea15c200f35695492581de44063e5cb42bab Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Thu, 4 May 2023 15:25:19 -0600 Subject: 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 Commit-Queue: Simon Glass Tested-by: Simon Glass --- common/charge_state_v2.c | 17 ++++++++++++----- 1 file 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(); -- cgit v1.2.1