ARM: dts: imx8mm-venice-gw7901: fix USB2 controller OC polarity

[PATCH] ARM: dts: imx8mm-venice-gw7901: fix USB2 controller OC polarity

Posted by Tim Harvey 2 years, 8 months ago

The GW7901 has USB2 routed to a USB VBUS supply with over-current
protection via an active-low pin. Define the OC pin polarity properly.

Fixes: 2b1649a83afc ("arm64: dts: imx: Add i.mx8mm Gateworks gw7901 dts support")
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
 arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
index d3ee6fc4baab..72311b55f06d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-venice-gw7901.dts
@@ -759,6 +759,7 @@ &usbotg1 {
 &usbotg2 {
 	dr_mode = "host";
 	vbus-supply = <&reg_usb2_vbus>;
+	over-current-active-low;
 	status = "okay";
 };
 
-- 
2.25.1

Re: [PATCH] ARM: dts: imx8mm-venice-gw7901: fix USB2 controller OC polarity

Posted by Shawn Guo 2 years, 8 months ago

On Wed, Dec 28, 2022 at 12:26:06PM -0800, Tim Harvey wrote:
> The GW7901 has USB2 routed to a USB VBUS supply with over-current
> protection via an active-low pin. Define the OC pin polarity properly.
> 
> Fixes: 2b1649a83afc ("arm64: dts: imx: Add i.mx8mm Gateworks gw7901 dts support")
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>

Applied, thanks!