diff options
| author | Andrew McRae <amcrae@google.com> | 2020-04-30 21:52:25 +1000 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-05-01 04:10:17 +0000 |
| commit | 2f7c6f8e08119255027d360e0fd2b1d7f05af75a (patch) | |
| tree | c110914593b69895b3a671a1b340b924da95c5aa /common/usb_pd_dual_role.c | |
| parent | 99c721b761d628e0f4236e12903cec65c62abd57 (diff) | |
| download | chrome-ec-2f7c6f8e08119255027d360e0fd2b1d7f05af75a.tar.gz | |
USB PD: Check supplier instead of port.
When in dual role, check the charger supplier rather than the
port number to verify it is a dedicated charger.
BUG=b:155368947
TEST=Build and verify on puff that dual role is selected.
BRANCH=none
Signed-off-by: Andrew McRae <amcrae@google.com>
Change-Id: If23594a4321da2370bd8f2f787a6091041073189
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2173930
Reviewed-by: Andrew McRae <amcrae@chromium.org>
Commit-Queue: Andrew McRae <amcrae@chromium.org>
Tested-by: Andrew McRae <amcrae@chromium.org>
Diffstat (limited to 'common/usb_pd_dual_role.c')
| -rw-r--r-- | common/usb_pd_dual_role.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/usb_pd_dual_role.c b/common/usb_pd_dual_role.c index a424555b26..bf5dfa06d5 100644 --- a/common/usb_pd_dual_role.c +++ b/common/usb_pd_dual_role.c @@ -382,10 +382,10 @@ bool pd_is_try_source_capable(void) #if CONFIG_DEDICATED_CHARGE_PORT_COUNT > 0 /* - * Since a dedicated charge port can source power allow PD - * trying as source. + * If a dedicated supplier is present, power is not a concern and + * therefore always allow Try.Src. */ - new_try_src |= (charge_manager_get_active_charge_port() == + new_try_src |= (charge_manager_get_supplier() == CHARGE_SUPPLIER_DEDICATED); #endif /* CONFIG_DEDICATED_CHARGE_PORT_COUNT */ |