diff options
author | Scott Collyer <scollyer@google.com> | 2019-05-15 11:17:11 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-05-24 15:09:41 -0700 |
commit | 8610d1cd4c101e7aaa77721a5e68e2f16ea0dd78 (patch) | |
tree | 6afdc9ccc3d45381241a39e15d415ca2e2307e82 | |
parent | 02e05cfe5b298cdf8e326333d96ae2958423f10c (diff) | |
download | chrome-ec-8610d1cd4c101e7aaa77721a5e68e2f16ea0dd78.tar.gz |
hatch: Enable CONFIG_CHARGER_BQ25710_IDCHG_LIMIT_MA
This CL enables CONFIG_CHARGER_BQ25710_IDCHG_LIMIT_MA with a 8192 mA limit.
With his config enabled, the bq25710 will be kept in performance mode
when the AP is in S0. The IDCHG (battery current) prochot trigger is
activated and whenever the battery current draw spikes above 8192 mA,
prochot will be asserted for its defined pulse width (default is 10
msec).
BUG=b:132197575
BRANCH=none
TEST=Was tested at the factory on battery only. When the battery SOC
was less than 5% the unit failed to boot 5 out 5 attempts. Using this
CL to enable IDCHG prochot, the unit booted successfully in same
scenario on all 5 attempts.
Change-Id: I60956fc0311e95cef6621287e93424a37bcfdc98
Signed-off-by: Scott Collyer <scollyer@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1614785
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Scott Collyer <scollyer@chromium.org>
Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
-rw-r--r-- | board/hatch/board.h | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/board/hatch/board.h b/board/hatch/board.h index 560b16d84b..06a7e7945c 100644 --- a/board/hatch/board.h +++ b/board/hatch/board.h @@ -68,6 +68,18 @@ /* BC 1.2 */ #define CONFIG_BC12_DETECT_PI3USB9201 +/* Charger features */ +/* + * The IDCHG current limit is set in 512 mA steps. The value set here is + * somewhat specific to the battery pack being currently used. The limit here + * was set based on the battery's discharge current limit and what was tested to + * prevent the AP rebooting with low charge level batteries. + * + * TODO(b/133447140): Revisit this threshold once peak power consumption tuning + * for the AP is completed. + */ +#define CONFIG_CHARGER_BQ25710_IDCHG_LIMIT_MA 8192 + /* Volume Button feature */ #define CONFIG_VOLUME_BUTTONS #define GPIO_VOLUME_UP_L GPIO_EC_VOLUP_BTN_ODL |