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

[Frank Li <Frank.li@xxxxxxx>]

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@xxxxxxx>
> ---
>  .../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

Reviewed-by: Frank Li <Frank.Li@xxxxxxx>


>  	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
>