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

David Heidelberg posted 1 patch 4 years, 5 months ago
arch/arm/boot/dts/qcom-msm8960.dtsi | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
[PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus
Posted by David Heidelberg 4 years, 5 months ago
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 <david@ixit.cz>
---
 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

Re: (subset) [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus
Posted by Bjorn Andersson 4 years, 4 months ago
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 <bjorn.andersson@linaro.org>