Re: [PATCH v1 2/3] arm64: dts: rockchip: rk3588-rock-5b-5bp-5t: Fix USB host phy-supply on Rock 5b-5bp-5t SbC
From: Anand Moon <hidden>
Date: 2026-01-11 19:31:16
Also in:
linux-devicetree, linux-rockchip, linux-usb, lkml
Hi Sebastian, I am having difficulty articulating my thoughts clearly right now; please bear with me as I try to explain. On Sat, 10 Jan 2026 at 05:03, Sebastian Reichel [off-list ref] wrote:
Hi, On Thu, Jan 08, 2026 at 12:25:12PM +0530, Anand Moon wrote:quoted
Thanks for your review comments. On Sat, 3 Jan 2026 at 19:35, Sebastian Reichel wrote:quoted
On Sat, Jan 03, 2026 at 02:01:18PM +0530, Anand Moon wrote:quoted
The USB host USB2 and USB3 on the Rock 5B, 5BP, and 5T is powered by the USB_HOST_PWREN_H gpio pin which is used to enable vcc5v0_host host regulator which ensures proper power sequencing and management for onboard Double-USB-HOST. Update the u2phy2_host node to reference the correct phy-supply. Cc: Sebastian Reichel <redacted> Signed-off-by: Anand Moon <redacted> ---u2phy2_host is connected to an onboard USB hub chip, which is powered by &vcc5v0_sys as the comment says. You can see this on page 9 of the v1.45 ROCK 5B schematics (page name USB2HUB). The usb hub chip uses VCC5V0_SYS + VCC_3V3_S0 and is connected to USB20_HOST0_DP, USB20_HOST0_DM. The related USB controller for that in the upstream DT is &usb_host0_ehci, which in turn is using &u2phy2_host. Thus at least for the ROCK 5B the existing DT is correct. Properly describing the downstream port VBUS regulator would require something like the following series: https://lore.kernel.org/all/20250911-v6-16-topic-usb-onboard-dev-v4-0-1af288125d74@pengutronix.de/ (local)Actually, the Radxa Rock 5 B doesn't have an onboard USB hub chip.Seriously? I provided the exact page for the hub chip in the schematics and it's easily visisble in lsusb too.
My apologies for the confusion. To clarify, this board lacks a USB hub chip that matches the detected USB IDs and is capable of powering both USB 2.0 and USB 3.0 interfaces. [1] https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/usb/microchip%2Cusb2514.yaml This is the reason we have a separate power regulator for USB 2.0 and USB 3.0 ports and Type-C controllers. There is a USB2HUB -> U90078 FE1.1s_QFN chip (page 27) in the schematics, It's probably connected to the PCIE M.2 NGFF for Bluetooth [2] https://www.snapeda.com/parts/FE1.1s-BQFN24B/Terminus-Tech/datasheet/ $ lsusb -tv /: Bus 001.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M ID 1d6b:0002 Linux Foundation 2.0 root hub |__ Port 001: Dev 002, If 0, Class=Hub, Driver=hub/4p, 480M ID 1a40:0101 Terminus Technology Inc. Hub |__ Port 003: Dev 003, If 0, Class=Wireless, Driver=btusb, 12M ID 8087:0a2b Intel Corp. Bluetooth wireless interface |__ Port 003: Dev 003, If 1, Class=Wireless, Driver=btusb, 12M ID 8087:0a2b Intel Corp. Bluetooth wireless interface /: Bus 002.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M ID 1d6b:0001 Linux Foundation 1.1 root hub /: Bus 003.Port 001: Dev 001, Class=root_hub, Driver=ehci-platform/1p, 480M ID 1d6b:0002 Linux Foundation 2.0 root hub /: Bus 004.Port 001: Dev 001, Class=root_hub, Driver=ohci-platform/1p, 12M ID 1d6b:0001 Linux Foundation 1.1 root hub /: Bus 005.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M ID 1d6b:0002 Linux Foundation 2.0 root hub /: Bus 006.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M ID 1d6b:0003 Linux Foundation 3.0 root hub /: Bus 007.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 480M ID 1d6b:0002 Linux Foundation 2.0 root hub /: Bus 008.Port 001: Dev 001, Class=root_hub, Driver=xhci-hcd/1p, 5000M ID 1d6b:0003 Linux Foundation 3.0 root hub
I don't want to waste my time and expect a certain level of quality and this is certainly way below it...
Please accept my apologies for the miscommunication. Could you check the enable status of the USB_HOST_PWREN_H pin in the schematics? USB_HOST_PWREN_H -> VCC5V0_USB20_HOST0 / VCC5V0_USB20_HOST2 USB_HOST_PWREN_H -> VCC5V0_USB30_HOST0 / VCC5V0_USB30_HOST1 My current patch is trying to address this issue.
-- Sebastian
Thanks -Anand