diff options
| author | Diana Z <dzigterman@chromium.org> | 2020-06-30 16:46:55 -0600 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-07-11 06:05:23 +0000 |
| commit | 7051495b17aaaa5c87e2470e49cc5d42ed04f11e (patch) | |
| tree | 46815a86df3359bf7727359759acea915b422965 /common/ocpc.c | |
| parent | a0efb9c8039ca50962976d506e738e2071bb431f (diff) | |
| download | chrome-ec-7051495b17aaaa5c87e2470e49cc5d42ed04f11e.tar.gz | |
OCPC: Only fetch secondary charger values when present
Check the runtime charger count to verify whether secondary charger ADCs
should be pulled.
BRANCH=None
BUG=b:155963446
TEST=make -j buildall
Signed-off-by: Diana Z <dzigterman@chromium.org>
Change-Id: I18d0c2717fd4db740f25c31a0766a0dfd3aef5aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2277835
Commit-Queue: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'common/ocpc.c')
| -rw-r--r-- | common/ocpc.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/common/ocpc.c b/common/ocpc.c index 7240331890..e03afa9bbd 100644 --- a/common/ocpc.c +++ b/common/ocpc.c @@ -245,13 +245,19 @@ void ocpc_get_adcs(struct ocpc_data *ocpc) ocpc->primary_vbus_mv = val; val = 0; - if (!charger_get_vbus_voltage(CHARGER_SECONDARY, &val)) - ocpc->secondary_vbus_mv = val; - - val = 0; if (!charger_get_input_current(CHARGER_PRIMARY, &val)) ocpc->primary_ibus_ma = val; + if (board_get_charger_chip_count() <= CHARGER_SECONDARY) { + ocpc->secondary_vbus_mv = 0; + ocpc->secondary_ibus_ma = 0; + return; + } + + val = 0; + if (!charger_get_vbus_voltage(CHARGER_SECONDARY, &val)) + ocpc->secondary_vbus_mv = val; + val = 0; if (!charger_get_input_current(CHARGER_SECONDARY, &val)) ocpc->secondary_ibus_ma = val; |