diff options
author | Sam Hurst <shurst@google.com> | 2019-11-21 13:57:26 -0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-12-10 03:35:19 +0000 |
commit | 0ee58e53ecb9417b7c264be0d3c199e1919fafd6 (patch) | |
tree | 070eb605ff49b8b27cc847da0e34104e804b0100 /test | |
parent | 3973af40f9ca5dd7ba06629b6968b39e912d1c40 (diff) | |
download | chrome-ec-0ee58e53ecb9417b7c264be0d3c199e1919fafd6.tar.gz |
TCPMv2: Fixed PR swap request from charge manager
The PR swap request from the charge manager was locking up the
TC state machine. The pd_request_power_swap function was changed
so that it sends the request to the policy engine instead of
initiating it directly.
BUG=chromium:1027247
BRANCH=none
TEST=make -j buildall
Tested with apple dongle "A2119 2019 HBR3" known to reproduce this
problem.
Change-Id: Ic59036056434ed525a839cd781cff93e3b5bfc53
Signed-off-by: Sam Hurst <shurst@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1928795
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/fake_usbc.c | 3 | ||||
-rw-r--r-- | test/usb_pe_drp.c | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/test/fake_usbc.c b/test/fake_usbc.c index b0137cb893..a563f3d4d8 100644 --- a/test/fake_usbc.c +++ b/test/fake_usbc.c @@ -16,9 +16,6 @@ int pd_is_vbus_present(int port) void pd_request_data_swap(int port) {} -void pd_request_power_swap(int port) -{} - void pd_request_vconn_swap_off(int port) {} diff --git a/test/usb_pe_drp.c b/test/usb_pe_drp.c index c6a1dc0af7..e8167cc1ac 100644 --- a/test/usb_pe_drp.c +++ b/test/usb_pe_drp.c @@ -29,6 +29,10 @@ int board_vbus_source_enabled(int port) { return 0; } +void tc_request_power_swap(int port) +{ + /* Do nothing */ +} /** * Test section |