2022-01-08 20:26:06

by David Heidelberg

[permalink] [raw]
Subject: [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus

It is not recommended to place regulator nodes inside simple-bus,
so move it out in order to fix the warnings generated by
dtschema/schemas/simple-bus.yaml schema.

Signed-off-by: David Heidelberg <[email protected]>
---
arch/arm/boot/dts/qcom-msm8960.dtsi | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-msm8960.dtsi b/arch/arm/boot/dts/qcom-msm8960.dtsi
index 25ff1f5fffcf..e008b6df108f 100644
--- a/arch/arm/boot/dts/qcom-msm8960.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8960.dtsi
@@ -78,6 +78,15 @@ sleep_clk {
};
};

+ /* Temporary fixed regulator */
+ vsdcc_fixed: vsdcc-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "SDCC Power";
+ regulator-min-microvolt = <2700000>;
+ regulator-max-microvolt = <2700000>;
+ regulator-always-on;
+ };
+
soc: soc {
#address-cells = <1>;
#size-cells = <1>;
@@ -252,15 +261,6 @@ rng@1a500000 {
clock-names = "core";
};

- /* Temporary fixed regulator */
- vsdcc_fixed: vsdcc-regulator {
- compatible = "regulator-fixed";
- regulator-name = "SDCC Power";
- regulator-min-microvolt = <2700000>;
- regulator-max-microvolt = <2700000>;
- regulator-always-on;
- };
-
amba {
compatible = "simple-bus";
#address-cells = <1>;
--
2.34.1



2022-02-02 10:43:00

by Bjorn Andersson

[permalink] [raw]
Subject: Re: (subset) [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus

On Sat, 8 Jan 2022 21:25:58 +0100, David Heidelberg wrote:
> It is not recommended to place regulator nodes inside simple-bus,
> so move it out in order to fix the warnings generated by
> dtschema/schemas/simple-bus.yaml schema.
>
>

Applied, thanks!

[1/1] ARM: dts: msm8960: move vsdcc regulator out of simple-bus
commit: 96b2f11780d550e68dc1a5276861eb6eb3378b0c

Best regards,
--
Bjorn Andersson <[email protected]>