diff options
author | Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com> | 2022-11-21 16:26:22 +0530 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-11-30 19:35:39 +0000 |
commit | 2888f43dfd8b1b58021ad650888efd5af641641c (patch) | |
tree | feb0688ef0df5e6f63c80821977103c5b612c9aa | |
parent | eea5b0dfff88ec51f23b5d4f1290dcfcc327c8df (diff) | |
download | chrome-ec-2888f43dfd8b1b58021ad650888efd5af641641c.tar.gz |
common/charger: read charger parameters using primary charger
when no active charger is connected to the system, retrieve charger
parameters using primary charger(0).
This avoids setting the 'problems_exist' flag which allows
charger_task() to sleep for a period of CHARGE_POLL_PERIOD_VERY_LONG
in susped scenarios.
BUG=b:258613461
BRANCH=none
TEST=zmake build craask, EC S0ix power is optimized
Change-Id: Ic177acb74f943ec8b3eafaae7bf6b56db758d6e5
Signed-off-by: Deepti Deshatty <deepti.deshatty@intel.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4050321
Code-Coverage: Diana Z <dzigterman@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
-rw-r--r-- | common/charger.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/common/charger.c b/common/charger.c index af9b246adb..198dc0db8c 100644 --- a/common/charger.c +++ b/common/charger.c @@ -97,8 +97,12 @@ void charger_get_params(struct charger_params *chg) { int chgnum = 0; - if (IS_ENABLED(CONFIG_OCPC)) + if (IS_ENABLED(CONFIG_OCPC)) { chgnum = charge_get_active_chg_chip(); + /* set to CHARGE_PORT_NONE when no charger connected */ + if (chgnum < 0) + chgnum = 0; + } memset(chg, 0, sizeof(*chg)); |