summaryrefslogtreecommitdiff
path: root/common
diff options
context:
space:
mode:
authorScott Collyer <scollyer@google.com>2021-06-01 10:35:34 -0700
committerCommit Bot <commit-bot@chromium.org>2021-06-06 04:02:27 +0000
commite3d0d94a559dff751a4b4dc28617170be4f05ac8 (patch)
treeb1485f1f314dcb6c7e7c60bc53f6c2a7bc753508 /common
parentf2f46e879f5cc9a3197caefb1616918baa5ece21 (diff)
downloadchrome-ec-e3d0d94a559dff751a4b4dc28617170be4f05ac8.tar.gz
honeybuns: Limit data role swap request attempts
This CL modifies honeybuns data role swap policy in 2 ways. First, the dependence on dr_swap_flag is removed. Second, a new variable is added to track data role swap attempts per each usbc attach. Previosly, dr_swap_flag was used as a means to limit the number of data role swap attempts, but the flag that is passed in is set in PE code that is specific to a typical chromebook port policy. The new policy is to always try to be the desired data role, but limit the attempts for situations where the port partner rejects the request. This case happens for example with compliance testing. HOOK_USB_PD_CONNECT is used to clear the max attempt counter as this should apply once per each usbc attach event. BUG=b:188756664 BRANCH=quiche TEST=Verifed that the host port will request a data role swap when testing against the iPad. Against the ipad, host port always attaches as a source, and would remain a DFP. Signed-off-by: Scott Collyer <scollyer@google.com> Change-Id: Ib913f8bbba76d2bd12dccb6a6a49ef4249d428ed Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2934358 Tested-by: Scott Collyer <scollyer@chromium.org> Reviewed-by: Diana Z <dzigterman@chromium.org> Reviewed-by: Abe Levkoy <alevkoy@chromium.org> Commit-Queue: Scott Collyer <scollyer@chromium.org>
Diffstat (limited to 'common')
0 files changed, 0 insertions, 0 deletions