summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorTing Shen <phoenixshen@google.com>2020-06-04 18:35:58 +0800
committerCommit Bot <commit-bot@chromium.org>2020-06-10 11:32:18 +0000
commit3dc4147d8792447671b3a37726e4f62f72c0db8b (patch)
treed3e88f0791875b930d0605303199f562130a1a6d /include
parent7589c88c048700f349a7515539f8fe8f5c67b155 (diff)
downloadchrome-ec-3dc4147d8792447671b3a37726e4f62f72c0db8b.tar.gz
charger/isl9238c: enable slew rate control
Enable charge current and maximum system voltage slew rate control on ISL9238C by default for better charger stability. Since the control is only available on ISL9238C, not on A/B, and there's no way to distinguish C and A/B in software side, introduce a new config CONFIG_CHARGER_ISL9238C for 9238C users. BUG=b:155366741 TEST=1) on asurada, verify the register configured correctly 2) verify the waveform looks better BRANCH=kukui Signed-off-by: Ting Shen <phoenixshen@google.com> Change-Id: I577db3eebcad0ed3793f4b7d525e4246515f8503 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2231725 Tested-by: Hsu Alvis <alvishsu@google.com> Tested-by: Ting Shen <phoenixshen@chromium.org> Reviewed-by: Eric Yilun Lin <yllin@chromium.org> Reviewed-by: Ting Shen <phoenixshen@chromium.org> Commit-Queue: Ting Shen <phoenixshen@chromium.org>
Diffstat (limited to 'include')
-rw-r--r--include/config.h5
1 files changed, 3 insertions, 2 deletions
diff --git a/include/config.h b/include/config.h
index d2355a0fe7..763b932c0e 100644
--- a/include/config.h
+++ b/include/config.h
@@ -795,7 +795,8 @@
#undef CONFIG_CHARGER_BQ24773
#undef CONFIG_CHARGER_BQ25710
#undef CONFIG_CHARGER_ISL9237
-#undef CONFIG_CHARGER_ISL9238
+#undef CONFIG_CHARGER_ISL9238 /* For ISL9238 A/B */
+#undef CONFIG_CHARGER_ISL9238C
#undef CONFIG_CHARGER_ISL9241
#undef CONFIG_CHARGER_MT6370
#undef CONFIG_CHARGER_RAA489000
@@ -4892,7 +4893,7 @@
* architecture.
*/
#if defined(CONFIG_CHARGER_ISL9237) || defined(CONFIG_CHARGER_ISL9238) || \
- defined(CONFIG_CHARGER_ISL9241) || \
+ defined(CONFIG_CHARGER_ISL9238C) || defined(CONFIG_CHARGER_ISL9241) || \
defined(CONFIG_CHARGER_RAA489000) || defined(CONFIG_CHARGER_SM5803)
#define CONFIG_CHARGER_NARROW_VDC
#endif