[PATCH v4] dts: rockpro64: Disable usb regulators-always-on

Kever Yang kever.yang at rock-chips.com
Thu Jan 18 05:57:52 CET 2024


Hi Shantur,

     Since you are working on the dts change in mainline kernel, you can 
sync the dts after kernel apply.

This patch will be dropped.


Thanks,

- Kever

On 2023/12/22 22:42, Shantur Rathore wrote:
> USB port regulators should be controlled by PHYs
> so we remove always-on property and let phy manage the
> regulator.
>
> phy-supply isn't configured for TypeC port in upstream and
> now that we are removing always-on, we need to add the
> phy-supply until its fixed upstream.
>
> Signed-off-by: Shantur Rathore <i at shantur.com>
>
> ---
>
> Changes in v4:
> - otg port must be configured with phy-supply instead of host port node
>
> Changes in v3:
> - Split up patches as seperate series
>
> Changes in v2:
> - As requested, added fix for regulator-always-on in RockPro64
>
>   arch/arm/dts/rk3399-rockpro64-u-boot.dtsi | 12 ++++++++++++
>   1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm/dts/rk3399-rockpro64-u-boot.dtsi b/arch/arm/dts/rk3399-rockpro64-u-boot.dtsi
> index 732727d9b0..0ac9fa9a03 100644
> --- a/arch/arm/dts/rk3399-rockpro64-u-boot.dtsi
> +++ b/arch/arm/dts/rk3399-rockpro64-u-boot.dtsi
> @@ -22,6 +22,18 @@
>   	};
>   };
>   
> +&u2phy0_otg {
> +	phy-supply = <&vcc5v0_typec>;
> +};
> +
> +&vcc5v0_host {
> +	/delete-property/ regulator-always-on;
> +};
> +
> +&vcc5v0_typec {
> +	/delete-property/ regulator-always-on;
> +};
> +
>   &vdd_center {
>   	regulator-min-microvolt = <950000>;
>   	regulator-max-microvolt = <950000>;


More information about the U-Boot mailing list