Message ID | d117473da3bab39a82fe900cecd8ca78df9adeda.1683034376.git.michal.simek@amd.com |
---|---|
State | Accepted |
Commit | 028d125a30ff5646b3ad22445080d8f2946d67d7 |
Headers | show |
Series | arm64: zynqmp: Misc zynqmp changes | expand |
On 5/2/23 15:35, Michal Simek wrote: > The commit 53ba1b2bdaf7 ("arm64: dts: zynqmp: Add mode-pin GPIO controller > DT node") added usb phy reset over bootmode pins by default on usb0 only. > zcu100 is using usb0 as peripheral and usb1 as host. Unfortunately reset > line is shared for both usb ulpi phys but usb_rst_b is connected to usb5744 > hub which is used only in host mode. Especially this chip requires reset to > operate properly that's why better assign gpio reset to usb1 instead of > usb0. > > Signed-off-by: Michal Simek <michal.simek@amd.com> > --- > > arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > index 2dd552cf51fb..c99abb99efcb 100644 > --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts > @@ -552,6 +552,7 @@ &usb0 { > pinctrl-0 = <&pinctrl_usb0_default>; > phy-names = "usb3-phy"; > phys = <&psgtr 2 PHY_TYPE_USB3 0 0>; > + /delete-property/ reset-gpios; > }; > > &dwc3_0 { > @@ -567,6 +568,7 @@ &usb1 { > pinctrl-0 = <&pinctrl_usb1_default>; > phy-names = "usb3-phy"; > phys = <&psgtr 3 PHY_TYPE_USB3 1 0>; > + reset-gpios = <&modepin_gpio 1 GPIO_ACTIVE_LOW>; > }; > > &dwc3_1 { Applied. M
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts index 2dd552cf51fb..c99abb99efcb 100644 --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts @@ -552,6 +552,7 @@ &usb0 { pinctrl-0 = <&pinctrl_usb0_default>; phy-names = "usb3-phy"; phys = <&psgtr 2 PHY_TYPE_USB3 0 0>; + /delete-property/ reset-gpios; }; &dwc3_0 { @@ -567,6 +568,7 @@ &usb1 { pinctrl-0 = <&pinctrl_usb1_default>; phy-names = "usb3-phy"; phys = <&psgtr 3 PHY_TYPE_USB3 1 0>; + reset-gpios = <&modepin_gpio 1 GPIO_ACTIVE_LOW>; }; &dwc3_1 {
The commit 53ba1b2bdaf7 ("arm64: dts: zynqmp: Add mode-pin GPIO controller DT node") added usb phy reset over bootmode pins by default on usb0 only. zcu100 is using usb0 as peripheral and usb1 as host. Unfortunately reset line is shared for both usb ulpi phys but usb_rst_b is connected to usb5744 hub which is used only in host mode. Especially this chip requires reset to operate properly that's why better assign gpio reset to usb1 instead of usb0. Signed-off-by: Michal Simek <michal.simek@amd.com> --- arch/arm64/boot/dts/xilinx/zynqmp-zcu100-revC.dts | 2 ++ 1 file changed, 2 insertions(+)