public inbox for linux-rockchip@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B
@ 2024-11-30  1:40 FUKAUMI Naoki
  2024-11-30  1:40 ` [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom " FUKAUMI Naoki
  2024-12-03 16:18 ` [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST " Heiko Stübner
  0 siblings, 2 replies; 8+ messages in thread
From: FUKAUMI Naoki @ 2024-11-30  1:40 UTC (permalink / raw)
  To: heiko
  Cc: robh, conor+dt, inindev, devicetree, sebastian.reichel,
	linux-rockchip, alchark, FUKAUMI Naoki, krzk+dt, dsimic

Type-C port is dual-role USB 3.0 port. add usb_host0_xhci and phy
nodes to use it as HOST port.

Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
---
 arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
index 0ec4992b43cd..c5776e3b4aab 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
@@ -840,6 +840,14 @@ &tsadc {
 	status = "okay";
 };
 
+&u2phy0 {
+	status = "okay";
+};
+
+&u2phy0_otg {
+	status = "okay";
+};
+
 &u2phy1 {
 	status = "okay";
 };
@@ -883,6 +891,11 @@ &usb_host0_ehci {
 	status = "okay";
 };
 
+&usb_host0_xhci {
+	dr_mode = "host";
+	status = "okay";
+};
+
 &usb_host1_ehci {
 	status = "okay";
 };
@@ -900,6 +913,10 @@ &usb_host2_xhci {
 	status = "okay";
 };
 
+&usbdp_phy0 {
+	status = "okay";
+};
+
 &usbdp_phy1 {
 	status = "okay";
 };
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom port for Radxa ROCK 5B
  2024-11-30  1:40 [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B FUKAUMI Naoki
@ 2024-11-30  1:40 ` FUKAUMI Naoki
  2024-12-03 16:19   ` Heiko Stübner
  2024-12-03 16:18 ` [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST " Heiko Stübner
  1 sibling, 1 reply; 8+ messages in thread
From: FUKAUMI Naoki @ 2024-11-30  1:40 UTC (permalink / raw)
  To: heiko
  Cc: robh, conor+dt, inindev, devicetree, sebastian.reichel,
	linux-rockchip, alchark, FUKAUMI Naoki, krzk+dt, dsimic

add rockchip,dp-lane-mux to usbdp_phy1 to make it work properly.

tested with ELECOM U3HC-A424P10BK[1]:

$ lsusb -t
/:  Bus 001.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
    |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 480M
        |__ Port 003: Dev 003, If 0, Class=Wireless, Driver=btusb, 12M
        |__ Port 003: Dev 003, If 1, Class=Wireless, Driver=btusb, 12M
/:  Bus 002.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M
/:  Bus 003.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
/:  Bus 004.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
/:  Bus 005.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
    |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
/:  Bus 006.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
    |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/5p, 480M
        |__ Port 005: Dev 003, If 0, Class=Billboard, Driver=[none], 480M
/:  Bus 007.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
    |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 5000M
        |__ Port 004: Dev 003, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
/:  Bus 008.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
/:  Bus 009.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
    |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M

[1] https://www.elecom.co.jp/products/U3HC-A424P10BK.html

Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
---
 arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
index c5776e3b4aab..43fc1f24dfc9 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
@@ -918,6 +918,7 @@ &usbdp_phy0 {
 };
 
 &usbdp_phy1 {
+	rockchip,dp-lane-mux = <2 3>;
 	status = "okay";
 };
 
-- 
2.43.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B
  2024-11-30  1:40 [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B FUKAUMI Naoki
  2024-11-30  1:40 ` [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom " FUKAUMI Naoki
@ 2024-12-03 16:18 ` Heiko Stübner
  2024-12-03 17:11   ` Sebastian Reichel
  1 sibling, 1 reply; 8+ messages in thread
From: Heiko Stübner @ 2024-12-03 16:18 UTC (permalink / raw)
  To: FUKAUMI Naoki
  Cc: robh, conor+dt, inindev, devicetree, sebastian.reichel,
	linux-rockchip, alchark, FUKAUMI Naoki, krzk+dt, dsimic

Am Samstag, 30. November 2024, 02:40:42 CET schrieb FUKAUMI Naoki:
> Type-C port is dual-role USB 3.0 port. add usb_host0_xhci and phy
> nodes to use it as HOST port.

This needs a bit more explanation on _why_ you're statically
setting this to host-mode.

Heiko

> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
> ---
>  arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> index 0ec4992b43cd..c5776e3b4aab 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> @@ -840,6 +840,14 @@ &tsadc {
>  	status = "okay";
>  };
>  
> +&u2phy0 {
> +	status = "okay";
> +};
> +
> +&u2phy0_otg {
> +	status = "okay";
> +};
> +
>  &u2phy1 {
>  	status = "okay";
>  };
> @@ -883,6 +891,11 @@ &usb_host0_ehci {
>  	status = "okay";
>  };
>  
> +&usb_host0_xhci {
> +	dr_mode = "host";
> +	status = "okay";
> +};
> +
>  &usb_host1_ehci {
>  	status = "okay";
>  };
> @@ -900,6 +913,10 @@ &usb_host2_xhci {
>  	status = "okay";
>  };
>  
> +&usbdp_phy0 {
> +	status = "okay";
> +};
> +
>  &usbdp_phy1 {
>  	status = "okay";
>  };
> 





_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom port for Radxa ROCK 5B
  2024-11-30  1:40 ` [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom " FUKAUMI Naoki
@ 2024-12-03 16:19   ` Heiko Stübner
  2024-12-03 17:01     ` Sebastian Reichel
  0 siblings, 1 reply; 8+ messages in thread
From: Heiko Stübner @ 2024-12-03 16:19 UTC (permalink / raw)
  To: FUKAUMI Naoki
  Cc: robh, conor+dt, inindev, devicetree, sebastian.reichel,
	linux-rockchip, alchark, FUKAUMI Naoki, krzk+dt, dsimic

Am Samstag, 30. November 2024, 02:40:43 CET schrieb FUKAUMI Naoki:
> add rockchip,dp-lane-mux to usbdp_phy1 to make it work properly.

Make it work properly is not an ideal commit description.

I guess the lane assignment for the dp port should be discoverable
in the device schematics, so the commit description should explain
as much.


Heiko

> tested with ELECOM U3HC-A424P10BK[1]:
> 
> $ lsusb -t
> /:  Bus 001.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
>     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 480M
>         |__ Port 003: Dev 003, If 0, Class=Wireless, Driver=btusb, 12M
>         |__ Port 003: Dev 003, If 1, Class=Wireless, Driver=btusb, 12M
> /:  Bus 002.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M
> /:  Bus 003.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
> /:  Bus 004.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
> /:  Bus 005.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>     |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> /:  Bus 006.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
>     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/5p, 480M
>         |__ Port 005: Dev 003, If 0, Class=Billboard, Driver=[none], 480M
> /:  Bus 007.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 5000M
>         |__ Port 004: Dev 003, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> /:  Bus 008.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
> /:  Bus 009.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>     |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> 
> [1] https://www.elecom.co.jp/products/U3HC-A424P10BK.html
> 
> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
> ---
>  arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> index c5776e3b4aab..43fc1f24dfc9 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> @@ -918,6 +918,7 @@ &usbdp_phy0 {
>  };
>  
>  &usbdp_phy1 {
> +	rockchip,dp-lane-mux = <2 3>;
>  	status = "okay";
>  };
>  
> 





_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom port for Radxa ROCK 5B
  2024-12-03 16:19   ` Heiko Stübner
@ 2024-12-03 17:01     ` Sebastian Reichel
  2024-12-04  1:33       ` FUKAUMI Naoki
  0 siblings, 1 reply; 8+ messages in thread
From: Sebastian Reichel @ 2024-12-03 17:01 UTC (permalink / raw)
  To: Heiko Stübner
  Cc: robh, conor+dt, inindev, devicetree, linux-rockchip, alchark,
	FUKAUMI Naoki, krzk+dt, dsimic


[-- Attachment #1.1: Type: text/plain, Size: 3064 bytes --]

Hi,

On Tue, Dec 03, 2024 at 05:19:55PM +0100, Heiko Stübner wrote:
> Am Samstag, 30. November 2024, 02:40:43 CET schrieb FUKAUMI Naoki:
> > add rockchip,dp-lane-mux to usbdp_phy1 to make it work properly.
> 
> Make it work properly is not an ideal commit description.
> 
> I guess the lane assignment for the dp port should be discoverable
> in the device schematics, so the commit description should explain
> as much.

Rock 5B does not have SSRX2P/SSRX2N and SSTX2P/SSTX2N connected to
anything, so this description is just wrong. The other lanes are
connected to a USB-A port and thus no USB-C controller is involved,
which takes care of any muxing. Not specifying any
rockchip,dp-lane-mux configuration sets mode to UDPHY_MODE_USB,
which is correct.

Naoki, if you see any problems, it is most likely a bug in the USBDP
PHY or DWC3 driver.

Greetings,

-- Sebastian

> 
> > tested with ELECOM U3HC-A424P10BK[1]:
> > 
> > $ lsusb -t
> > /:  Bus 001.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
> >     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 480M
> >         |__ Port 003: Dev 003, If 0, Class=Wireless, Driver=btusb, 12M
> >         |__ Port 003: Dev 003, If 1, Class=Wireless, Driver=btusb, 12M
> > /:  Bus 002.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M
> > /:  Bus 003.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
> > /:  Bus 004.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
> > /:  Bus 005.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
> >     |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> > /:  Bus 006.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
> >     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/5p, 480M
> >         |__ Port 005: Dev 003, If 0, Class=Billboard, Driver=[none], 480M
> > /:  Bus 007.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
> >     |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 5000M
> >         |__ Port 004: Dev 003, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> > /:  Bus 008.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
> > /:  Bus 009.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
> >     |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
> > 
> > [1] https://www.elecom.co.jp/products/U3HC-A424P10BK.html
> > 
> > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
> > ---
> >  arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > index c5776e3b4aab..43fc1f24dfc9 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > @@ -918,6 +918,7 @@ &usbdp_phy0 {
> >  };
> >  
> >  &usbdp_phy1 {
> > +	rockchip,dp-lane-mux = <2 3>;
> >  	status = "okay";
> >  };
> >  
> > 
> 
> 
> 
> 

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 170 bytes --]

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B
  2024-12-03 16:18 ` [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST " Heiko Stübner
@ 2024-12-03 17:11   ` Sebastian Reichel
  2024-12-04  1:31     ` FUKAUMI Naoki
  0 siblings, 1 reply; 8+ messages in thread
From: Sebastian Reichel @ 2024-12-03 17:11 UTC (permalink / raw)
  To: Heiko Stübner
  Cc: robh, conor+dt, inindev, devicetree, linux-rockchip, alchark,
	FUKAUMI Naoki, krzk+dt, dsimic


[-- Attachment #1.1: Type: text/plain, Size: 2171 bytes --]

Hi,

On Tue, Dec 03, 2024 at 05:18:39PM +0100, Heiko Stübner wrote:
> Am Samstag, 30. November 2024, 02:40:42 CET schrieb FUKAUMI Naoki:
> > Type-C port is dual-role USB 3.0 port. add usb_host0_xhci and phy
> > nodes to use it as HOST port.
> 
> This needs a bit more explanation on _why_ you're statically
> setting this to host-mode.

The USB-C port can be used in peripheral mode, so this is just
wrong. Also it should be added together with the USB PD controller
for proper muxing and role switch support. The reason I have not yet
send the patch is that this needed a fix in U-Boot first.

Most users are powering their Rock 5Bs via the USB-C port and a
bad description results in unexpected hardware resets at boot
time.

I mostly waited for the rc1 tag to happen now that U-Boot support
has landed. Also I expect that even with all those precautions
some people will start seeing issues. Unfortunately there are
many non-compliant USB-PD power sources :(

-- Sebastian

> 
> Heiko
> 
> > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
> > ---
> >  arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > index 0ec4992b43cd..c5776e3b4aab 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
> > @@ -840,6 +840,14 @@ &tsadc {
> >  	status = "okay";
> >  };
> >  
> > +&u2phy0 {
> > +	status = "okay";
> > +};
> > +
> > +&u2phy0_otg {
> > +	status = "okay";
> > +};
> > +
> >  &u2phy1 {
> >  	status = "okay";
> >  };
> > @@ -883,6 +891,11 @@ &usb_host0_ehci {
> >  	status = "okay";
> >  };
> >  
> > +&usb_host0_xhci {
> > +	dr_mode = "host";
> > +	status = "okay";
> > +};
> > +
> >  &usb_host1_ehci {
> >  	status = "okay";
> >  };
> > @@ -900,6 +913,10 @@ &usb_host2_xhci {
> >  	status = "okay";
> >  };
> >  
> > +&usbdp_phy0 {
> > +	status = "okay";
> > +};
> > +
> >  &usbdp_phy1 {
> >  	status = "okay";
> >  };
> > 
> 
> 
> 
> 

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 170 bytes --]

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B
  2024-12-03 17:11   ` Sebastian Reichel
@ 2024-12-04  1:31     ` FUKAUMI Naoki
  0 siblings, 0 replies; 8+ messages in thread
From: FUKAUMI Naoki @ 2024-12-04  1:31 UTC (permalink / raw)
  To: Sebastian Reichel, Heiko Stübner
  Cc: robh, conor+dt, inindev, devicetree, linux-rockchip, alchark,
	krzk+dt, dsimic

Hi Heiko and Sebastian,

thank you for your comments!

On 12/4/24 02:11, Sebastian Reichel wrote:
> Hi,
> 
> On Tue, Dec 03, 2024 at 05:18:39PM +0100, Heiko Stübner wrote:
>> Am Samstag, 30. November 2024, 02:40:42 CET schrieb FUKAUMI Naoki:
>>> Type-C port is dual-role USB 3.0 port. add usb_host0_xhci and phy
>>> nodes to use it as HOST port.
>>
>> This needs a bit more explanation on _why_ you're statically
>> setting this to host-mode.
> 
> The USB-C port can be used in peripheral mode, so this is just
> wrong. Also it should be added together with the USB PD controller
> for proper muxing and role switch support. The reason I have not yet
> send the patch is that this needed a fix in U-Boot first.
> 
> Most users are powering their Rock 5Bs via the USB-C port and a
> bad description results in unexpected hardware resets at boot
> time.
> 
> I mostly waited for the rc1 tag to happen now that U-Boot support
> has landed. Also I expect that even with all those precautions
> some people will start seeing issues. Unfortunately there are
> many non-compliant USB-PD power sources :(

I see. I'll wait proper implementation.
please ignore my patch.

Best regards,

--
FUKAUMI Naoki
Radxa Computer (Shenzhen) Co., Ltd.

> -- Sebastian
> 
>>
>> Heiko
>>
>>> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
>>> ---
>>>   arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 17 +++++++++++++++++
>>>   1 file changed, 17 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> index 0ec4992b43cd..c5776e3b4aab 100644
>>> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> @@ -840,6 +840,14 @@ &tsadc {
>>>   	status = "okay";
>>>   };
>>>   
>>> +&u2phy0 {
>>> +	status = "okay";
>>> +};
>>> +
>>> +&u2phy0_otg {
>>> +	status = "okay";
>>> +};
>>> +
>>>   &u2phy1 {
>>>   	status = "okay";
>>>   };
>>> @@ -883,6 +891,11 @@ &usb_host0_ehci {
>>>   	status = "okay";
>>>   };
>>>   
>>> +&usb_host0_xhci {
>>> +	dr_mode = "host";
>>> +	status = "okay";
>>> +};
>>> +
>>>   &usb_host1_ehci {
>>>   	status = "okay";
>>>   };
>>> @@ -900,6 +913,10 @@ &usb_host2_xhci {
>>>   	status = "okay";
>>>   };
>>>   
>>> +&usbdp_phy0 {
>>> +	status = "okay";
>>> +};
>>> +
>>>   &usbdp_phy1 {
>>>   	status = "okay";
>>>   };
>>>
>>
>>
>>
>>


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom port for Radxa ROCK 5B
  2024-12-03 17:01     ` Sebastian Reichel
@ 2024-12-04  1:33       ` FUKAUMI Naoki
  0 siblings, 0 replies; 8+ messages in thread
From: FUKAUMI Naoki @ 2024-12-04  1:33 UTC (permalink / raw)
  To: Sebastian Reichel, Heiko Stübner
  Cc: robh, conor+dt, inindev, devicetree, linux-rockchip, alchark,
	krzk+dt, dsimic

Hi,

On 12/4/24 02:01, Sebastian Reichel wrote:
> Hi,
> 
> On Tue, Dec 03, 2024 at 05:19:55PM +0100, Heiko Stübner wrote:
>> Am Samstag, 30. November 2024, 02:40:43 CET schrieb FUKAUMI Naoki:
>>> add rockchip,dp-lane-mux to usbdp_phy1 to make it work properly.
>>
>> Make it work properly is not an ideal commit description.
>>
>> I guess the lane assignment for the dp port should be discoverable
>> in the device schematics, so the commit description should explain
>> as much.
> 
> Rock 5B does not have SSRX2P/SSRX2N and SSTX2P/SSTX2N connected to
> anything, so this description is just wrong. The other lanes are
> connected to a USB-A port and thus no USB-C controller is involved,
> which takes care of any muxing. Not specifying any
> rockchip,dp-lane-mux configuration sets mode to UDPHY_MODE_USB,
> which is correct.
> 
> Naoki, if you see any problems, it is most likely a bug in the USBDP
> PHY or DWC3 driver.

sorry, I should write error message. I understand it, please ignore my 
patch.

Best regards,

--
FUKAUMI Naoki
Radxa Computer (Shenzhen) Co., Ltd.

> Greetings,
> 
> -- Sebastian
> 
>>
>>> tested with ELECOM U3HC-A424P10BK[1]:
>>>
>>> $ lsusb -t
>>> /:  Bus 001.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
>>>      |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 480M
>>>          |__ Port 003: Dev 003, If 0, Class=Wireless, Driver=btusb, 12M
>>>          |__ Port 003: Dev 003, If 1, Class=Wireless, Driver=btusb, 12M
>>> /:  Bus 002.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M
>>> /:  Bus 003.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M
>>> /:  Bus 004.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
>>> /:  Bus 005.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>>>      |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
>>> /:  Bus 006.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
>>>      |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/5p, 480M
>>>          |__ Port 005: Dev 003, If 0, Class=Billboard, Driver=[none], 480M
>>> /:  Bus 007.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>>>      |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 5000M
>>>          |__ Port 004: Dev 003, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
>>> /:  Bus 008.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M
>>> /:  Bus 009.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M
>>>      |__ Port 001: Dev 002, If 0, Class=Mass Storage, Driver=usb-storage, 5000M
>>>
>>> [1] https://www.elecom.co.jp/products/U3HC-A424P10BK.html
>>>
>>> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
>>> ---
>>>   arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 +
>>>   1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> index c5776e3b4aab..43fc1f24dfc9 100644
>>> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
>>> @@ -918,6 +918,7 @@ &usbdp_phy0 {
>>>   };
>>>   
>>>   &usbdp_phy1 {
>>> +	rockchip,dp-lane-mux = <2 3>;
>>>   	status = "okay";
>>>   };
>>>   
>>>
>>
>>
>>
>>


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2024-12-04  1:35 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-11-30  1:40 [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST port for Radxa ROCK 5B FUKAUMI Naoki
2024-11-30  1:40 ` [PATCH 2/2] arm64: dts: rockchip: fix USB 3.0 bottom " FUKAUMI Naoki
2024-12-03 16:19   ` Heiko Stübner
2024-12-03 17:01     ` Sebastian Reichel
2024-12-04  1:33       ` FUKAUMI Naoki
2024-12-03 16:18 ` [PATCH 1/2] arm64: dts: rockchip: use Type-C port as USB HOST " Heiko Stübner
2024-12-03 17:11   ` Sebastian Reichel
2024-12-04  1:31     ` FUKAUMI Naoki

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox