[PATCH v3 5/5] arm: dts: qcom-msm8660: align RPM regulators node name with bindings

Neil Armstrong posted 5 patches 3 years, 5 months ago
There is a newer version of this series
[PATCH v3 5/5] arm: dts: qcom-msm8660: align RPM regulators node name with bindings
Posted by Neil Armstrong 3 years, 5 months ago
Node names should be generic and new DT schema expects RPM regulators
node to be just "regulators".

Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
 arch/arm/boot/dts/qcom-msm8660.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-msm8660.dtsi b/arch/arm/boot/dts/qcom-msm8660.dtsi
index ddce7d64ba99..432bea52bfe9 100644
--- a/arch/arm/boot/dts/qcom-msm8660.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8660.dtsi
@@ -471,7 +471,7 @@ rpmcc: clock-controller {
 				clock-names = "pxo";
 			};
 
-			pm8901-regulators {
+			regulators-0 {
 				compatible = "qcom,rpm-pm8901-regulators";
 
 				pm8901_l0: l0 {};
@@ -495,7 +495,7 @@ pm8901-regulators {
 				pm8901_mvs: mvs {};
 			};
 
-			pm8058-regulators {
+			regulators-1 {
 				compatible = "qcom,rpm-pm8058-regulators";
 
 				pm8058_l0: l0 {};

-- 
b4 0.10.1
Re: [PATCH v3 5/5] arm: dts: qcom-msm8660: align RPM regulators node name with bindings
Posted by Krzysztof Kozlowski 3 years, 5 months ago
On 21/10/2022 11:27, Neil Armstrong wrote:
> Node names should be generic and new DT schema expects RPM regulators
> node to be just "regulators".
> 
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>


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

Best regards,
Krzysztof
Re: [PATCH v3 5/5] arm: dts: qcom-msm8660: align RPM regulators node name with bindings
Posted by Linus Walleij 3 years, 5 months ago
On Fri, Oct 21, 2022 at 5:28 PM Neil Armstrong
<neil.armstrong@linaro.org> wrote:

> Node names should be generic and new DT schema expects RPM regulators
> node to be just "regulators".
>
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij