Re: [PATCH 6/6] arm64: dts: imx95-19x19-evk: enable USB2.0 node

[Xu Yang <xu.yang_2@nxp.com>]

On Wed, Feb 19, 2025 at 03:39:08PM -0500, Frank Li wrote:
> On Wed, Feb 19, 2025 at 05:31:04PM +0800, Xu Yang wrote:
> > On this board, USB2.0 is a host-only port, add vbus regulator node
> > and enable USB2.0 node.
> >
> > Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
> > ---
> >  .../arm64/boot/dts/freescale/imx95-19x19-evk.dts | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> > index 25ac331f0318..0505cfe2778f 100644
> > --- a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> > @@ -133,6 +133,15 @@ reg_slot_pwr: regulator-slot-pwr {
> >  		enable-active-high;
> >  	};
> >
> > +	reg_usb_vbus: regulator-vbus {
> > +		compatible = "regulator-fixed";
> > +		regulator-name = "USB_VBUS";
> > +		regulator-min-microvolt = <5000000>;
> > +		regulator-max-microvolt = <5000000>;
> > +		gpio = <&i2c7_pcal6524 3 GPIO_ACTIVE_HIGH>;
> > +		enable-active-high;
> > +	};
> > +
> 
> try alphabet order, regulator-vbus should after regulator-usdhc2

Okay, will do.

Thanks,
Xu Yang

> 
> Reviewed-by: Frank Li <Frank.Li@nxp.com>
> 
> 
> >  	reg_usdhc2_vmmc: regulator-usdhc2 {
> >  		compatible = "regulator-fixed";
> >  		pinctrl-names = "default";
> > @@ -461,6 +470,13 @@ &sai3 {
> >  	status = "okay";
> >  };
> >
> > +&usb2 {
> > +	dr_mode = "host";
> > +	disable-over-current;
> > +	vbus-supply = <&reg_usb_vbus>;
> > +	status = "okay";
> > +};
> > +
> >  &usb3 {
> >  	status = "okay";
> >  };
> > --
> > 2.34.1
> >