From: Roger Quadros <rogerq@kernel.org> To: Andrejs Cainikovs <andrejs.cainikovs@gmail.com>, Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>, Tero Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Aswath Govindraju <a-govindraju@ti.com>, Sjoerd Simons <sjoerd@collabora.com> Cc: Andrejs Cainikovs <andrejs.cainikovs@toradex.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, "Bin Liu [EP]" <b-liu@ti.com>, "Gunasekaran, Ravi" <r-gunasekaran@ti.com> Subject: Re: [PATCH v1] arm64: dts: ti: k3-am62-main: disable usb lpm Date: Mon, 12 Feb 2024 14:13:56 +0200 [thread overview] Message-ID: <2629cd30-23aa-4f03-8452-ae13297fd6b6@kernel.org> (raw) In-Reply-To: <20240209130213.38908-1-andrejs.cainikovs@gmail.com> Hi Andrejs, On 09/02/2024 15:02, Andrejs Cainikovs wrote: > From: Andrejs Cainikovs <andrejs.cainikovs@toradex.com> > > AM62 USB works with some devices, while failing to operate with others. Could you please share which devices failed to work for you? > > [ 560.189822] xhci-hcd xhci-hcd.4.auto: xHCI Host Controller > [ 560.195631] xhci-hcd xhci-hcd.4.auto: new USB bus registered, assigned bus number 2 > [ 574.388509] xhci-hcd xhci-hcd.4.auto: can't setup: -110 > [ 574.393814] xhci-hcd xhci-hcd.4.auto: USB bus 2 deregistered > [ 574.399544] xhci-hcd: probe of xhci-hcd.4.auto failed with error -110 > > This seems to be related to LPM (Link Power Management), and disabling it > turns USB into reliable working state. > > As per AM62 reference manual: > >> 4.8.2.1 USB2SS Unsupported Features >> >> The following features are not supported on this family of devices: >> ... >> - USB 2.0 ECN: Link Power Management (LPM) >> ... > > Fixes: 2240f96cf3cd ("arm64: dts: ti: k3-am62-main: Add support for USB") > Signed-off-by: Andrejs Cainikovs <andrejs.cainikovs@toradex.com> > --- > arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > index 464b7565d085..c49fbce5cb70 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > @@ -640,6 +640,8 @@ usb0: usb@31000000 { > interrupt-names = "host", "peripheral"; > maximum-speed = "high-speed"; > dr_mode = "otg"; > + snps,usb2-gadget-lpm-disable; > + snps,usb2-lpm-disable; > }; > }; > > @@ -663,6 +665,8 @@ usb1: usb@31100000 { > interrupt-names = "host", "peripheral"; > maximum-speed = "high-speed"; > dr_mode = "otg"; > + snps,usb2-gadget-lpm-disable; > + snps,usb2-lpm-disable; > }; > }; > Instead of this could you please check if this series fixes the issue for you? https://lore.kernel.org/all/20240205141221.56076-1-rogerq@kernel.org/ -- cheers, -roger
WARNING: multiple messages have this Message-ID (diff)
From: Roger Quadros <rogerq@kernel.org> To: Andrejs Cainikovs <andrejs.cainikovs@gmail.com>, Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>, Tero Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Aswath Govindraju <a-govindraju@ti.com>, Sjoerd Simons <sjoerd@collabora.com> Cc: Andrejs Cainikovs <andrejs.cainikovs@toradex.com>, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, "Bin Liu [EP]" <b-liu@ti.com>, "Gunasekaran, Ravi" <r-gunasekaran@ti.com> Subject: Re: [PATCH v1] arm64: dts: ti: k3-am62-main: disable usb lpm Date: Mon, 12 Feb 2024 14:13:56 +0200 [thread overview] Message-ID: <2629cd30-23aa-4f03-8452-ae13297fd6b6@kernel.org> (raw) In-Reply-To: <20240209130213.38908-1-andrejs.cainikovs@gmail.com> Hi Andrejs, On 09/02/2024 15:02, Andrejs Cainikovs wrote: > From: Andrejs Cainikovs <andrejs.cainikovs@toradex.com> > > AM62 USB works with some devices, while failing to operate with others. Could you please share which devices failed to work for you? > > [ 560.189822] xhci-hcd xhci-hcd.4.auto: xHCI Host Controller > [ 560.195631] xhci-hcd xhci-hcd.4.auto: new USB bus registered, assigned bus number 2 > [ 574.388509] xhci-hcd xhci-hcd.4.auto: can't setup: -110 > [ 574.393814] xhci-hcd xhci-hcd.4.auto: USB bus 2 deregistered > [ 574.399544] xhci-hcd: probe of xhci-hcd.4.auto failed with error -110 > > This seems to be related to LPM (Link Power Management), and disabling it > turns USB into reliable working state. > > As per AM62 reference manual: > >> 4.8.2.1 USB2SS Unsupported Features >> >> The following features are not supported on this family of devices: >> ... >> - USB 2.0 ECN: Link Power Management (LPM) >> ... > > Fixes: 2240f96cf3cd ("arm64: dts: ti: k3-am62-main: Add support for USB") > Signed-off-by: Andrejs Cainikovs <andrejs.cainikovs@toradex.com> > --- > arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > index 464b7565d085..c49fbce5cb70 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi > @@ -640,6 +640,8 @@ usb0: usb@31000000 { > interrupt-names = "host", "peripheral"; > maximum-speed = "high-speed"; > dr_mode = "otg"; > + snps,usb2-gadget-lpm-disable; > + snps,usb2-lpm-disable; > }; > }; > > @@ -663,6 +665,8 @@ usb1: usb@31100000 { > interrupt-names = "host", "peripheral"; > maximum-speed = "high-speed"; > dr_mode = "otg"; > + snps,usb2-gadget-lpm-disable; > + snps,usb2-lpm-disable; > }; > }; > Instead of this could you please check if this series fixes the issue for you? https://lore.kernel.org/all/20240205141221.56076-1-rogerq@kernel.org/ -- cheers, -roger _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2024-02-12 12:14 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-02-09 13:02 [PATCH v1] arm64: dts: ti: k3-am62-main: disable usb lpm Andrejs Cainikovs 2024-02-09 13:02 ` Andrejs Cainikovs 2024-02-12 12:13 ` Roger Quadros [this message] 2024-02-12 12:13 ` Roger Quadros 2024-02-12 12:54 ` Francesco Dolcini 2024-02-12 12:54 ` Francesco Dolcini 2024-02-13 11:34 ` Roger Quadros 2024-02-13 11:34 ` Roger Quadros 2024-02-14 9:46 ` Andrejs Cainikovs 2024-02-14 9:46 ` Andrejs Cainikovs 2024-02-13 11:43 ` Francesco Dolcini 2024-02-13 11:43 ` Francesco Dolcini 2024-02-14 9:25 ` Roger Quadros 2024-02-14 9:25 ` Roger Quadros 2024-02-15 9:45 ` Vignesh Raghavendra 2024-02-15 9:45 ` Vignesh Raghavendra
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=2629cd30-23aa-4f03-8452-ae13297fd6b6@kernel.org \ --to=rogerq@kernel.org \ --cc=a-govindraju@ti.com \ --cc=andrejs.cainikovs@gmail.com \ --cc=andrejs.cainikovs@toradex.com \ --cc=b-liu@ti.com \ --cc=conor+dt@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=kristo@kernel.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=nm@ti.com \ --cc=r-gunasekaran@ti.com \ --cc=robh+dt@kernel.org \ --cc=sjoerd@collabora.com \ --cc=vigneshr@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.