diff options
| author | Scott Collyer <scollyer@google.com> | 2020-01-17 11:26:24 -0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-01-18 02:42:48 +0000 |
| commit | 1cb3412e1259e170de466f75e4539ec37538767c (patch) | |
| tree | 5c1d29adfff52ef2bfc9ce6c04c069905b12f277 /driver/build.mk | |
| parent | 794d7443506b63781379dae37a05a29825bbd8ef (diff) | |
| download | chrome-ec-1cb3412e1259e170de466f75e4539ec37538767c.tar.gz | |
bq25710: Check battery disconnect state prior to enabling ICO mode
The bq25710 has an ICO (input current optimization) feature that's
used for hw current ramp. When ICO mode is enabled, VSYS is not
regulated, and therefore ICO mode can only be utilized when a battery
is present. The driver was checking for battery presence, but did
not account for the battery discharge FET being disabled. This latter
case can happen when a battery has been put into ship mode and AC
power gets applied again.
This CL adds the check for battery disconnected state and will not
allow the hw ramp to be enabled if the battery discharge FET is
disabled.
BUG=b:146914669
BRANCH=firmware-hatch-12672.B
TEST=tested on kindred, put battery in ship mode, then reconnected AC
power and verified that the AP power sequences correctly, and that
there is no PPVAR_VSYS collapse.
Change-Id: I0535d6a89a135e6bfd64400f17f8072ec6a6a7aa
Signed-off-by: Scott Collyer <scollyer@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2008096
Tested-by: Scott Collyer <scollyer@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Philip Chen <philipchen@chromium.org>
Commit-Queue: Scott Collyer <scollyer@chromium.org>
Diffstat (limited to 'driver/build.mk')
0 files changed, 0 insertions, 0 deletions