On Fri, Nov 14, 2025 at 12:08:02PM +0530, Balaji Selvanathan wrote: > Remove GCC_USB3_PRIM_CLKREF_CLK from the USB controller node as it is > not implemented in the U-Boot clock driver. Keep only the supported > clocks to avoid clock warnings during boot.
Rather than overriding DT, why can't this be implmented in U-Boot? And how why it isn't an essential clock for U-Boot operation? -Sumit > > Signed-off-by: Balaji Selvanathan <[email protected]> > --- > arch/arm/dts/qcs615-ride-u-boot.dtsi | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/arch/arm/dts/qcs615-ride-u-boot.dtsi > b/arch/arm/dts/qcs615-ride-u-boot.dtsi > index 68fffc70fcb..d8af124cc27 100644 > --- a/arch/arm/dts/qcs615-ride-u-boot.dtsi > +++ b/arch/arm/dts/qcs615-ride-u-boot.dtsi > @@ -11,4 +11,18 @@ > <0x0 0xc0000000 0x0 0xc0000000>, > <0x1 0x80000000 0x1 0x00000000>; > }; > + soc@0 { > + /* Remove GCC_USB3_PRIM_CLKREF_CLK from usb_1 node to avoid > U-Boot clock warnings */ > + usb_1: usb@a6f8800 { > + /delete-property/ clocks; > + /delete-property/ clock-names; > + /* Keep only the clocks that are implemented in U-Boot > clock driver */ > + clocks = <&gcc GCC_CFG_NOC_USB3_PRIM_AXI_CLK>, > + <&gcc GCC_USB30_PRIM_MASTER_CLK>, > + <&gcc GCC_AGGRE_USB3_PRIM_AXI_CLK>, > + <&gcc GCC_USB30_PRIM_SLEEP_CLK>, > + <&gcc GCC_USB30_PRIM_MOCK_UTMI_CLK>; > + clock-names = "cfg_noc", "core", "iface", "sleep", > "mock_utmi"; > + }; > + }; > }; > -- > 2.34.1 >
