From bf3f8ca53fac14cafaa8e66f682f5542f71a97ca Mon Sep 17 00:00:00 2001 From: Duncan Laurie Date: Fri, 30 Dec 2016 01:50:54 -0800 Subject: [PATCH] eve: Do not attempt to set 1.5A USB config The EC attempts to limit USB ports to 1.5A if a device is plugged into both ports, but this ends up with no power to the ports instead. Until this can be debugged with the right equipment just pretend we can do 3A to both ports so factory tests can pass. BUG=chrome-os-partner:61431 BRANCH=none TEST=plug device into both ports on P1 board and see that they work Change-Id: Icd4430d0026dc323e56b7ce88b9d8e79e6e825c5 Signed-off-by: Duncan Laurie Reviewed-on: https://chromium-review.googlesource.com/424453 --- board/eve/usb_pd_policy.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/board/eve/usb_pd_policy.c b/board/eve/usb_pd_policy.c index dbc510caf1..4e3c3d9872 100644 --- a/board/eve/usb_pd_policy.c +++ b/board/eve/usb_pd_policy.c @@ -67,8 +67,9 @@ int board_vbus_source_enabled(int port) static void board_vbus_update_source_current(int port) { enum gpio_signal gpio = port ? GPIO_USB_C1_5V_EN : GPIO_USB_C0_5V_EN; + /* TODO(crosbug.com/p/61431): do not attempt to set 1.5A config */ int flags = (vbus_rp[port] == TYPEC_RP_1A5 && vbus_en[port]) ? - (GPIO_INPUT | GPIO_PULL_UP) : (GPIO_OUTPUT | GPIO_PULL_UP); + (GPIO_OUTPUT | GPIO_PULL_UP) : (GPIO_OUTPUT | GPIO_PULL_UP); /* * Driving USB_Cx_5V_EN high, actually put a 16.5k resistance