[PATCH v6 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.
EHCI probing in Rock pi 4 currently fails. Add a clock driver for usb2phy so that probing EHCI does not fail when missing one of the clocks in the bundle for usb_host0_ehci, since usb2phy is UCLASS_PHY but not UCLASS_CLK. Xavier Drudis Ferran (2): arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2. arm: rk3399: usb2phy: phy-rockchip-inno-usb2.c: Implement operations for the 480MHz usb2phy clock in rk3399. drivers/phy/rockchip/phy-rockchip-inno-usb2.c | 97 ++- 1 file changed, 96 insertions(+), 1 deletion(-) Cc: Simon Glass Cc: Philipp Tomsich Cc: Kever Yang Cc: Lukasz Majewski Cc: Sean Anderson Cc: Marek Vasut Cc: Christoph Fritz Cc: Jagan Teki Signed-off-by: Xavier Drudis Ferran --- Changes: v6: just retested over current next branch and some corrections to message and headers (no changes to code). v5: fixes a bug that Christoph Fritz discovered, consisting in the wrong eror code returned when enabling or disabling the clock because property_enable() returns an error code in linux but the modified register value in U-Boot. This caused the clk disable to abort before freeing the clock. v4: move v3 to one patch in the series and add a second patch to add operations to enable disable the usb2phy 480Mhz clock. Also, honour clock-output-names for what is worth. v3: implement option 5 (bind usb2phy as a clk driver too) instead of option 1 (ehci-generic.c tolerates missing clocks). v2: implement option 1 (ehci-generic.c tolerates missing clocks) instead of option 3 (change dts node to remove the missing clock). -- 2.20.1
Re: [PATCH v6 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.
On Sun, Jun 4, 2023 at 1:42 PM Xavier Drudis Ferran wrote: > > EHCI probing in Rock pi 4 currently fails. > > Add a clock driver for usb2phy so that probing EHCI does not fail when > missing one of the clocks in the bundle for usb_host0_ehci, since > usb2phy is UCLASS_PHY but not UCLASS_CLK. > > Xavier Drudis Ferran (2): > arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to > access peripherals by USB 2. > arm: rk3399: usb2phy: phy-rockchip-inno-usb2.c: Implement operations > for the 480MHz usb2phy clock in rk3399. Please note that commit head on both the patches seems improper to me. Commit body looks fine, but the head should start phy: rockchip-inno-usb2: Add usb2phy clock provider of 480MHz clock phy: rockchip-inno-usb2: Implement clock operations for usb2phy clock Thanks, Jagan.
Re: [PATCH v6 0/2] arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to access peripherals by USB 2.
El Mon, Jun 05, 2023 at 08:11:07AM +0530, Jagan Teki deia: > On Sun, Jun 4, 2023 at 1:42 PM Xavier Drudis Ferran wrote: > > > > EHCI probing in Rock pi 4 currently fails. > > > > Add a clock driver for usb2phy so that probing EHCI does not fail when > > missing one of the clocks in the bundle for usb_host0_ehci, since > > usb2phy is UCLASS_PHY but not UCLASS_CLK. > > > > Xavier Drudis Ferran (2): > > arm: dts: rockchip: rk3399: usb: ehci: Fix EHCI probe in rk3399 to > > access peripherals by USB 2. > > arm: rk3399: usb2phy: phy-rockchip-inno-usb2.c: Implement operations > > for the 480MHz usb2phy clock in rk3399. > > Please note that commit head on both the patches seems improper to me. > > Commit body looks fine, but the head should start > > phy: rockchip-inno-usb2: Add usb2phy clock provider of 480MHz clock > phy: rockchip-inno-usb2: Implement clock operations for usb2phy clock > > Thanks, > Jagan. Done, thank you. I hope it's right now.