Re: [PATCH 2/2] arm/dts: Cleanup regulator naming and remove @0,1..

[Tony Lindgren]

* Rajendra Nayak rna...@ti.com [120827 23:22]:
 Hi Tony,
 
 * Rajendra Nayakrna...@ti.com  [120730 06:17]:
 regulators do not have a 'reg' property, hence the regulator@0,
 regulator@1 do not make sense. get rid of it.
 
 Looks like this needs to be refreshed to apply. Care to
 refresh against current devel-dt branch in case other
 places need the same change?
 
 Looks like devel-dt is missing the PATCH 1/2 from this series,
 which you have already pushed in the -rc for merge.
 With that applied, this ones applies cleanly too.

Oh OK thanks :) I'll apply it on top of the fix and merge
into devel-dt.

Tony
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] arm/dts: Cleanup regulator naming and remove @0,1..

[Rajendra Nayak]

Hi Tony,


* Rajendra Nayakrna...@ti.com  [120730 06:17]:

regulators do not have a 'reg' property, hence the regulator@0,
regulator@1 do not make sense. get rid of it.


Looks like this needs to be refreshed to apply. Care to
refresh against current devel-dt branch in case other
places need the same change?


Looks like devel-dt is missing the PATCH 1/2 from this series,
which you have already pushed in the -rc for merge.
With that applied, this ones applies cleanly too.

regards,
Rajendra



Regards,

Tony


--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] arm/dts: Cleanup regulator naming and remove @0,1..

[Tony Lindgren]

* Rajendra Nayak rna...@ti.com [120730 06:17]:
 regulators do not have a 'reg' property, hence the regulator@0,
 regulator@1 do not make sense. get rid of it.

Looks like this needs to be refreshed to apply. Care to
refresh against current devel-dt branch in case other
places need the same change?

Regards,

Tony
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] arm/dts: Cleanup regulator naming and remove @0,1..

[Rajendra Nayak]

regulators do not have a 'reg' property, hence the regulator@0,
regulator@1 do not make sense. get rid of it.

Reported-by: David Brown dav...@codeaurora.org
Signed-off-by: Rajendra Nayak rna...@ti.com
---
 arch/arm/boot/dts/omap3-beagle.dts |2 +-
 arch/arm/boot/dts/omap4-sdp.dts|4 ++--
 arch/arm/boot/dts/twl4030.dtsi |6 +++---
 arch/arm/boot/dts/twl6030.dtsi |   26 +-
 4 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/arch/arm/boot/dts/omap3-beagle.dts 
b/arch/arm/boot/dts/omap3-beagle.dts
index cdcb98c..e60cba0 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -27,7 +27,7 @@
interrupts = 7; /* SYS_NIRQ cascaded to intc */
interrupt-parent = intc;
 
-   vsim: regulator@10 {
+   vsim: regulator-vsim {
compatible = ti,twl4030-vsim;
regulator-min-microvolt = 180;
regulator-max-microvolt = 300;
diff --git a/arch/arm/boot/dts/omap4-sdp.dts b/arch/arm/boot/dts/omap4-sdp.dts
index 72216e9..dbcdc4a 100644
--- a/arch/arm/boot/dts/omap4-sdp.dts
+++ b/arch/arm/boot/dts/omap4-sdp.dts
@@ -18,7 +18,7 @@
reg = 0x8000 0x4000; /* 1 GB */
};
 
-   vdd_eth: fixedregulator@0 {
+   vdd_eth: fixedregulator-vdd-eth {
compatible = regulator-fixed;
regulator-name = VDD_ETH;
regulator-min-microvolt = 330;
@@ -28,7 +28,7 @@
regulator-boot-on;
};
 
-   vbat: fixedregulator@2 {
+   vbat: fixedregulator-vbat {
compatible = regulator-fixed;
regulator-name = VBAT;
regulator-min-microvolt = 375;
diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
index 22f4d13..ff00017 100644
--- a/arch/arm/boot/dts/twl4030.dtsi
+++ b/arch/arm/boot/dts/twl4030.dtsi
@@ -19,19 +19,19 @@
interrupts = 11;
};
 
-   vdac: regulator@0 {
+   vdac: regulator-vdac {
compatible = ti,twl4030-vdac;
regulator-min-microvolt = 180;
regulator-max-microvolt = 180;
};
 
-   vpll2: regulator@1 {
+   vpll2: regulator-vpll2 {
compatible = ti,twl4030-vpll2;
regulator-min-microvolt = 180;
regulator-max-microvolt = 180;
};
 
-   vmmc1: regulator@2 {
+   vmmc1: regulator-vmmc1 {
compatible = ti,twl4030-vmmc1;
regulator-min-microvolt = 185;
regulator-max-microvolt = 315;
diff --git a/arch/arm/boot/dts/twl6030.dtsi b/arch/arm/boot/dts/twl6030.dtsi
index d351b27..123e2c4 100644
--- a/arch/arm/boot/dts/twl6030.dtsi
+++ b/arch/arm/boot/dts/twl6030.dtsi
@@ -20,70 +20,70 @@
interrupts = 11;
};
 
-   vaux1: regulator@0 {
+   vaux1: regulator-vaux1 {
compatible = ti,twl6030-vaux1;
regulator-min-microvolt = 100;
regulator-max-microvolt = 300;
};
 
-   vaux2: regulator@1 {
+   vaux2: regulator-vaux2 {
compatible = ti,twl6030-vaux2;
regulator-min-microvolt = 120;
regulator-max-microvolt = 280;
};
 
-   vaux3: regulator@2 {
+   vaux3: regulator-vaux3 {
compatible = ti,twl6030-vaux3;
regulator-min-microvolt = 100;
regulator-max-microvolt = 300;
};
 
-   vmmc: regulator@3 {
+   vmmc: regulator-vmmc {
compatible = ti,twl6030-vmmc;
regulator-min-microvolt = 120;
regulator-max-microvolt = 300;
};
 
-   vpp: regulator@4 {
+   vpp: regulator-vpp {
compatible = ti,twl6030-vpp;
regulator-min-microvolt = 180;
regulator-max-microvolt = 250;
};
 
-   vusim: regulator@5 {
+   vusim: regulator-vusim {
compatible = ti,twl6030-vusim;
regulator-min-microvolt = 120;
regulator-max-microvolt = 290;
};
 
-   vdac: regulator@6 {
+   vdac: regulator-vdac {
compatible = ti,twl6030-vdac;
};
 
-   vana: regulator@7 {
+   vana: regulator-vana {
compatible = ti,twl6030-vana;
};
 
-   vcxio: regulator@8 {
+   vcxio: regulator-vcxio {
compatible = ti,twl6030-vcxio;
regulator-always-on;
};
 
-   vusb: regulator@9 {
+   vusb: regulator-vusb {
compatible = ti,twl6030-vusb;
};
 
-   v1v8: regulator@10 {
+   v1v8: regulator-v1v8 {
compatible = ti,twl6030-v1v8;
regulator-always-on;
};
 
-   v2v1: regulator@11 {
+   v2v1: