devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: mt8135-evbp1: remove regulator-compatible usage
@ 2015-09-25 13:46 Javier Martinez Canillas
  2015-09-27 13:39 ` Matthias Brugger
  0 siblings, 1 reply; 2+ messages in thread
From: Javier Martinez Canillas @ 2015-09-25 13:46 UTC (permalink / raw)
  To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
  Cc: Javier Martinez Canillas, Russell King,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Kumar Gala, Ian Campbell,
	Rob Herring, linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Pawel Moll,
	Mark Rutland, Matthias Brugger

The regulator-compatible property from the regulator DT binding was
deprecated and the correct approach is to use the node's name.

This patch has no functional changes since the values of the node's
name and the regulator-compatible match for all the regulators.

Signed-off-by: Javier Martinez Canillas <javier-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>

---

 arch/arm/boot/dts/mt8135-evbp1.dts | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/arch/arm/boot/dts/mt8135-evbp1.dts b/arch/arm/boot/dts/mt8135-evbp1.dts
index 357a91fc2d1d..460db6d05952 100644
--- a/arch/arm/boot/dts/mt8135-evbp1.dts
+++ b/arch/arm/boot/dts/mt8135-evbp1.dts
@@ -32,7 +32,6 @@
 			compatible = "mediatek,mt6397-regulator";
 
 			mt6397_vpca15_reg: buck_vpca15 {
-				regulator-compatible = "buck_vpca15";
 				regulator-name = "vpca15";
 				regulator-min-microvolt = < 850000>;
 				regulator-max-microvolt = <1350000>;
@@ -41,7 +40,6 @@
 			};
 
 			mt6397_vpca7_reg: buck_vpca7 {
-				regulator-compatible = "buck_vpca7";
 				regulator-name = "vpca7";
 				regulator-min-microvolt = < 850000>;
 				regulator-max-microvolt = <1350000>;
@@ -50,7 +48,6 @@
 			};
 
 			mt6397_vsramca15_reg: buck_vsramca15 {
-				regulator-compatible = "buck_vsramca15";
 				regulator-name = "vsramca15";
 				regulator-min-microvolt = < 850000>;
 				regulator-max-microvolt = <1350000>;
@@ -59,7 +56,6 @@
 			};
 
 			mt6397_vsramca7_reg: buck_vsramca7 {
-				regulator-compatible = "buck_vsramca7";
 				regulator-name = "vsramca7";
 				regulator-min-microvolt = < 850000>;
 				regulator-max-microvolt = <1350000>;
@@ -68,7 +64,6 @@
 			};
 
 			mt6397_vcore_reg: buck_vcore {
-				regulator-compatible = "buck_vcore";
 				regulator-name = "vcore";
 				regulator-min-microvolt = < 850000>;
 				regulator-max-microvolt = <1350000>;
@@ -77,7 +72,6 @@
 			};
 
 			mt6397_vgpu_reg: buck_vgpu {
-				regulator-compatible = "buck_vgpu";
 				regulator-name = "vgpu";
 				regulator-min-microvolt = < 700000>;
 				regulator-max-microvolt = <1350000>;
@@ -86,7 +80,6 @@
 			};
 
 			mt6397_vdrm_reg: buck_vdrm {
-				regulator-compatible = "buck_vdrm";
 				regulator-name = "vdrm";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <1400000>;
@@ -95,7 +88,6 @@
 			};
 
 			mt6397_vio18_reg: buck_vio18 {
-				regulator-compatible = "buck_vio18";
 				regulator-name = "vio18";
 				regulator-min-microvolt = <1620000>;
 				regulator-max-microvolt = <1980000>;
@@ -104,19 +96,16 @@
 			};
 
 			mt6397_vtcxo_reg: ldo_vtcxo {
-				regulator-compatible = "ldo_vtcxo";
 				regulator-name = "vtcxo";
 				regulator-always-on;
 			};
 
 			mt6397_va28_reg: ldo_va28 {
-				regulator-compatible = "ldo_va28";
 				regulator-name = "va28";
 				regulator-always-on;
 			};
 
 			mt6397_vcama_reg: ldo_vcama {
-				regulator-compatible = "ldo_vcama";
 				regulator-name = "vcama";
 				regulator-min-microvolt = <1500000>;
 				regulator-max-microvolt = <2800000>;
@@ -124,18 +113,15 @@
 			};
 
 			mt6397_vio28_reg: ldo_vio28 {
-				regulator-compatible = "ldo_vio28";
 				regulator-name = "vio28";
 				regulator-always-on;
 			};
 
 			mt6397_vusb_reg: ldo_vusb {
-				regulator-compatible = "ldo_vusb";
 				regulator-name = "vusb";
 			};
 
 			mt6397_vmc_reg: ldo_vmc {
-				regulator-compatible = "ldo_vmc";
 				regulator-name = "vmc";
 				regulator-min-microvolt = <1800000>;
 				regulator-max-microvolt = <3300000>;
@@ -143,7 +129,6 @@
 			};
 
 			mt6397_vmch_reg: ldo_vmch {
-				regulator-compatible = "ldo_vmch";
 				regulator-name = "vmch";
 				regulator-min-microvolt = <3000000>;
 				regulator-max-microvolt = <3300000>;
@@ -151,7 +136,6 @@
 			};
 
 			mt6397_vemc_3v3_reg: ldo_vemc3v3 {
-				regulator-compatible = "ldo_vemc3v3";
 				regulator-name = "vemc_3v3";
 				regulator-min-microvolt = <3000000>;
 				regulator-max-microvolt = <3300000>;
@@ -159,7 +143,6 @@
 			};
 
 			mt6397_vgp1_reg: ldo_vgp1 {
-				regulator-compatible = "ldo_vgp1";
 				regulator-name = "vcamd";
 				regulator-min-microvolt = <1220000>;
 				regulator-max-microvolt = <3300000>;
@@ -167,7 +150,6 @@
 			};
 
 			mt6397_vgp2_reg: ldo_vgp2 {
-				regulator-compatible = "ldo_vgp2";
 				regulator-name = "vcamio";
 				regulator-min-microvolt = <1000000>;
 				regulator-max-microvolt = <3300000>;
@@ -175,7 +157,6 @@
 			};
 
 			mt6397_vgp3_reg: ldo_vgp3 {
-				regulator-compatible = "ldo_vgp3";
 				regulator-name = "vcamaf";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <3300000>;
@@ -183,7 +164,6 @@
 			};
 
 			mt6397_vgp4_reg: ldo_vgp4 {
-				regulator-compatible = "ldo_vgp4";
 				regulator-name = "vgp4";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <3300000>;
@@ -191,7 +171,6 @@
 			};
 
 			mt6397_vgp5_reg: ldo_vgp5 {
-				regulator-compatible = "ldo_vgp5";
 				regulator-name = "vgp5";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <3000000>;
@@ -199,7 +178,6 @@
 			};
 
 			mt6397_vgp6_reg: ldo_vgp6 {
-				regulator-compatible = "ldo_vgp6";
 				regulator-name = "vgp6";
 				regulator-min-microvolt = <1200000>;
 				regulator-max-microvolt = <3300000>;
@@ -207,7 +185,6 @@
 			};
 
 			mt6397_vibr_reg: ldo_vibr {
-				regulator-compatible = "ldo_vibr";
 				regulator-name = "vibr";
 				regulator-min-microvolt = <1300000>;
 				regulator-max-microvolt = <3300000>;
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] ARM: dts: mt8135-evbp1: remove regulator-compatible usage
  2015-09-25 13:46 [PATCH] ARM: dts: mt8135-evbp1: remove regulator-compatible usage Javier Martinez Canillas
@ 2015-09-27 13:39 ` Matthias Brugger
  0 siblings, 0 replies; 2+ messages in thread
From: Matthias Brugger @ 2015-09-27 13:39 UTC (permalink / raw)
  To: Javier Martinez Canillas, linux-kernel
  Cc: Russell King, devicetree, Kumar Gala, Ian Campbell, Rob Herring,
	linux-mediatek, linux-arm-kernel, Pawel Moll, Mark Rutland



On 25/09/15 15:46, Javier Martinez Canillas wrote:
> The regulator-compatible property from the regulator DT binding was
> deprecated and the correct approach is to use the node's name.
>
> This patch has no functional changes since the values of the node's
> name and the regulator-compatible match for all the regulators.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Applied, thanks.

>
> ---
>
>   arch/arm/boot/dts/mt8135-evbp1.dts | 23 -----------------------
>   1 file changed, 23 deletions(-)
>
> diff --git a/arch/arm/boot/dts/mt8135-evbp1.dts b/arch/arm/boot/dts/mt8135-evbp1.dts
> index 357a91fc2d1d..460db6d05952 100644
> --- a/arch/arm/boot/dts/mt8135-evbp1.dts
> +++ b/arch/arm/boot/dts/mt8135-evbp1.dts
> @@ -32,7 +32,6 @@
>   			compatible = "mediatek,mt6397-regulator";
>
>   			mt6397_vpca15_reg: buck_vpca15 {
> -				regulator-compatible = "buck_vpca15";
>   				regulator-name = "vpca15";
>   				regulator-min-microvolt = < 850000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -41,7 +40,6 @@
>   			};
>
>   			mt6397_vpca7_reg: buck_vpca7 {
> -				regulator-compatible = "buck_vpca7";
>   				regulator-name = "vpca7";
>   				regulator-min-microvolt = < 850000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -50,7 +48,6 @@
>   			};
>
>   			mt6397_vsramca15_reg: buck_vsramca15 {
> -				regulator-compatible = "buck_vsramca15";
>   				regulator-name = "vsramca15";
>   				regulator-min-microvolt = < 850000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -59,7 +56,6 @@
>   			};
>
>   			mt6397_vsramca7_reg: buck_vsramca7 {
> -				regulator-compatible = "buck_vsramca7";
>   				regulator-name = "vsramca7";
>   				regulator-min-microvolt = < 850000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -68,7 +64,6 @@
>   			};
>
>   			mt6397_vcore_reg: buck_vcore {
> -				regulator-compatible = "buck_vcore";
>   				regulator-name = "vcore";
>   				regulator-min-microvolt = < 850000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -77,7 +72,6 @@
>   			};
>
>   			mt6397_vgpu_reg: buck_vgpu {
> -				regulator-compatible = "buck_vgpu";
>   				regulator-name = "vgpu";
>   				regulator-min-microvolt = < 700000>;
>   				regulator-max-microvolt = <1350000>;
> @@ -86,7 +80,6 @@
>   			};
>
>   			mt6397_vdrm_reg: buck_vdrm {
> -				regulator-compatible = "buck_vdrm";
>   				regulator-name = "vdrm";
>   				regulator-min-microvolt = <1200000>;
>   				regulator-max-microvolt = <1400000>;
> @@ -95,7 +88,6 @@
>   			};
>
>   			mt6397_vio18_reg: buck_vio18 {
> -				regulator-compatible = "buck_vio18";
>   				regulator-name = "vio18";
>   				regulator-min-microvolt = <1620000>;
>   				regulator-max-microvolt = <1980000>;
> @@ -104,19 +96,16 @@
>   			};
>
>   			mt6397_vtcxo_reg: ldo_vtcxo {
> -				regulator-compatible = "ldo_vtcxo";
>   				regulator-name = "vtcxo";
>   				regulator-always-on;
>   			};
>
>   			mt6397_va28_reg: ldo_va28 {
> -				regulator-compatible = "ldo_va28";
>   				regulator-name = "va28";
>   				regulator-always-on;
>   			};
>
>   			mt6397_vcama_reg: ldo_vcama {
> -				regulator-compatible = "ldo_vcama";
>   				regulator-name = "vcama";
>   				regulator-min-microvolt = <1500000>;
>   				regulator-max-microvolt = <2800000>;
> @@ -124,18 +113,15 @@
>   			};
>
>   			mt6397_vio28_reg: ldo_vio28 {
> -				regulator-compatible = "ldo_vio28";
>   				regulator-name = "vio28";
>   				regulator-always-on;
>   			};
>
>   			mt6397_vusb_reg: ldo_vusb {
> -				regulator-compatible = "ldo_vusb";
>   				regulator-name = "vusb";
>   			};
>
>   			mt6397_vmc_reg: ldo_vmc {
> -				regulator-compatible = "ldo_vmc";
>   				regulator-name = "vmc";
>   				regulator-min-microvolt = <1800000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -143,7 +129,6 @@
>   			};
>
>   			mt6397_vmch_reg: ldo_vmch {
> -				regulator-compatible = "ldo_vmch";
>   				regulator-name = "vmch";
>   				regulator-min-microvolt = <3000000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -151,7 +136,6 @@
>   			};
>
>   			mt6397_vemc_3v3_reg: ldo_vemc3v3 {
> -				regulator-compatible = "ldo_vemc3v3";
>   				regulator-name = "vemc_3v3";
>   				regulator-min-microvolt = <3000000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -159,7 +143,6 @@
>   			};
>
>   			mt6397_vgp1_reg: ldo_vgp1 {
> -				regulator-compatible = "ldo_vgp1";
>   				regulator-name = "vcamd";
>   				regulator-min-microvolt = <1220000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -167,7 +150,6 @@
>   			};
>
>   			mt6397_vgp2_reg: ldo_vgp2 {
> -				regulator-compatible = "ldo_vgp2";
>   				regulator-name = "vcamio";
>   				regulator-min-microvolt = <1000000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -175,7 +157,6 @@
>   			};
>
>   			mt6397_vgp3_reg: ldo_vgp3 {
> -				regulator-compatible = "ldo_vgp3";
>   				regulator-name = "vcamaf";
>   				regulator-min-microvolt = <1200000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -183,7 +164,6 @@
>   			};
>
>   			mt6397_vgp4_reg: ldo_vgp4 {
> -				regulator-compatible = "ldo_vgp4";
>   				regulator-name = "vgp4";
>   				regulator-min-microvolt = <1200000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -191,7 +171,6 @@
>   			};
>
>   			mt6397_vgp5_reg: ldo_vgp5 {
> -				regulator-compatible = "ldo_vgp5";
>   				regulator-name = "vgp5";
>   				regulator-min-microvolt = <1200000>;
>   				regulator-max-microvolt = <3000000>;
> @@ -199,7 +178,6 @@
>   			};
>
>   			mt6397_vgp6_reg: ldo_vgp6 {
> -				regulator-compatible = "ldo_vgp6";
>   				regulator-name = "vgp6";
>   				regulator-min-microvolt = <1200000>;
>   				regulator-max-microvolt = <3300000>;
> @@ -207,7 +185,6 @@
>   			};
>
>   			mt6397_vibr_reg: ldo_vibr {
> -				regulator-compatible = "ldo_vibr";
>   				regulator-name = "vibr";
>   				regulator-min-microvolt = <1300000>;
>   				regulator-max-microvolt = <3300000>;
>

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

end of thread, other threads:[~2015-09-27 13:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-25 13:46 [PATCH] ARM: dts: mt8135-evbp1: remove regulator-compatible usage Javier Martinez Canillas
2015-09-27 13:39 ` Matthias Brugger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).