All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: "Kukjin Kim" <kgene@kernel.org>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Lars-Peter Clausen" <lars@metafoo.de>,
	"Peter Meerwald-Stadler" <pmeerw@pmeerw.net>,
	"Jonathan Bakker" <xc-racer2@live.ca>,
	"Paweł Chmiel" <pawel.mikolaj.chmiel@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org
Subject: [RFT 20/25] ARM: dts: s5pv210: move fixed regulators under root node in Aquila
Date: Mon,  7 Sep 2020 18:11:36 +0200	[thread overview]
Message-ID: <20200907161141.31034-21-krzk@kernel.org> (raw)
In-Reply-To: <20200907161141.31034-1-krzk@kernel.org>

The fixed regulators are kept under dedicated "regulators" node but this
causes multiple dtschema warnings:

  regulators: $nodename:0: 'regulators' does not match '^([a-z][a-z0-9\\-]+-bus|bus|soc|axi|ahb|apb)(@[0-9a-f]+)?$'
  regulators: #size-cells:0:0: 0 is not one of [1, 2]
  regulators: fixed-regulator@0:reg:0: [0] is too short
  regulators: fixed-regulator@1:reg:0: [1] is too short
  regulators: fixed-regulator@2:reg:0: [2] is too short

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 arch/arm/boot/dts/s5pv210-aquila.dts | 47 +++++++++++-----------------
 1 file changed, 19 insertions(+), 28 deletions(-)

diff --git a/arch/arm/boot/dts/s5pv210-aquila.dts b/arch/arm/boot/dts/s5pv210-aquila.dts
index 25d4ab4a52c5..28e9e81bb2d1 100644
--- a/arch/arm/boot/dts/s5pv210-aquila.dts
+++ b/arch/arm/boot/dts/s5pv210-aquila.dts
@@ -40,36 +40,27 @@
 		clock-frequency = <32768>;
 	};
 
-	regulators {
-		compatible = "simple-bus";
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		vtf_reg: fixed-regulator@0 {
-			compatible = "regulator-fixed";
-			reg = <0>;
-			regulator-name = "V_TF_2.8V";
-			regulator-min-microvolt = <2800000>;
-			regulator-max-microvolt = <2800000>;
-			gpio = <&mp05 4 GPIO_ACTIVE_HIGH>;
-			enable-active-high;
-		};
+	vtf_reg: regulator-0 {
+		compatible = "regulator-fixed";
+		regulator-name = "V_TF_2.8V";
+		regulator-min-microvolt = <2800000>;
+		regulator-max-microvolt = <2800000>;
+		gpio = <&mp05 4 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
 
-		pda_reg: fixed-regulator@1 {
-			compatible = "regulator-fixed";
-			regulator-name = "VCC_1.8V_PDA";
-			regulator-min-microvolt = <1800000>;
-			regulator-max-microvolt = <1800000>;
-			reg = <1>;
-		};
+	pda_reg: regulator-1 {
+		compatible = "regulator-fixed";
+		regulator-name = "VCC_1.8V_PDA";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
 
-		bat_reg: fixed-regulator@2 {
-			compatible = "regulator-fixed";
-			regulator-name = "V_BAT";
-			regulator-min-microvolt = <3700000>;
-			regulator-max-microvolt = <3700000>;
-			reg = <2>;
-		};
+	bat_reg: regulator-2 {
+		compatible = "regulator-fixed";
+		regulator-name = "V_BAT";
+		regulator-min-microvolt = <3700000>;
+		regulator-max-microvolt = <3700000>;
 	};
 
 	i2c_pmic: i2c-pmic {
-- 
2.17.1


WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org>
To: "Kukjin Kim" <kgene@kernel.org>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Jonathan Cameron" <jic23@kernel.org>,
	"Lars-Peter Clausen" <lars@metafoo.de>,
	"Peter Meerwald-Stadler" <pmeerw@pmeerw.net>,
	"Jonathan Bakker" <xc-racer2@live.ca>,
	"Paweł Chmiel" <pawel.mikolaj.chmiel@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org
Subject: [RFT 20/25] ARM: dts: s5pv210: move fixed regulators under root node in Aquila
Date: Mon,  7 Sep 2020 18:11:36 +0200	[thread overview]
Message-ID: <20200907161141.31034-21-krzk@kernel.org> (raw)
In-Reply-To: <20200907161141.31034-1-krzk@kernel.org>

The fixed regulators are kept under dedicated "regulators" node but this
causes multiple dtschema warnings:

  regulators: $nodename:0: 'regulators' does not match '^([a-z][a-z0-9\\-]+-bus|bus|soc|axi|ahb|apb)(@[0-9a-f]+)?$'
  regulators: #size-cells:0:0: 0 is not one of [1, 2]
  regulators: fixed-regulator@0:reg:0: [0] is too short
  regulators: fixed-regulator@1:reg:0: [1] is too short
  regulators: fixed-regulator@2:reg:0: [2] is too short

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 arch/arm/boot/dts/s5pv210-aquila.dts | 47 +++++++++++-----------------
 1 file changed, 19 insertions(+), 28 deletions(-)

diff --git a/arch/arm/boot/dts/s5pv210-aquila.dts b/arch/arm/boot/dts/s5pv210-aquila.dts
index 25d4ab4a52c5..28e9e81bb2d1 100644
--- a/arch/arm/boot/dts/s5pv210-aquila.dts
+++ b/arch/arm/boot/dts/s5pv210-aquila.dts
@@ -40,36 +40,27 @@
 		clock-frequency = <32768>;
 	};
 
-	regulators {
-		compatible = "simple-bus";
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		vtf_reg: fixed-regulator@0 {
-			compatible = "regulator-fixed";
-			reg = <0>;
-			regulator-name = "V_TF_2.8V";
-			regulator-min-microvolt = <2800000>;
-			regulator-max-microvolt = <2800000>;
-			gpio = <&mp05 4 GPIO_ACTIVE_HIGH>;
-			enable-active-high;
-		};
+	vtf_reg: regulator-0 {
+		compatible = "regulator-fixed";
+		regulator-name = "V_TF_2.8V";
+		regulator-min-microvolt = <2800000>;
+		regulator-max-microvolt = <2800000>;
+		gpio = <&mp05 4 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
 
-		pda_reg: fixed-regulator@1 {
-			compatible = "regulator-fixed";
-			regulator-name = "VCC_1.8V_PDA";
-			regulator-min-microvolt = <1800000>;
-			regulator-max-microvolt = <1800000>;
-			reg = <1>;
-		};
+	pda_reg: regulator-1 {
+		compatible = "regulator-fixed";
+		regulator-name = "VCC_1.8V_PDA";
+		regulator-min-microvolt = <1800000>;
+		regulator-max-microvolt = <1800000>;
+	};
 
-		bat_reg: fixed-regulator@2 {
-			compatible = "regulator-fixed";
-			regulator-name = "V_BAT";
-			regulator-min-microvolt = <3700000>;
-			regulator-max-microvolt = <3700000>;
-			reg = <2>;
-		};
+	bat_reg: regulator-2 {
+		compatible = "regulator-fixed";
+		regulator-name = "V_BAT";
+		regulator-min-microvolt = <3700000>;
+		regulator-max-microvolt = <3700000>;
 	};
 
 	i2c_pmic: i2c-pmic {
-- 
2.17.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2020-09-07 16:19 UTC|newest]

Thread overview: 110+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-07 16:11 [PATCH 00/25] ARM: dts: s5pv210: Cleanup - dtschema warnings Krzysztof Kozlowski
2020-09-07 16:11 ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 01/25] dt-bindings: samsung: pmu: document S5Pv210 Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-09 19:36   ` Krzysztof Kozlowski
2020-09-09 19:36     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 02/25] dt-bindings: iio: adc: exynos-adc: require second interrupt with touch screen Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:38   ` Jonathan Cameron
2020-09-07 16:38     ` Jonathan Cameron
2020-09-07 17:11     ` Krzysztof Kozlowski
2020-09-07 17:11       ` Krzysztof Kozlowski
2020-09-08 20:24   ` Rob Herring
2020-09-08 20:24     ` Rob Herring
2020-09-08 20:25   ` Rob Herring
2020-09-08 20:25     ` Rob Herring
2020-09-10 11:55     ` Krzysztof Kozlowski
2020-09-10 11:55       ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 03/25] dt-bindings: iio: adc: exynos-adc: do not require syscon on S5Pv210 Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:39   ` Jonathan Cameron
2020-09-07 16:39     ` Jonathan Cameron
2020-09-07 21:49   ` Jonathan Bakker
2020-09-07 21:49     ` Jonathan Bakker
2020-09-08  6:48     ` Krzysztof Kozlowski
2020-09-08  6:48       ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 04/25] ARM: dts: s5pv210: fix pinctrl property of "vibrator-en" regulator in Aries Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-09 19:39   ` Krzysztof Kozlowski
2020-09-09 19:39     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 05/25] ARM: dts: s5pv210: remove DMA controller bus node name to fix dtschema warnings Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 21:56   ` Jonathan Bakker
2020-09-07 21:56     ` Jonathan Bakker
2020-09-07 16:11 ` [RFT 06/25] ARM: dts: s5pv210: move fixed clocks under root node Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 21:59   ` Jonathan Bakker
2020-09-07 21:59     ` Jonathan Bakker
2020-09-07 16:11 ` [RFT 07/25] ARM: dts: s5pv210: move PMU node out of clock controller Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 22:02   ` Jonathan Bakker
2020-09-07 22:02     ` Jonathan Bakker
2020-09-07 16:11 ` [RFT 08/25] ARM: dts: s5pv210: remove dedicated 'audio-subsystem' node Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 22:41   ` Jonathan Bakker
2020-09-07 22:41     ` Jonathan Bakker
2020-09-07 16:11 ` [RFT 09/25] ARM: dts: s5pv210: fix number of I2S DAI cells Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 23:55   ` Jonathan Bakker
2020-09-07 23:55     ` Jonathan Bakker
2020-09-08  6:53     ` Krzysztof Kozlowski
2020-09-08  6:53       ` Krzysztof Kozlowski
2020-09-08  8:38       ` Sylwester Nawrocki
2020-09-08  8:38         ` Sylwester Nawrocki
2020-09-07 16:11 ` [PATCH 10/25] ARM: dts: s5pv210: add RTC 32 KHz clock in Aquilla Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-09 19:43   ` Krzysztof Kozlowski
2020-09-09 19:43     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 11/25] ARM: dts: s5pv210: add RTC 32 KHz clock in Aries family Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 23:57   ` Jonathan Bakker
2020-09-07 23:57     ` Jonathan Bakker
2020-09-08  6:54     ` Krzysztof Kozlowski
2020-09-08  6:54       ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 12/25] ARM: dts: s5pv210: add RTC 32 KHz clock in Goni Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 13/25] ARM: dts: s5pv210: add RTC 32 KHz clock in SMDKC110 Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 14/25] ARM: dts: s5pv210: add RTC 32 KHz clock in SMDKV210 Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 15/25] ARM: dts: s5pv210: add RTC 32 KHz clock in Torbreck Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 16/25] ARM: dts: s5pv210: use defines for GPIO flags in Aquila Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 17/25] ARM: dts: s5pv210: use defines for GPIO flags in Goni Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 18/25] ARM: dts: s5pv210: use defines for IRQ flags in SMDKV210 Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` [PATCH 19/25] ARM: dts: s5pv210: use defines for IRQ flags in Goni Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 16:11 ` Krzysztof Kozlowski [this message]
2020-09-07 16:11   ` [RFT 20/25] ARM: dts: s5pv210: move fixed regulators under root node in Aquila Krzysztof Kozlowski
2020-09-16 17:16   ` Krzysztof Kozlowski
2020-09-16 17:16     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 21/25] ARM: dts: s5pv210: move fixed regulators under root node in Goni Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-16 17:16   ` Krzysztof Kozlowski
2020-09-16 17:16     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 22/25] ARM: dts: s5pv210: replace deprecated "gpios" i2c-gpio property in Aquila Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-16 17:17   ` Krzysztof Kozlowski
2020-09-16 17:17     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 23/25] ARM: dts: s5pv210: replace deprecated "gpios" i2c-gpio property in Goni Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-16 17:17   ` Krzysztof Kozlowski
2020-09-16 17:17     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 24/25] ARM: dts: s5pv210: align SPI GPIO node name with dtschema in Aries Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-07 23:34   ` Jonathan Bakker
2020-09-07 23:34     ` Jonathan Bakker
2020-09-09 19:44   ` Krzysztof Kozlowski
2020-09-09 19:44     ` Krzysztof Kozlowski
2020-09-07 16:11 ` [RFT 25/25] ARM: dts: s5pv210: align DMA channels with dtschema Krzysztof Kozlowski
2020-09-07 16:11   ` Krzysztof Kozlowski
2020-09-08  0:17   ` Jonathan Bakker
2020-09-08  0:17     ` Jonathan Bakker
     [not found]   ` <25178674-e4af-ba35-b7f0-42091208e0e8@live.ca>
2020-09-08  0:28     ` Jonathan Bakker
2020-09-08  0:28       ` Jonathan Bakker
2020-09-08  6:57       ` Krzysztof Kozlowski
2020-09-08  6:57         ` Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200907161141.31034-21-krzk@kernel.org \
    --to=krzk@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jic23@kernel.org \
    --cc=kgene@kernel.org \
    --cc=lars@metafoo.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=pawel.mikolaj.chmiel@gmail.com \
    --cc=pmeerw@pmeerw.net \
    --cc=robh+dt@kernel.org \
    --cc=xc-racer2@live.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.