diff options
| author | Shawn Nematbakhsh <shawnn@chromium.org> | 2017-11-06 10:07:15 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-11-07 12:17:43 -0800 |
| commit | f4ee6caa665009ca3306f61706074b82e1e2c347 (patch) | |
| tree | c61dd1cd667f35324443a64c1d540d67e6ee5ed9 /board/eve | |
| parent | fca1b7b710898ff347465d839c14164d610f140d (diff) | |
| download | chrome-ec-f4ee6caa665009ca3306f61706074b82e1e2c347.tar.gz | |
bd9995x: Use fixed PD-port-to-VBUS/VCC mapping
The bd9995x driver was written to allow any PD port # to be VBUS or VCC,
but the mapping is broken in a few places. Since all boards use VBUS =
port 0, remove the conversion entirely.
BUG=chromium:781849
BRANCH=kevin
TEST=Verify PD and BC1.2 charging still works on kevin.
Change-Id: I3687866835d1684342d9f746d91b3a6079ab5cc4
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/755000
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'board/eve')
| -rw-r--r-- | board/eve/board.c | 15 | ||||
| -rw-r--r-- | board/eve/usb_pd_policy.c | 2 |
2 files changed, 4 insertions, 13 deletions
diff --git a/board/eve/board.c b/board/eve/board.c index d6b4d87c38..f580c30651 100644 --- a/board/eve/board.c +++ b/board/eve/board.c @@ -445,19 +445,10 @@ DECLARE_HOOK(HOOK_AC_CHANGE, board_extpower, HOOK_PRIO_DEFAULT); int pd_snk_is_vbus_provided(int port) { - enum bd9995x_charge_port bd9995x_port; - - switch (port) { - case 0: - case 1: - bd9995x_port = bd9995x_pd_port_to_chg_port(port); - break; - default: + if (port != 0 && port != 1) panic("Invalid charge port\n"); - break; - } - return bd9995x_is_vbus_provided(bd9995x_port); + return bd9995x_is_vbus_provided(port); } /** @@ -480,7 +471,7 @@ int board_set_active_charge_port(int charge_port) if (board_vbus_source_enabled(charge_port)) return -1; - bd9995x_port = bd9995x_pd_port_to_chg_port(charge_port); + bd9995x_port = charge_port; break; case CHARGE_PORT_NONE: bd9995x_port_select = 0; diff --git a/board/eve/usb_pd_policy.c b/board/eve/usb_pd_policy.c index 62664d95ae..73cb23aa85 100644 --- a/board/eve/usb_pd_policy.c +++ b/board/eve/usb_pd_policy.c @@ -112,7 +112,7 @@ void typec_set_source_current_limit(int port, int rp) int pd_set_power_supply_ready(int port) { /* Ensure we're not charging from this port */ - bd9995x_select_input_port(bd9995x_pd_port_to_chg_port(port), 0); + bd9995x_select_input_port(port, 0); /* Ensure we advertise the proper available current quota */ charge_manager_source_port(port, 1); |