On 11/03/2024 13:08, Komal Bajaj wrote: > @@ -6,6 +6,7 @@ > #include <dt-bindings/clock/qcom,qdu1000-gcc.h> > #include <dt-bindings/clock/qcom,rpmh.h> > #include <dt-bindings/dma/qcom-gpi.h> > +#include <dt-bindings/interconnect/qcom,icc.h> > #include <dt-bindings/interconnect/qcom,qdu1000-rpmh.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/power/qcom-rpmpd.h> > @@ -913,6 +914,124 @@ opp-384000000 { > }; > }; > > + usb_1_hsphy: phy@88e3000 { > + compatible = "qcom,qdu1000-usb-hs-phy", > + "qcom,usb-snps-hs-7nm-phy"; > + reg = <0x0 0x088e3000 0x0 0x120>; > + #phy-cells = <0>; > + > + clocks =<&gcc GCC_USB2_CLKREF_EN>; > + clock-names = "ref"; > + > + resets = <&gcc GCC_QUSB2PHY_PRIM_BCR>; > + > + status = "disabled"; > + }; > + > + usb_1_qmpphy: phy-wrapper@88e5000 { That's a phy, isn't it? So node name "phy". Best regards, Krzysztof