mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus
@ 2022-01-08 20:25 David Heidelberg
  2022-02-01  5:19 ` (subset) " Bjorn Andersson
  0 siblings, 1 reply; 2+ messages in thread
From: David Heidelberg @ 2022-01-08 20:25 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Rob Herring
  Cc: ~okias/devicetree, David Heidelberg, linux-arm-msm, devicetree,
	linux-kernel

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


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: (subset) [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus
  2022-01-08 20:25 [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus David Heidelberg
@ 2022-02-01  5:19 ` Bjorn Andersson
  0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Andersson @ 2022-02-01  5:19 UTC (permalink / raw)
  To: Rob Herring, Andy Gross, David Heidelberg
  Cc: devicetree, ~okias/devicetree, linux-arm-msm, linux-kernel

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>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-02-01  5:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-08 20:25 [PATCH] ARM: dts: msm8960: move vsdcc regulator out of simple-bus David Heidelberg
2022-02-01  5:19 ` (subset) " Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®