diff options
| author | Denis Brockus <dbrockus@chromium.org> | 2020-01-08 06:54:30 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-01-09 19:24:10 +0000 |
| commit | d3a1820df21b12842232ab88a5c1a0490640bb25 (patch) | |
| tree | b0855ad62d8c5f42178cec6cf9038563c9253fb1 /board/coffeecake | |
| parent | a6591e9d5c7caa04f94e604d8a649d9f8e6d5b5b (diff) | |
| download | chrome-ec-d3a1820df21b12842232ab88a5c1a0490640bb25.tar.gz | |
pd: cleanup pd_get_role
pd_get_role in the TCPMv1 stack meant pd_get_power_role.
pd_get_role in the TCPMv2 stack meant pd_get_data_role.
This CL will clean that up and make them the correct naming.
pd_get_power_role will also return an enum pd_power_role
type instead of an int.
BUG=b:147290482
BRANCH=none
TEST=make buildall -j
Change-Id: I73ee465401ccd050c2bd151f2fc043a59d95e079
Signed-off-by: Denis Brockus <dbrockus@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1991844
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'board/coffeecake')
| -rw-r--r-- | board/coffeecake/usb_pd_policy.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/board/coffeecake/usb_pd_policy.c b/board/coffeecake/usb_pd_policy.c index 89f6bd28ad..f263ca11ab 100644 --- a/board/coffeecake/usb_pd_policy.c +++ b/board/coffeecake/usb_pd_policy.c @@ -134,9 +134,11 @@ void pd_execute_data_swap(int port, int data_role) /* TODO: turn on pp5000, pp3300 */ } -void pd_check_pr_role(int port, int pr_role, int flags) +void pd_check_pr_role(int port, + enum pd_power_role pr_role, + int flags) { - if (pr_role == PD_ROLE_UFP && !gpio_get_level(GPIO_AC_PRESENT_L)) + if (pr_role == PD_ROLE_SINK && !gpio_get_level(GPIO_AC_PRESENT_L)) pd_request_power_swap(port); } |