diff options
author | Ayushee <ayushee.shah@intel.com> | 2020-04-27 12:16:52 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-04-29 22:39:03 +0000 |
commit | 534b6b0884363660e9762368582fc45ae6256fbe (patch) | |
tree | a28fb467a9b4aac3145f0e759c2f7be33ef2c052 /common/usb_pd_policy.c | |
parent | d1f0d51080ba59fcaf266381fcceb4f32315af05 (diff) | |
download | chrome-ec-534b6b0884363660e9762368582fc45ae6256fbe.tar.gz |
usb_pd: Correct processing discover SVID
In current code, Discover SVID SOP' is sent if the cable supports Intel
SVID. But, according to USB Type-C Cable and Connector Spec figure F-1:
TBT3 Discovery Flow, if the cable supports Intel SVID, Discover Mode SOP
should be sent.
BUG=b:154308721
BRANCH=None
TEST=Tested on Volteer, able to enter Thunderbolt and USB4 mode with
Passive Gen2 cables:
1. Supporting Intel SVID.
2. Not supporting Intel SVID.
Signed-off-by: Ayushee <ayushee.shah@intel.com>
Change-Id: I0a8ab913836b018547a5306dd82d3bed80273f87
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2166741
Reviewed-by: Keith Short <keithshort@chromium.org>
Reviewed-by: Vijay P Hiremath <vijay.p.hiremath@intel.com>
Diffstat (limited to 'common/usb_pd_policy.c')
-rw-r--r-- | common/usb_pd_policy.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/usb_pd_policy.c b/common/usb_pd_policy.c index ec5b9aba7c..51ed5113da 100644 --- a/common/usb_pd_policy.c +++ b/common/usb_pd_policy.c @@ -711,13 +711,13 @@ static int process_am_discover_svids(int port, int cnt, uint32_t *payload) limit_tbt_cable_speed(port); else disable_tbt_compat_mode(port); - } else { + } else if (!is_transmit_msg_sop_prime(port)) { enable_transmit_sop_prime(port); return dfp_discover_svids(payload); } - } - disable_transmit_sop_prime(port); + disable_transmit_sop_prime(port); + } return dfp_discover_modes(port, payload); } |