[PATCH 01/11] ARM: dts: qcom-sdx55: align RPMh regulator nodes with bindings

[PATCH 01/11] ARM: dts: qcom-sdx55: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

See:
https://lore.kernel.org/all/20230112203653.23139-1-quic_molvera@quicinc.com/
---
 arch/arm/boot/dts/qcom-sdx55-mtp.dts             | 2 +-
 arch/arm/boot/dts/qcom-sdx55-t55.dts             | 2 +-
 arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-sdx55-mtp.dts b/arch/arm/boot/dts/qcom-sdx55-mtp.dts
index 6f8909731faf..7e97ad5803d8 100644
--- a/arch/arm/boot/dts/qcom-sdx55-mtp.dts
+++ b/arch/arm/boot/dts/qcom-sdx55-mtp.dts
@@ -75,7 +75,7 @@ vreg_s7e_mx_0p752: pmx55-s7e {
 };
 
 &apps_rsc {
-	pmx55-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pmx55-rpmh-regulators";
 		qcom,pmic-id = "e";
 
diff --git a/arch/arm/boot/dts/qcom-sdx55-t55.dts b/arch/arm/boot/dts/qcom-sdx55-t55.dts
index ccf2b8b42b16..d5343bb0daee 100644
--- a/arch/arm/boot/dts/qcom-sdx55-t55.dts
+++ b/arch/arm/boot/dts/qcom-sdx55-t55.dts
@@ -98,7 +98,7 @@ vreg_vddpx_2: vddpx-2 {
 };
 
 &apps_rsc {
-	pmx55-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pmx55-rpmh-regulators";
 		qcom,pmic-id = "e";
 
diff --git a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
index db7b3d5b0bd0..ad74ecc2a196 100644
--- a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
+++ b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
@@ -98,7 +98,7 @@ vreg_vddpx_2: vddpx-2 {
 };
 
 &apps_rsc {
-	pmx55-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pmx55-rpmh-regulators";
 		qcom,pmic-id = "e";
 
-- 
2.34.1

[PATCH 02/11] ARM: dts: qcom-sdx65: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/qcom-sdx65-mtp.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/qcom-sdx65-mtp.dts b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
index 85ea02d8362d..ed98c83c141f 100644
--- a/arch/arm/boot/dts/qcom-sdx65-mtp.dts
+++ b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
@@ -65,7 +65,7 @@ vreg_bob_3p3: pmx65_bob {
 };
 
 &apps_rsc {
-	pmx65-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pmx65-rpmh-regulators";
 		qcom,pmic-id = "b";
 
-- 
2.34.1

Re: [PATCH 02/11] ARM: dts: qcom-sdx65: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm/boot/dts/qcom-sdx65-mtp.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-sdx65-mtp.dts b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> index 85ea02d8362d..ed98c83c141f 100644
> --- a/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> +++ b/arch/arm/boot/dts/qcom-sdx65-mtp.dts
> @@ -65,7 +65,7 @@ vreg_bob_3p3: pmx65_bob {
>  };
>  
>  &apps_rsc {
> -	pmx65-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmx65-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  

[PATCH 03/11] arm64: dts: qcom: sa8295p-adp: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sa8295p-adp.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
index bb4270e8f551..fd253942e5e5 100644
--- a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
+++ b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
@@ -111,7 +111,7 @@ edp3_connector_in: endpoint {
 };
 
 &apps_rsc {
-	pmm8540-a-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -151,7 +151,7 @@ vreg_l11a: ldo11 {
 		};
 	};
 
-	pmm8540-c-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -224,7 +224,7 @@ vreg_l17c: ldo17 {
 		};
 	};
 
-	pmm8540-g-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "g";
 
-- 
2.34.1

Re: [PATCH 03/11] arm64: dts: qcom: sa8295p-adp: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sa8295p-adp.dts | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
> index bb4270e8f551..fd253942e5e5 100644
> --- a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
> +++ b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts
> @@ -111,7 +111,7 @@ edp3_connector_in: endpoint {
>  };
>  
>  &apps_rsc {
> -	pmm8540-a-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -151,7 +151,7 @@ vreg_l11a: ldo11 {
>  		};
>  	};
>  
> -	pmm8540-c-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -224,7 +224,7 @@ vreg_l17c: ldo17 {
>  		};
>  	};
>  
> -	pmm8540-g-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "g";
>  

[PATCH 04/11] arm64: dts: qcom: sc7180: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc7180-idp.dts      | 4 ++--
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7180-idp.dts b/arch/arm64/boot/dts/qcom/sc7180-idp.dts
index 6ea8dd7d3e01..c3bdd3295c02 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-idp.dts
+++ b/arch/arm64/boot/dts/qcom/sc7180-idp.dts
@@ -90,7 +90,7 @@ mdata_mem: mpss-metadata {
 };
 
 &apps_rsc {
-	pm6150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm6150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -212,7 +212,7 @@ vreg_l19a_2p9: ldo19 {
 		};
 	};
 
-	pm6150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm6150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
index dcb179b2a3fb..423630c4d02c 100644
--- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
@@ -438,7 +438,7 @@ flash@0 {
 };
 
 &apps_rsc {
-	pm6150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm6150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -551,7 +551,7 @@ pp2850_l19a: ldo19 {
 		};
 	};
 
-	pm6150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm6150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
-- 
2.34.1

Re: [PATCH 04/11] arm64: dts: qcom: sc7180: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sc7180-idp.dts      | 4 ++--
>  arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-idp.dts b/arch/arm64/boot/dts/qcom/sc7180-idp.dts
> index 6ea8dd7d3e01..c3bdd3295c02 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-idp.dts
> +++ b/arch/arm64/boot/dts/qcom/sc7180-idp.dts
> @@ -90,7 +90,7 @@ mdata_mem: mpss-metadata {
>  };
>  
>  &apps_rsc {
> -	pm6150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm6150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -212,7 +212,7 @@ vreg_l19a_2p9: ldo19 {
>  		};
>  	};
>  
> -	pm6150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm6150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> index dcb179b2a3fb..423630c4d02c 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> @@ -438,7 +438,7 @@ flash@0 {
>  };
>  
>  &apps_rsc {
> -	pm6150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm6150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -551,7 +551,7 @@ pp2850_l19a: ldo19 {
>  		};
>  	};
>  
> -	pm6150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm6150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  

[PATCH 05/11] arm64: dts: qcom: sc8280xp: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc8280xp-crd.dts                  | 6 +++---
 arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
index 20c629172477..2f43b2fbc2a1 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
@@ -142,7 +142,7 @@ linux,cma {
 };
 
 &apps_rsc {
-	pmc8280-1-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -179,7 +179,7 @@ vreg_l6b: ldo6 {
 		};
 	};
 
-	pmc8280c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -208,7 +208,7 @@ vreg_l13c: ldo13 {
 		};
 	};
 
-	pmc8280-2-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "d";
 
diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
index b60185d9bdea..e41b630141fc 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
@@ -287,7 +287,7 @@ map1 {
 };
 
 &apps_rsc {
-	pmc8280-1-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -338,7 +338,7 @@ vreg_l6b: ldo6 {
 		};
 	};
 
-	pmc8280c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 		vdd-bob-supply = <&vreg_vph_pwr>;
@@ -372,7 +372,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pmc8280-2-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "d";
 
-- 
2.34.1

Re: [PATCH 05/11] arm64: dts: qcom: sc8280xp: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sc8280xp-crd.dts                  | 6 +++---
>  arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 6 +++---
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> index 20c629172477..2f43b2fbc2a1 100644
> --- a/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> +++ b/arch/arm64/boot/dts/qcom/sc8280xp-crd.dts
> @@ -142,7 +142,7 @@ linux,cma {
>  };
>  
>  &apps_rsc {
> -	pmc8280-1-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -179,7 +179,7 @@ vreg_l6b: ldo6 {
>  		};
>  	};
>  
> -	pmc8280c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -208,7 +208,7 @@ vreg_l13c: ldo13 {
>  		};
>  	};
>  
> -	pmc8280-2-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "d";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
> index b60185d9bdea..e41b630141fc 100644
> --- a/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
> +++ b/arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts
> @@ -287,7 +287,7 @@ map1 {
>  };
>  
>  &apps_rsc {
> -	pmc8280-1-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -338,7 +338,7 @@ vreg_l6b: ldo6 {
>  		};
>  	};
>  
> -	pmc8280c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  		vdd-bob-supply = <&vreg_vph_pwr>;
> @@ -372,7 +372,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pmc8280-2-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "d";
>  

[PATCH 06/11] arm64: dts: qcom: sdm845: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi                  | 4 ++--
 arch/arm64/boot/dts/qcom/sdm845-db845c.dts                  | 4 ++--
 arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi              | 6 +++---
 arch/arm64/boot/dts/qcom/sdm845-mtp.dts                     | 6 +++---
 arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi         | 6 +++---
 arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts     | 4 ++--
 arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts           | 6 +++---
 arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi       | 6 +++---
 .../arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi | 2 +-
 arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts          | 6 +++---
 arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts        | 2 +-
 arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts            | 2 +-
 12 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
index e19178c1ec44..f2b48241d15c 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
@@ -339,7 +339,7 @@ flash@0 {
 
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -621,7 +621,7 @@ src_pp1800_lvs2: lvs2 {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
index 9c9a587ea7a9..0b7940c130af 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
@@ -270,7 +270,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 		vdd-s1-supply = <&vph_pwr>;
@@ -395,7 +395,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
index 36f291d4d691..f942c5afea9b 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
@@ -166,7 +166,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -419,7 +419,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -433,7 +433,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
index 482f43fe0151..d1440b790fa6 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
@@ -117,7 +117,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -382,7 +382,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -396,7 +396,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
index 548e34632de2..601f6467146d 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
@@ -145,7 +145,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -281,7 +281,7 @@ vreg_l28a_3p0: ldo28 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -295,7 +295,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
index e742c27fe91f..d37a433130b9 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
@@ -73,7 +73,7 @@ memory@a1300000 {
 
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -332,7 +332,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
index 5d0509f61fe8..b54e304abf71 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
@@ -111,7 +111,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -376,7 +376,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -390,7 +390,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
index 85ff0a0789ea..4984c7496c31 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
@@ -99,7 +99,7 @@ ramoops@ffc00000 {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -329,7 +329,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -340,7 +340,7 @@ src_vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
index 0d19506dc54c..8de01a84f8a0 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
@@ -124,7 +124,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
index 40185eda28a8..a08d9c97e414 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
+++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
@@ -144,7 +144,7 @@ vreg_s4a_1p8: vreg-s4a-1p8 {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -344,7 +344,7 @@ vreg_lvs2a_1p8: lvs2 {
 		};
 	};
 
-	pmi8998-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmi8998-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -356,7 +356,7 @@ vreg_bob: bob {
 		};
 	};
 
-	pm8005-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8005-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
index e9e960b1ad9f..25f88215986b 100644
--- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
+++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
@@ -100,7 +100,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
diff --git a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
index eae696f0dbe4..b650fa846d23 100644
--- a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
+++ b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
@@ -130,7 +130,7 @@ &adsp_pas {
 };
 
 &apps_rsc {
-	pm8998-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8998-rpmh-regulators";
 		qcom,pmic-id = "a";
 
-- 
2.34.1

Re: [PATCH 06/11] arm64: dts: qcom: sdm845: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-db845c.dts                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi              | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-mtp.dts                     | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi         | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts     | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts           | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi       | 6 +++---
>  .../arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi | 2 +-
>  arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts          | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts        | 2 +-
>  arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts            | 2 +-
>  12 files changed, 27 insertions(+), 27 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> index e19178c1ec44..f2b48241d15c 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> @@ -339,7 +339,7 @@ flash@0 {
>  
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -621,7 +621,7 @@ src_pp1800_lvs2: lvs2 {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> index 9c9a587ea7a9..0b7940c130af 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> @@ -270,7 +270,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  		vdd-s1-supply = <&vph_pwr>;
> @@ -395,7 +395,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> index 36f291d4d691..f942c5afea9b 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> @@ -166,7 +166,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -419,7 +419,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -433,7 +433,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> index 482f43fe0151..d1440b790fa6 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> @@ -117,7 +117,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -382,7 +382,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -396,7 +396,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> index 548e34632de2..601f6467146d 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> @@ -145,7 +145,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -281,7 +281,7 @@ vreg_l28a_3p0: ldo28 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -295,7 +295,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> index e742c27fe91f..d37a433130b9 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> @@ -73,7 +73,7 @@ memory@a1300000 {
>  
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -332,7 +332,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> index 5d0509f61fe8..b54e304abf71 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> @@ -111,7 +111,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -376,7 +376,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -390,7 +390,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> index 85ff0a0789ea..4984c7496c31 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> @@ -99,7 +99,7 @@ ramoops@ffc00000 {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -329,7 +329,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -340,7 +340,7 @@ src_vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> index 0d19506dc54c..8de01a84f8a0 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> @@ -124,7 +124,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> index 40185eda28a8..a08d9c97e414 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> @@ -144,7 +144,7 @@ vreg_s4a_1p8: vreg-s4a-1p8 {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -344,7 +344,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -356,7 +356,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> index e9e960b1ad9f..25f88215986b 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> @@ -100,7 +100,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> index eae696f0dbe4..b650fa846d23 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> @@ -130,7 +130,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  

Re: [PATCH 06/11] arm64: dts: qcom: sdm845: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-db845c.dts                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi              | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-mtp.dts                     | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi         | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts     | 4 ++--
>  arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts           | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi       | 6 +++---
>  .../arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi | 2 +-
>  arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts          | 6 +++---
>  arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts        | 2 +-
>  arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts            | 2 +-
>  12 files changed, 27 insertions(+), 27 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> index e19178c1ec44..f2b48241d15c 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> @@ -339,7 +339,7 @@ flash@0 {
>  
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -621,7 +621,7 @@ src_pp1800_lvs2: lvs2 {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> index 9c9a587ea7a9..0b7940c130af 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> @@ -270,7 +270,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  		vdd-s1-supply = <&vph_pwr>;
> @@ -395,7 +395,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> index 36f291d4d691..f942c5afea9b 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-lg-common.dtsi
> @@ -166,7 +166,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -419,7 +419,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -433,7 +433,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> index 482f43fe0151..d1440b790fa6 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-mtp.dts
> @@ -117,7 +117,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -382,7 +382,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -396,7 +396,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> index 548e34632de2..601f6467146d 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-oneplus-common.dtsi
> @@ -145,7 +145,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -281,7 +281,7 @@ vreg_l28a_3p0: ldo28 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -295,7 +295,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> index e742c27fe91f..d37a433130b9 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-samsung-starqltechn.dts
> @@ -73,7 +73,7 @@ memory@a1300000 {
>  
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -332,7 +332,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> index 5d0509f61fe8..b54e304abf71 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> @@ -111,7 +111,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -376,7 +376,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -390,7 +390,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> index 85ff0a0789ea..4984c7496c31 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-sony-xperia-tama.dtsi
> @@ -99,7 +99,7 @@ ramoops@ffc00000 {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -329,7 +329,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -340,7 +340,7 @@ src_vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> index 0d19506dc54c..8de01a84f8a0 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
> @@ -124,7 +124,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> index 40185eda28a8..a08d9c97e414 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-polaris.dts
> @@ -144,7 +144,7 @@ vreg_s4a_1p8: vreg-s4a-1p8 {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -344,7 +344,7 @@ vreg_lvs2a_1p8: lvs2 {
>  		};
>  	};
>  
> -	pmi8998-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmi8998-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -356,7 +356,7 @@ vreg_bob: bob {
>  		};
>  	};
>  
> -	pm8005-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8005-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> index e9e960b1ad9f..25f88215986b 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> @@ -100,7 +100,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> index eae696f0dbe4..b650fa846d23 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> @@ -130,7 +130,7 @@ &adsp_pas {
>  };
>  
>  &apps_rsc {
> -	pm8998-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8998-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  

[PATCH 07/11] arm64: dts: qcom: sm7225: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
index b86a1c6f08be..52bb48c02536 100644
--- a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
+++ b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
@@ -115,7 +115,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm6350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm6350-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -244,7 +244,7 @@ vreg_l22a: ldo22 {
 		};
 	};
 
-	pm6150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm6150l-rpmh-regulators";
 		qcom,pmic-id = "e";
 
-- 
2.34.1

Re: [PATCH 07/11] arm64: dts: qcom: sm7225: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
> index b86a1c6f08be..52bb48c02536 100644
> --- a/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
> +++ b/arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dts
> @@ -115,7 +115,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm6350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm6350-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -244,7 +244,7 @@ vreg_l22a: ldo22 {
>  		};
>  	};
>  
> -	pm6150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm6150l-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  

[PATCH 08/11] arm64: dts: qcom: sm8150: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sa8155p-adp.dts                  | 4 ++--
 arch/arm64/boot/dts/qcom/sm8150-hdk.dts                   | 6 +++---
 arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts | 6 +++---
 arch/arm64/boot/dts/qcom/sm8150-mtp.dts                   | 6 +++---
 arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi   | 4 ++--
 5 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
index eafdfbbf40b9..459384ec8f23 100644
--- a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
+++ b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
@@ -72,7 +72,7 @@ queue0 {
 };
 
 &apps_rsc {
-	pmm8155au-1-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pmm8155au-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -201,7 +201,7 @@ vreg_l17a_2p96: ldo17 {
 		};
 	};
 
-	pmm8155au-2-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pmm8155au-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
index 3331ee957d64..8f014a232526 100644
--- a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
@@ -56,7 +56,7 @@ key-vol-up {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -211,7 +211,7 @@ vreg_l17a_3p0: ldo17 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -324,7 +324,7 @@ vreg_l11c_3p3: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
index 5397fba9417b..69024f7c7f10 100644
--- a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
+++ b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
@@ -61,7 +61,7 @@ key-vol-up {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -216,7 +216,7 @@ vreg_l17a_3p0: ldo17 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -329,7 +329,7 @@ vreg_l11c_3p3: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
index 46b5cf9a1192..eff995a07ab7 100644
--- a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
@@ -51,7 +51,7 @@ vreg_s4a_1p8: pm8150-s4 {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -206,7 +206,7 @@ vreg_l17a_3p0: ldo17 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -319,7 +319,7 @@ vreg_l11c_3p3: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
index 64602748c657..ff77cc3c879a 100644
--- a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
@@ -173,7 +173,7 @@ &adsp_mem {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -306,7 +306,7 @@ vreg_l18a_0p8: ldo18 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
-- 
2.34.1

Re: [PATCH 08/11] arm64: dts: qcom: sm8150: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sa8155p-adp.dts                  | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8150-hdk.dts                   | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-mtp.dts                   | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi   | 4 ++--
>  5 files changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> index eafdfbbf40b9..459384ec8f23 100644
> --- a/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> +++ b/arch/arm64/boot/dts/qcom/sa8155p-adp.dts
> @@ -72,7 +72,7 @@ queue0 {
>  };
>  
>  &apps_rsc {
> -	pmm8155au-1-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmm8155au-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -201,7 +201,7 @@ vreg_l17a_2p96: ldo17 {
>  		};
>  	};
>  
> -	pmm8155au-2-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pmm8155au-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> index 3331ee957d64..8f014a232526 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-hdk.dts
> @@ -56,7 +56,7 @@ key-vol-up {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -211,7 +211,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -324,7 +324,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> index 5397fba9417b..69024f7c7f10 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-microsoft-surface-duo.dts
> @@ -61,7 +61,7 @@ key-vol-up {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -216,7 +216,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -329,7 +329,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> index 46b5cf9a1192..eff995a07ab7 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8150-mtp.dts
> @@ -51,7 +51,7 @@ vreg_s4a_1p8: pm8150-s4 {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -206,7 +206,7 @@ vreg_l17a_3p0: ldo17 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -319,7 +319,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> index 64602748c657..ff77cc3c879a 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano.dtsi
> @@ -173,7 +173,7 @@ &adsp_mem {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -306,7 +306,7 @@ vreg_l18a_0p8: ldo18 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  

[PATCH 09/11] arm64: dts: qcom: sm8250: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/qrb5165-rb5.dts             | 6 +++---
 arch/arm64/boot/dts/qcom/sm8250-hdk.dts              | 6 +++---
 arch/arm64/boot/dts/qcom/sm8250-mtp.dts              | 6 +++---
 arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi | 6 +++---
 4 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
index 8c64cb060e21..e96aa5ccb679 100644
--- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
+++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
@@ -238,7 +238,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8009-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8009-1-rpmh-regulators";
 		qcom,pmic-id = "f";
 
@@ -284,7 +284,7 @@ vreg_l7f_1p8: ldo7 {
 		};
 	};
 
-	pm8150-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -417,7 +417,7 @@ vreg_s6a_0p95: smps6 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8250-hdk.dts b/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
index 632e98193d27..0aee7f8658b4 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
@@ -65,7 +65,7 @@ key-vol-up {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -199,7 +199,7 @@ vreg_l18a_0p92: ldo18 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -310,7 +310,7 @@ vreg_l11c_3p3: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
index 0991b34a8e49..0de8f2320b01 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
@@ -187,7 +187,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -321,7 +321,7 @@ vreg_l18a_0p9: ldo18 {
 		};
 	};
 
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -432,7 +432,7 @@ vreg_l11c_3p3: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
index e76d0ef5aec9..b9c982a059df 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
@@ -123,7 +123,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8150-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8150-rpmh-regulators";
 		qcom,pmic-id = "a";
 
@@ -247,7 +247,7 @@ vreg_l18a_0p9: ldo18 {
 	 * ab: 4600000-6100000
 	 * ibb: 800000-5400000
 	 */
-	pm8150l-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8150l-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -360,7 +360,7 @@ vreg_l11c_3p0: ldo11 {
 		};
 	};
 
-	pm8009-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8009-rpmh-regulators";
 		qcom,pmic-id = "f";
 
-- 
2.34.1

Re: [PATCH 09/11] arm64: dts: qcom: sm8250: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/qrb5165-rb5.dts             | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8250-hdk.dts              | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8250-mtp.dts              | 6 +++---
>  arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi | 6 +++---
>  4 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> index 8c64cb060e21..e96aa5ccb679 100644
> --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> @@ -238,7 +238,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8009-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8009-1-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> @@ -284,7 +284,7 @@ vreg_l7f_1p8: ldo7 {
>  		};
>  	};
>  
> -	pm8150-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -417,7 +417,7 @@ vreg_s6a_0p95: smps6 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-hdk.dts b/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
> index 632e98193d27..0aee7f8658b4 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8250-hdk.dts
> @@ -65,7 +65,7 @@ key-vol-up {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -199,7 +199,7 @@ vreg_l18a_0p92: ldo18 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -310,7 +310,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
> index 0991b34a8e49..0de8f2320b01 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8250-mtp.dts
> @@ -187,7 +187,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -321,7 +321,7 @@ vreg_l18a_0p9: ldo18 {
>  		};
>  	};
>  
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -432,7 +432,7 @@ vreg_l11c_3p3: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> index e76d0ef5aec9..b9c982a059df 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo.dtsi
> @@ -123,7 +123,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8150-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8150-rpmh-regulators";
>  		qcom,pmic-id = "a";
>  
> @@ -247,7 +247,7 @@ vreg_l18a_0p9: ldo18 {
>  	 * ab: 4600000-6100000
>  	 * ibb: 800000-5400000
>  	 */
> -	pm8150l-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8150l-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -360,7 +360,7 @@ vreg_l11c_3p0: ldo11 {
>  		};
>  	};
>  
> -	pm8009-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8009-rpmh-regulators";
>  		qcom,pmic-id = "f";
>  

[PATCH 10/11] arm64: dts: qcom: sm8350: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm8350-hdk.dts                    | 4 ++--
 arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts | 4 ++--
 arch/arm64/boot/dts/qcom/sm8350-mtp.dts                    | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
index 5a4c4ea4d122..0dab73c0c537 100644
--- a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
@@ -73,7 +73,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -178,7 +178,7 @@ vreg_l9b_1p2: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts b/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
index 00f16cde6c4a..b536ae36ae6d 100644
--- a/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
+++ b/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
@@ -44,7 +44,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -135,7 +135,7 @@ vreg_l9b_1p2: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8350-mtp.dts b/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
index f70e0de0509c..d21d2aacf201 100644
--- a/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
@@ -43,7 +43,7 @@ &adsp {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -134,7 +134,7 @@ vreg_l9b_1p2: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
-- 
2.34.1

Re: [PATCH 10/11] arm64: dts: qcom: sm8350: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sm8350-hdk.dts                    | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts | 4 ++--
>  arch/arm64/boot/dts/qcom/sm8350-mtp.dts                    | 4 ++--
>  3 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
> index 5a4c4ea4d122..0dab73c0c537 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts
> @@ -73,7 +73,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -178,7 +178,7 @@ vreg_l9b_1p2: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts b/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
> index 00f16cde6c4a..b536ae36ae6d 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8350-microsoft-surface-duo2.dts
> @@ -44,7 +44,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -135,7 +135,7 @@ vreg_l9b_1p2: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8350-mtp.dts b/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
> index f70e0de0509c..d21d2aacf201 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8350-mtp.dts
> @@ -43,7 +43,7 @@ &adsp {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -134,7 +134,7 @@ vreg_l9b_1p2: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  

[PATCH 11/11] arm64: dts: qcom: sm8450: align RPMh regulator nodes with bindings

[Krzysztof Kozlowski]

Device node names should be generic and bindings expect certain pattern
for RPMh regulator nodes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/qcom/sm8450-hdk.dts                 | 8 ++++----
 arch/arm64/boot/dts/qcom/sm8450-qrd.dts                 | 8 ++++----
 arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi | 8 ++++----
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
index 5bdc2c1159ae..feef3837e4cd 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
+++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
@@ -99,7 +99,7 @@ vph_pwr: vph-pwr-regulator {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -190,7 +190,7 @@ vreg_l9b_1p2: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -303,7 +303,7 @@ vreg_l13c_3p0: ldo13 {
 		};
 	};
 
-	pm8450-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8450-rpmh-regulators";
 		qcom,pmic-id = "h";
 
@@ -346,7 +346,7 @@ vreg_l3h_0p91: ldo3 {
 
 	};
 
-	pmr735a-rpmh-regulators {
+	regulators-3 {
 		compatible = "qcom,pmr735a-rpmh-regulators";
 		qcom,pmic-id = "e";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8450-qrd.dts b/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
index 134ffdfc2c63..f7592946c783 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
+++ b/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
@@ -39,7 +39,7 @@ vph_pwr: vph-pwr-regulator {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -130,7 +130,7 @@ vreg_l9b_1p2: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -242,7 +242,7 @@ vreg_l13c_3p0: ldo13 {
 		};
 	};
 
-	pm8450-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8450-rpmh-regulators";
 		qcom,pmic-id = "h";
 
@@ -285,7 +285,7 @@ vreg_l3h_0p91: ldo3 {
 
 	};
 
-	pmr735a-rpmh-regulators {
+	regulators-3 {
 		compatible = "qcom,pmr735a-rpmh-regulators";
 		qcom,pmic-id = "e";
 
diff --git a/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi b/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
index 53d0ee2dbfa9..5be52468489b 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
@@ -116,7 +116,7 @@ vph_pwr: vph-pwr-regulator {
 };
 
 &apps_rsc {
-	pm8350-rpmh-regulators {
+	regulators-0 {
 		compatible = "qcom,pm8350-rpmh-regulators";
 		qcom,pmic-id = "b";
 
@@ -212,7 +212,7 @@ pm8350_l9: ldo9 {
 		};
 	};
 
-	pm8350c-rpmh-regulators {
+	regulators-1 {
 		compatible = "qcom,pm8350c-rpmh-regulators";
 		qcom,pmic-id = "c";
 
@@ -348,7 +348,7 @@ pm8350c_l13: ldo13 {
 		};
 	};
 
-	pm8450-rpmh-regulators {
+	regulators-2 {
 		compatible = "qcom,pm8450-rpmh-regulators";
 		qcom,pmic-id = "h";
 
@@ -392,7 +392,7 @@ pm8450_l3: ldo3 {
 		};
 	};
 
-	pmr735a-rpmh-regulators {
+	regulators-3 {
 		compatible = "qcom,pmr735a-rpmh-regulators";
 		qcom,pmic-id = "e";
 
-- 
2.34.1

Re: [PATCH 11/11] arm64: dts: qcom: sm8450: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sm8450-hdk.dts                 | 8 ++++----
>  arch/arm64/boot/dts/qcom/sm8450-qrd.dts                 | 8 ++++----
>  arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi | 8 ++++----
>  3 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> index 5bdc2c1159ae..feef3837e4cd 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8450-hdk.dts
> @@ -99,7 +99,7 @@ vph_pwr: vph-pwr-regulator {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -190,7 +190,7 @@ vreg_l9b_1p2: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -303,7 +303,7 @@ vreg_l13c_3p0: ldo13 {
>  		};
>  	};
>  
> -	pm8450-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8450-rpmh-regulators";
>  		qcom,pmic-id = "h";
>  
> @@ -346,7 +346,7 @@ vreg_l3h_0p91: ldo3 {
>  
>  	};
>  
> -	pmr735a-rpmh-regulators {
> +	regulators-3 {
>  		compatible = "qcom,pmr735a-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8450-qrd.dts b/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
> index 134ffdfc2c63..f7592946c783 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8450-qrd.dts
> @@ -39,7 +39,7 @@ vph_pwr: vph-pwr-regulator {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -130,7 +130,7 @@ vreg_l9b_1p2: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -242,7 +242,7 @@ vreg_l13c_3p0: ldo13 {
>  		};
>  	};
>  
> -	pm8450-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8450-rpmh-regulators";
>  		qcom,pmic-id = "h";
>  
> @@ -285,7 +285,7 @@ vreg_l3h_0p91: ldo3 {
>  
>  	};
>  
> -	pmr735a-rpmh-regulators {
> +	regulators-3 {
>  		compatible = "qcom,pmr735a-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  
> diff --git a/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi b/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
> index 53d0ee2dbfa9..5be52468489b 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara.dtsi
> @@ -116,7 +116,7 @@ vph_pwr: vph-pwr-regulator {
>  };
>  
>  &apps_rsc {
> -	pm8350-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pm8350-rpmh-regulators";
>  		qcom,pmic-id = "b";
>  
> @@ -212,7 +212,7 @@ pm8350_l9: ldo9 {
>  		};
>  	};
>  
> -	pm8350c-rpmh-regulators {
> +	regulators-1 {
>  		compatible = "qcom,pm8350c-rpmh-regulators";
>  		qcom,pmic-id = "c";
>  
> @@ -348,7 +348,7 @@ pm8350c_l13: ldo13 {
>  		};
>  	};
>  
> -	pm8450-rpmh-regulators {
> +	regulators-2 {
>  		compatible = "qcom,pm8450-rpmh-regulators";
>  		qcom,pmic-id = "h";
>  
> @@ -392,7 +392,7 @@ pm8450_l3: ldo3 {
>  		};
>  	};
>  
> -	pmr735a-rpmh-regulators {
> +	regulators-3 {
>  		compatible = "qcom,pmr735a-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  

Re: [PATCH 01/11] ARM: dts: qcom-sdx55: align RPMh regulator nodes with bindings

[Konrad Dybcio]

On 27.01.2023 12:43, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
> 
> See:
> https://lore.kernel.org/all/20230112203653.23139-1-quic_molvera@quicinc.com/
> ---
>  arch/arm/boot/dts/qcom-sdx55-mtp.dts             | 2 +-
>  arch/arm/boot/dts/qcom-sdx55-t55.dts             | 2 +-
>  arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-sdx55-mtp.dts b/arch/arm/boot/dts/qcom-sdx55-mtp.dts
> index 6f8909731faf..7e97ad5803d8 100644
> --- a/arch/arm/boot/dts/qcom-sdx55-mtp.dts
> +++ b/arch/arm/boot/dts/qcom-sdx55-mtp.dts
> @@ -75,7 +75,7 @@ vreg_s7e_mx_0p752: pmx55-s7e {
>  };
>  
>  &apps_rsc {
> -	pmx55-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmx55-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  
> diff --git a/arch/arm/boot/dts/qcom-sdx55-t55.dts b/arch/arm/boot/dts/qcom-sdx55-t55.dts
> index ccf2b8b42b16..d5343bb0daee 100644
> --- a/arch/arm/boot/dts/qcom-sdx55-t55.dts
> +++ b/arch/arm/boot/dts/qcom-sdx55-t55.dts
> @@ -98,7 +98,7 @@ vreg_vddpx_2: vddpx-2 {
>  };
>  
>  &apps_rsc {
> -	pmx55-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmx55-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  
> diff --git a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> index db7b3d5b0bd0..ad74ecc2a196 100644
> --- a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> +++ b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> @@ -98,7 +98,7 @@ vreg_vddpx_2: vddpx-2 {
>  };
>  
>  &apps_rsc {
> -	pmx55-rpmh-regulators {
> +	regulators-0 {
>  		compatible = "qcom,pmx55-rpmh-regulators";
>  		qcom,pmic-id = "e";
>  

Re: (subset) [PATCH 01/11] ARM: dts: qcom-sdx55: align RPMh regulator nodes with bindings

[Bjorn Andersson]

On Fri, 27 Jan 2023 12:43:37 +0100, Krzysztof Kozlowski wrote:
> Device node names should be generic and bindings expect certain pattern
> for RPMh regulator nodes.
> 
> 

Applied, thanks!

[03/11] arm64: dts: qcom: sa8295p-adp: align RPMh regulator nodes with bindings
        commit: f8e749dc7193a4b727659a69f5760a6a94ffcb1e
[04/11] arm64: dts: qcom: sc7180: align RPMh regulator nodes with bindings
        commit: adfb9d68e976c79e07cb64a87f58ea5f4d243a6d
[05/11] arm64: dts: qcom: sc8280xp: align RPMh regulator nodes with bindings
        commit: 4275d2802b4bfbf06780be778538d0872e657dfd
[06/11] arm64: dts: qcom: sdm845: align RPMh regulator nodes with bindings
        commit: 86dd19bbdea2b7d3feb69c0c39f141de30a18ec9
[07/11] arm64: dts: qcom: sm7225: align RPMh regulator nodes with bindings
        commit: 45e9251bdd1118cd97061aec502a5747a1cf1bd6
[08/11] arm64: dts: qcom: sm8150: align RPMh regulator nodes with bindings
        commit: 5a88bb6146b39773534b627d3431bf7d49115abd
[09/11] arm64: dts: qcom: sm8250: align RPMh regulator nodes with bindings
        commit: 537f572b2f681f48e58a26bef46611f03b33681e
[10/11] arm64: dts: qcom: sm8350: align RPMh regulator nodes with bindings
        commit: 0223fe2b3f46e6cfe644563ec7217d2dc7d603b6
[11/11] arm64: dts: qcom: sm8450: align RPMh regulator nodes with bindings
        commit: 6216007a3bcd0deab1a5695c32a141e8a4493ad9

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>