diff options
| author | Jonathan Brandmeyer <jbrandmeyer@chromium.org> | 2018-07-10 14:11:15 -0600 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-07-11 23:06:45 -0700 |
| commit | caa47a8f560b48b0499520644bb0c6491875cf9c (patch) | |
| tree | 5634cf25c210c75e9e3b429c8c6fef8435d6c630 /baseboard | |
| parent | febbdc42a4c1ddd8ec5cc6c46388aa84180f8ac7 (diff) | |
| download | chrome-ec-caa47a8f560b48b0499520644bb0c6491875cf9c.tar.gz | |
grunt,careena: Raise the charger current supremum.
Raise the current limit to the maximum Sink Standby limit allowable by
USB-PD. The former current limit is intended to improve compatibility
with BC1.2 chargers that brownout at the full 500 mA of supply.
However, there is a risk that it will also cause battery brownouts and
failure to boot in a startup condition of a highly imbalanced battery
pack. This startup condition may happen after an extended period of
rest prior to delivery to the end customer, which leads to poor customer
first impressions and a high RMA rate.
TEST=Grunt EVT clamshell and USB-PD analyzer, verify that the lowest
current drawn during an initial boot sequence is ~0.5A.
BRANCH=none
BUG=b:111214767
Change-Id: I4dc7b143fd947c1c5bbdb8787ab30f92e587ed47
Signed-off-by: Jonathan Brandmeyer <jbrandmeyer@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1133424
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'baseboard')
| -rw-r--r-- | baseboard/grunt/baseboard.h | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/baseboard/grunt/baseboard.h b/baseboard/grunt/baseboard.h index e894a81f31..1ba6354f33 100644 --- a/baseboard/grunt/baseboard.h +++ b/baseboard/grunt/baseboard.h @@ -54,7 +54,18 @@ #define CONFIG_CHARGER_V2 #define CONFIG_CHARGE_MANAGER #define CONFIG_CHARGER_DISCHARGE_ON_AC -#define CONFIG_CHARGER_INPUT_CURRENT 128 + +/* + * This limit impairs compatibility with BC1.2 chargers that are not actually + * capable of supplying 500 mA of current. When the charger is paralleled with + * the battery, raising this limit allows the power system to draw more current + * from the charger during startup. This improves compatibility with system + * batteries that may become excessively imbalanced after extended periods of + * rest. + * + * See also b/111214767 + */ +#define CONFIG_CHARGER_INPUT_CURRENT 512 #define CONFIG_CHARGER_ISL9238 #define CONFIG_CHARGER_MIN_BAT_PCT_FOR_POWER_ON 1 #define CONFIG_CHARGER_SENSE_RESISTOR 10 |