| Message ID | 20231120-j7200-usb-suspend-v2-7-038c7e4a3df4@bootlin.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | usb: cdns: fix suspend on J7200 by assuming reset-on-resume | expand |
On 20/11/2023 18:06, Théo Lebrun wrote: > On our platform, suspend-to-idle or suspend-to-RAM turn the controller > off. This compatible triggers reset on resume behavior to reconfigure > the hardware. > > Signed-off-by: Théo Lebrun <theo.lebrun@bootlin.com> Fix your addressee list... > --- > arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > index 709081cd1e7f..52c4ee0fa334 100644 > --- a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > @@ -788,7 +788,7 @@ pcie1_ep: pcie-ep@2910000 { > }; > > usbss0: cdns-usb@4104000 { > - compatible = "ti,j721e-usb"; > + compatible = "ti,j7200-usb", "ti,j721e-usb"; It does not match your bindings. Bindings say something entirely else. Best regards, Krzysztof
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi index 709081cd1e7f..52c4ee0fa334 100644 --- a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi @@ -788,7 +788,7 @@ pcie1_ep: pcie-ep@2910000 { }; usbss0: cdns-usb@4104000 { - compatible = "ti,j721e-usb"; + compatible = "ti,j7200-usb", "ti,j721e-usb"; reg = <0x00 0x4104000 0x00 0x100>; dma-coherent; power-domains = <&k3_pds 288 TI_SCI_PD_EXCLUSIVE>;
On our platform, suspend-to-idle or suspend-to-RAM turn the controller off. This compatible triggers reset on resume behavior to reconfigure the hardware. Signed-off-by: Théo Lebrun <theo.lebrun@bootlin.com> --- arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)