diff options
author | Ting Shen <phoenixshen@google.com> | 2020-05-06 18:06:52 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-05-29 05:11:24 +0000 |
commit | 8975cbb645f1f07c3682334d02510a8a1d3f730b (patch) | |
tree | 1c65f1c460b25e785181b15c2d78b303b4cfacfc /common/charge_ramp.c | |
parent | fad8f404df3e1b2278326c08247a62edce434f32 (diff) | |
download | chrome-ec-8975cbb645f1f07c3682334d02510a8a1d3f730b.tar.gz |
bc12: create bc12 driver structure
To support linking multiple bc12 drivers into ec binary, we have to
remove the common symbols in drivers.
This CL creates a bc12_drv structure to hold the original driver-specific
usb_charger_* functions, and implements common usb_charger_* functions
which whill dispatch the function call based on the information in
bc12_drv table.
Also add a CONFIG_BC12_SINGLE_DRIVER and enabled by default for
backward compatibility. If CONFIG_BC12_SINGLE_DRIVER is defined,
a default bc12_drv array with proper size and content will be created
for the board.
BUG=b:155611686
TEST=1) make buildall.
2) verify single driver mode on krane (rt946x) and juniper
(pi3usb9201).
3) verify multiple driver works on asurada, see CL:2189624 for
example usage.
4) verify single driver multi chip on blooglet.
BRANCH=none
Signed-off-by: Ting Shen <phoenixshen@google.com>
Change-Id: I8a96eda47c94aeb6cc150b498cfa1a6eefcc4a5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2187080
Reviewed-by: Jett Rink <jettrink@chromium.org>
Commit-Queue: Ting Shen <phoenixshen@chromium.org>
Tested-by: Ting Shen <phoenixshen@chromium.org>
Diffstat (limited to 'common/charge_ramp.c')
-rw-r--r-- | common/charge_ramp.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/common/charge_ramp.c b/common/charge_ramp.c index 3ca5bd89bb..e95cb0ed06 100644 --- a/common/charge_ramp.c +++ b/common/charge_ramp.c @@ -11,7 +11,7 @@ #include "usb_charge.h" #include "util.h" -test_mockable int chg_ramp_allowed(int supplier) +test_mockable int chg_ramp_allowed(int port, int supplier) { /* Don't allow ramping in RO when write protected. */ if (!system_is_in_rw() && system_is_locked()) @@ -28,11 +28,11 @@ test_mockable int chg_ramp_allowed(int supplier) /* default: fall through */ } - /* Othewise ask the BC1.2 detect module */ - return usb_charger_ramp_allowed(supplier); + /* Otherwise ask the BC1.2 detect module */ + return usb_charger_ramp_allowed(port, supplier); } -test_mockable int chg_ramp_max(int supplier, int sup_curr) +test_mockable int chg_ramp_max(int port, int supplier, int sup_curr) { switch (supplier) { #ifdef CONFIG_CHARGE_RAMP_HW @@ -49,5 +49,5 @@ test_mockable int chg_ramp_max(int supplier, int sup_curr) } /* Otherwise ask the BC1.2 detect module */ - return usb_charger_ramp_max(supplier, sup_curr); + return usb_charger_ramp_max(port, supplier, sup_curr); } |