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

[Kever Yang]

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...@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>;