diff options
author | Wai-Hong Tam <waihong@google.com> | 2020-05-21 09:56:22 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-05-21 20:35:57 +0000 |
commit | a998171f5b4db4ab21cef33e80e5acc129b35097 (patch) | |
tree | 5aca3d55e691d14be8eee1a23ecacc62c47a2540 /board/trogdor | |
parent | a64e3930338fceeb258c16dfc6b713f022fbbf41 (diff) | |
download | chrome-ec-a998171f5b4db4ab21cef33e80e5acc129b35097.tar.gz |
Trogdor: Fix the current limitation of USB Type-A port
The ILIM pin of the load switch is through a MOFET to EC GPIO.
When EC outputs this GPIO High, it closes the MOFET and applies
impedance (28.7K || 41.2K) to the ILIM pin. The load switch limits
the CDP current: 1413-1626 mA.
When EC outputs this GPIO Low, it opens the MOFET and applies
impedance (28.7K) to the ILIM pin. The load switch limits the
SDP current: 825-989 mA.
The original GPIO default to Low was wrong that actually limits
the SDP current. Should make it default to High, to limit the CDP
current.
BRANCH=None
BUG=b:156965411
TEST=Applied a load to the USB Type-A port and checked it can
source ~5V @ 1.5A.
Change-Id: I1d8d52d1a82b5f837bc1a2de544425c8f2aff759
Signed-off-by: Wai-Hong Tam <waihong@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2211958
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Alexandru M Stan <amstan@chromium.org>
Diffstat (limited to 'board/trogdor')
-rw-r--r-- | board/trogdor/gpio.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/trogdor/gpio.inc b/board/trogdor/gpio.inc index f22339435d..1ce44cbb22 100644 --- a/board/trogdor/gpio.inc +++ b/board/trogdor/gpio.inc @@ -88,7 +88,7 @@ GPIO(USB_C1_VBUS_DET_L, PIN(8, 3), GPIO_INPUT) /* Deprecated BC1.2 VBUS /* USB-A */ GPIO(EN_USB_A_5V, PIN(8, 6), GPIO_OUT_LOW) -GPIO(USB_A_CDP_ILIM_EN_L, PIN(7, 5), GPIO_OUT_LOW) /* Only one USB-A port, always CDP */ +GPIO(USB_A_CDP_ILIM_EN_L, PIN(7, 5), GPIO_OUT_HIGH) /* H:CDP, L:SDP. Only one USB-A port, always CDP */ GPIO(USB_A0_OC_ODL, PIN(D, 1), GPIO_ODR_HIGH) /* LEDs */ |