* [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
@ 2015-10-27 12:36 ` Heiko Schocher
0 siblings, 0 replies; 7+ messages in thread
From: Heiko Schocher @ 2015-10-27 12:36 UTC (permalink / raw)
To: linux-kernel
Cc: Heiko Schocher, linux-arm-kernel, Benoît Cousson,
linux-omap, devicetree, Kumar Gala, Ian Campbell,
Sebastian Reichel, Rob Herring, Georg.Soffel, Pawel Moll,
Mark Rutland, Russell King, Mark Jackson, Tony Lindgren
remove tps65217.dtsi and adapt all boards, which
used it.
Signed-off-by: Heiko Schocher <hs@denx.de>
Tested-by: Keerthy <j-keerthy@ti.com>
Acked-by: Mark Brown <broonie@kernel.org>
---
Suggested by Mark Brown, see:
https://lkml.org/lkml/2015/10/21/581
Changes in v2:
- accidentially removed tps65217.txt
do not remove it, add Sebastian Reichel to
cc, as he also deteted this.
- add Acked-by from Mark Brown
- add Tested-by from j-keerthy
.../devicetree/bindings/regulator/tps65217.txt | 10 ++++
arch/arm/boot/dts/am335x-bone-common.dtsi | 14 ++++--
arch/arm/boot/dts/am335x-chilisom.dtsi | 14 +++++-
arch/arm/boot/dts/am335x-nano.dts | 14 +++++-
arch/arm/boot/dts/am335x-pepper.dts | 14 +++++-
arch/arm/boot/dts/am335x-sl50.dts | 13 ++++-
arch/arm/boot/dts/tps65217.dtsi | 56 ----------------------
7 files changed, 68 insertions(+), 67 deletions(-)
delete mode 100644 arch/arm/boot/dts/tps65217.dtsi
diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
index 4f05d20..d181096 100644
--- a/Documentation/devicetree/bindings/regulator/tps65217.txt
+++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
@@ -26,7 +26,11 @@ Example:
ti,pmic-shutdown-controller;
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: dcdc1 {
+ reg = <0>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1800000>;
regulator-boot-on;
@@ -34,6 +38,7 @@ Example:
};
dcdc2_reg: dcdc2 {
+ reg = <1>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -41,6 +46,7 @@ Example:
};
dcdc3_reg: dcc3 {
+ reg = <2>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1500000>;
regulator-boot-on;
@@ -48,6 +54,7 @@ Example:
};
ldo1_reg: ldo1 {
+ reg = <3>;
regulator-min-microvolt = <1000000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -55,6 +62,7 @@ Example:
};
ldo2_reg: ldo2 {
+ reg = <4>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -62,6 +70,7 @@ Example:
};
ldo3_reg: ldo3 {
+ reg = <5>;
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -69,6 +78,7 @@ Example:
};
ldo4_reg: ldo4 {
+ reg = <6>;
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index fec7834..0c4bde0 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -285,10 +285,8 @@
};
};
-
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
/*
* Configure pmic to enter OFF-state instead of SLEEP-state ("RTC-only
* mode") at poweroff. Most BeagleBone versions do not support RTC-only
@@ -309,12 +307,17 @@
ti,pmic-shutdown-controller;
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator@0 {
+ reg = <0>;
regulator-name = "vdds_dpr";
regulator-always-on;
};
dcdc2_reg: regulator@1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -324,6 +327,7 @@
};
dcdc3_reg: regulator@2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -333,21 +337,25 @@
};
ldo1_reg: regulator@3 {
+ reg = <3>;
regulator-name = "vio,vrtc,vdds";
regulator-always-on;
};
ldo2_reg: regulator@4 {
+ reg = <4>;
regulator-name = "vdd_3v3aux";
regulator-always-on;
};
ldo3_reg: regulator@5 {
+ reg = <5>;
regulator-name = "vdd_1v8";
regulator-always-on;
};
ldo4_reg: regulator@6 {
+ reg = <6>;
regulator-name = "vdd_3v3a";
regulator-always-on;
};
diff --git a/arch/arm/boot/dts/am335x-chilisom.dtsi b/arch/arm/boot/dts/am335x-chilisom.dtsi
index 7e9a34d..9f5ee9a 100644
--- a/arch/arm/boot/dts/am335x-chilisom.dtsi
+++ b/arch/arm/boot/dts/am335x-chilisom.dtsi
@@ -128,16 +128,21 @@
};
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator@0 {
+ reg = <0>;
regulator-name = "vdds_dpr";
regulator-always-on;
};
dcdc2_reg: regulator@1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -147,6 +152,7 @@
};
dcdc3_reg: regulator@2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -156,24 +162,28 @@
};
ldo1_reg: regulator@3 {
+ reg = <3>;
regulator-name = "vio,vrtc,vdds";
regulator-boot-on;
regulator-always-on;
};
ldo2_reg: regulator@4 {
+ reg = <4>;
regulator-name = "vdd_3v3aux";
regulator-boot-on;
regulator-always-on;
};
ldo3_reg: regulator@5 {
+ reg = <5>;
regulator-name = "vdd_1v8";
regulator-boot-on;
regulator-always-on;
};
ldo4_reg: regulator@6 {
+ reg = <6>;
regulator-name = "vdd_3v3d";
regulator-boot-on;
regulator-always-on;
diff --git a/arch/arm/boot/dts/am335x-nano.dts b/arch/arm/boot/dts/am335x-nano.dts
index 5ed4ca6..041adb8 100644
--- a/arch/arm/boot/dts/am335x-nano.dts
+++ b/arch/arm/boot/dts/am335x-nano.dts
@@ -375,11 +375,15 @@
wp-gpios = <&gpio3 18 0>;
};
-#include "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator@0 {
+ reg = <0>;
/* +1.5V voltage with ±4% tolerance */
regulator-min-microvolt = <1450000>;
regulator-max-microvolt = <1550000>;
@@ -388,6 +392,7 @@
};
dcdc2_reg: regulator@1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.1V with ±4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <915000>;
@@ -397,6 +402,7 @@
};
dcdc3_reg: regulator@2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with ±4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <915000>;
@@ -406,6 +412,7 @@
};
ldo1_reg: regulator@3 {
+ reg = <3>;
/* +1.8V voltage with ±4% tolerance */
regulator-min-microvolt = <1750000>;
regulator-max-microvolt = <1870000>;
@@ -414,6 +421,7 @@
};
ldo2_reg: regulator@4 {
+ reg = <4>;
/* +3.3V voltage with ±4% tolerance */
regulator-min-microvolt = <3175000>;
regulator-max-microvolt = <3430000>;
@@ -422,6 +430,7 @@
};
ldo3_reg: regulator@5 {
+ reg = <5>;
/* +1.8V voltage with ±4% tolerance */
regulator-min-microvolt = <1750000>;
regulator-max-microvolt = <1870000>;
@@ -430,6 +439,7 @@
};
ldo4_reg: regulator@6 {
+ reg = <6>;
/* +3.3V voltage with ±4% tolerance */
regulator-min-microvolt = <3175000>;
regulator-max-microvolt = <3430000>;
diff --git a/arch/arm/boot/dts/am335x-pepper.dts b/arch/arm/boot/dts/am335x-pepper.dts
index 7106114..32924a9 100644
--- a/arch/arm/boot/dts/am335x-pepper.dts
+++ b/arch/arm/boot/dts/am335x-pepper.dts
@@ -427,9 +427,9 @@
vin-supply = <&vbat>;
};
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
backlight {
isel = <1>; /* ISET1 */
fdim = <200>; /* TPS65217_BL_FDIM_200HZ */
@@ -437,12 +437,17 @@
};
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator@0 {
+ reg = <0>;
/* VDD_1V8 system supply */
regulator-always-on;
};
dcdc2_reg: regulator@1 {
+ reg = <1>;
/* VDD_CORE voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -452,6 +457,7 @@
};
dcdc3_reg: regulator@2 {
+ reg = <2>;
/* VDD_MPU voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -461,18 +467,21 @@
};
ldo1_reg: regulator@3 {
+ reg = <3>;
/* VRTC 1.8V always-on supply */
regulator-name = "vrtc,vdds";
regulator-always-on;
};
ldo2_reg: regulator@4 {
+ reg = <4>;
/* 3.3V rail */
regulator-name = "vdd_3v3aux";
regulator-always-on;
};
ldo3_reg: regulator@5 {
+ reg = <5>;
/* VDD_3V3A 3.3V rail */
regulator-name = "vdd_3v3a";
regulator-min-microvolt = <3300000>;
@@ -480,6 +489,7 @@
};
ldo4_reg: regulator@6 {
+ reg = <6>;
/* VDD_3V3B 3.3V rail */
regulator-name = "vdd_3v3b";
regulator-always-on;
diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
index 3303c28..d38edfa 100644
--- a/arch/arm/boot/dts/am335x-sl50.dts
+++ b/arch/arm/boot/dts/am335x-sl50.dts
@@ -375,16 +375,19 @@
pinctrl-0 = <&uart4_pins>;
};
-#include "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
ti,pmic-shutdown-controller;
interrupt-parent = <&intc>;
interrupts = <7>; /* NNMI */
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator@0 {
+ reg = <0>;
/* VDDS_DDR */
regulator-min-microvolt = <1500000>;
regulator-max-microvolt = <1500000>;
@@ -392,6 +395,7 @@
};
dcdc2_reg: regulator@1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -401,6 +405,7 @@
};
dcdc3_reg: regulator@2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -410,6 +415,7 @@
};
ldo1_reg: regulator@3 {
+ reg = <3>;
/* VRTC / VIO / VDDS*/
regulator-always-on;
regulator-min-microvolt = <1800000>;
@@ -417,6 +423,7 @@
};
ldo2_reg: regulator@4 {
+ reg = <4>;
/* VDD_3V3AUX */
regulator-always-on;
regulator-min-microvolt = <3300000>;
@@ -424,6 +431,7 @@
};
ldo3_reg: regulator@5 {
+ reg = <5>;
/* VDD_1V8 */
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
@@ -431,6 +439,7 @@
};
ldo4_reg: regulator@6 {
+ reg = <6>;
/* VDD_3V3A */
regulator-min-microvolt = <3300000>;
regulator-max-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tps65217.dtsi b/arch/arm/boot/dts/tps65217.dtsi
deleted file mode 100644
index a632724..0000000
--- a/arch/arm/boot/dts/tps65217.dtsi
+++ /dev/null
@@ -1,56 +0,0 @@
-/*
- * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- */
-
-/*
- * Integrated Power Management Chip
- * http://www.ti.com/lit/ds/symlink/tps65217.pdf
- */
-
-&tps {
- compatible = "ti,tps65217";
-
- regulators {
- #address-cells = <1>;
- #size-cells = <0>;
-
- dcdc1_reg: regulator@0 {
- reg = <0>;
- regulator-compatible = "dcdc1";
- };
-
- dcdc2_reg: regulator@1 {
- reg = <1>;
- regulator-compatible = "dcdc2";
- };
-
- dcdc3_reg: regulator@2 {
- reg = <2>;
- regulator-compatible = "dcdc3";
- };
-
- ldo1_reg: regulator@3 {
- reg = <3>;
- regulator-compatible = "ldo1";
- };
-
- ldo2_reg: regulator@4 {
- reg = <4>;
- regulator-compatible = "ldo2";
- };
-
- ldo3_reg: regulator@5 {
- reg = <5>;
- regulator-compatible = "ldo3";
- };
-
- ldo4_reg: regulator@6 {
- reg = <6>;
- regulator-compatible = "ldo4";
- };
- };
-};
--
2.1.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
@ 2015-10-27 12:36 ` Heiko Schocher
0 siblings, 0 replies; 7+ messages in thread
From: Heiko Schocher @ 2015-10-27 12:36 UTC (permalink / raw)
To: linux-arm-kernel
remove tps65217.dtsi and adapt all boards, which
used it.
Signed-off-by: Heiko Schocher <hs@denx.de>
Tested-by: Keerthy <j-keerthy@ti.com>
Acked-by: Mark Brown <broonie@kernel.org>
---
Suggested by Mark Brown, see:
https://lkml.org/lkml/2015/10/21/581
Changes in v2:
- accidentially removed tps65217.txt
do not remove it, add Sebastian Reichel to
cc, as he also deteted this.
- add Acked-by from Mark Brown
- add Tested-by from j-keerthy
.../devicetree/bindings/regulator/tps65217.txt | 10 ++++
arch/arm/boot/dts/am335x-bone-common.dtsi | 14 ++++--
arch/arm/boot/dts/am335x-chilisom.dtsi | 14 +++++-
arch/arm/boot/dts/am335x-nano.dts | 14 +++++-
arch/arm/boot/dts/am335x-pepper.dts | 14 +++++-
arch/arm/boot/dts/am335x-sl50.dts | 13 ++++-
arch/arm/boot/dts/tps65217.dtsi | 56 ----------------------
7 files changed, 68 insertions(+), 67 deletions(-)
delete mode 100644 arch/arm/boot/dts/tps65217.dtsi
diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
index 4f05d20..d181096 100644
--- a/Documentation/devicetree/bindings/regulator/tps65217.txt
+++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
@@ -26,7 +26,11 @@ Example:
ti,pmic-shutdown-controller;
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: dcdc1 {
+ reg = <0>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1800000>;
regulator-boot-on;
@@ -34,6 +38,7 @@ Example:
};
dcdc2_reg: dcdc2 {
+ reg = <1>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -41,6 +46,7 @@ Example:
};
dcdc3_reg: dcc3 {
+ reg = <2>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <1500000>;
regulator-boot-on;
@@ -48,6 +54,7 @@ Example:
};
ldo1_reg: ldo1 {
+ reg = <3>;
regulator-min-microvolt = <1000000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -55,6 +62,7 @@ Example:
};
ldo2_reg: ldo2 {
+ reg = <4>;
regulator-min-microvolt = <900000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -62,6 +70,7 @@ Example:
};
ldo3_reg: ldo3 {
+ reg = <5>;
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
@@ -69,6 +78,7 @@ Example:
};
ldo4_reg: ldo4 {
+ reg = <6>;
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
regulator-boot-on;
diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index fec7834..0c4bde0 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -285,10 +285,8 @@
};
};
-
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
/*
* Configure pmic to enter OFF-state instead of SLEEP-state ("RTC-only
* mode") at poweroff. Most BeagleBone versions do not support RTC-only
@@ -309,12 +307,17 @@
ti,pmic-shutdown-controller;
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator at 0 {
+ reg = <0>;
regulator-name = "vdds_dpr";
regulator-always-on;
};
dcdc2_reg: regulator at 1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -324,6 +327,7 @@
};
dcdc3_reg: regulator at 2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -333,21 +337,25 @@
};
ldo1_reg: regulator at 3 {
+ reg = <3>;
regulator-name = "vio,vrtc,vdds";
regulator-always-on;
};
ldo2_reg: regulator at 4 {
+ reg = <4>;
regulator-name = "vdd_3v3aux";
regulator-always-on;
};
ldo3_reg: regulator at 5 {
+ reg = <5>;
regulator-name = "vdd_1v8";
regulator-always-on;
};
ldo4_reg: regulator at 6 {
+ reg = <6>;
regulator-name = "vdd_3v3a";
regulator-always-on;
};
diff --git a/arch/arm/boot/dts/am335x-chilisom.dtsi b/arch/arm/boot/dts/am335x-chilisom.dtsi
index 7e9a34d..9f5ee9a 100644
--- a/arch/arm/boot/dts/am335x-chilisom.dtsi
+++ b/arch/arm/boot/dts/am335x-chilisom.dtsi
@@ -128,16 +128,21 @@
};
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator at 0 {
+ reg = <0>;
regulator-name = "vdds_dpr";
regulator-always-on;
};
dcdc2_reg: regulator at 1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -147,6 +152,7 @@
};
dcdc3_reg: regulator at 2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -156,24 +162,28 @@
};
ldo1_reg: regulator at 3 {
+ reg = <3>;
regulator-name = "vio,vrtc,vdds";
regulator-boot-on;
regulator-always-on;
};
ldo2_reg: regulator at 4 {
+ reg = <4>;
regulator-name = "vdd_3v3aux";
regulator-boot-on;
regulator-always-on;
};
ldo3_reg: regulator at 5 {
+ reg = <5>;
regulator-name = "vdd_1v8";
regulator-boot-on;
regulator-always-on;
};
ldo4_reg: regulator at 6 {
+ reg = <6>;
regulator-name = "vdd_3v3d";
regulator-boot-on;
regulator-always-on;
diff --git a/arch/arm/boot/dts/am335x-nano.dts b/arch/arm/boot/dts/am335x-nano.dts
index 5ed4ca6..041adb8 100644
--- a/arch/arm/boot/dts/am335x-nano.dts
+++ b/arch/arm/boot/dts/am335x-nano.dts
@@ -375,11 +375,15 @@
wp-gpios = <&gpio3 18 0>;
};
-#include "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator at 0 {
+ reg = <0>;
/* +1.5V voltage with ?4% tolerance */
regulator-min-microvolt = <1450000>;
regulator-max-microvolt = <1550000>;
@@ -388,6 +392,7 @@
};
dcdc2_reg: regulator at 1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.1V with ?4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <915000>;
@@ -397,6 +402,7 @@
};
dcdc3_reg: regulator at 2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with ?4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <915000>;
@@ -406,6 +412,7 @@
};
ldo1_reg: regulator at 3 {
+ reg = <3>;
/* +1.8V voltage with ?4% tolerance */
regulator-min-microvolt = <1750000>;
regulator-max-microvolt = <1870000>;
@@ -414,6 +421,7 @@
};
ldo2_reg: regulator at 4 {
+ reg = <4>;
/* +3.3V voltage with ?4% tolerance */
regulator-min-microvolt = <3175000>;
regulator-max-microvolt = <3430000>;
@@ -422,6 +430,7 @@
};
ldo3_reg: regulator at 5 {
+ reg = <5>;
/* +1.8V voltage with ?4% tolerance */
regulator-min-microvolt = <1750000>;
regulator-max-microvolt = <1870000>;
@@ -430,6 +439,7 @@
};
ldo4_reg: regulator at 6 {
+ reg = <6>;
/* +3.3V voltage with ?4% tolerance */
regulator-min-microvolt = <3175000>;
regulator-max-microvolt = <3430000>;
diff --git a/arch/arm/boot/dts/am335x-pepper.dts b/arch/arm/boot/dts/am335x-pepper.dts
index 7106114..32924a9 100644
--- a/arch/arm/boot/dts/am335x-pepper.dts
+++ b/arch/arm/boot/dts/am335x-pepper.dts
@@ -427,9 +427,9 @@
vin-supply = <&vbat>;
};
-/include/ "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
+
backlight {
isel = <1>; /* ISET1 */
fdim = <200>; /* TPS65217_BL_FDIM_200HZ */
@@ -437,12 +437,17 @@
};
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator at 0 {
+ reg = <0>;
/* VDD_1V8 system supply */
regulator-always-on;
};
dcdc2_reg: regulator at 1 {
+ reg = <1>;
/* VDD_CORE voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -452,6 +457,7 @@
};
dcdc3_reg: regulator at 2 {
+ reg = <2>;
/* VDD_MPU voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -461,18 +467,21 @@
};
ldo1_reg: regulator at 3 {
+ reg = <3>;
/* VRTC 1.8V always-on supply */
regulator-name = "vrtc,vdds";
regulator-always-on;
};
ldo2_reg: regulator at 4 {
+ reg = <4>;
/* 3.3V rail */
regulator-name = "vdd_3v3aux";
regulator-always-on;
};
ldo3_reg: regulator at 5 {
+ reg = <5>;
/* VDD_3V3A 3.3V rail */
regulator-name = "vdd_3v3a";
regulator-min-microvolt = <3300000>;
@@ -480,6 +489,7 @@
};
ldo4_reg: regulator at 6 {
+ reg = <6>;
/* VDD_3V3B 3.3V rail */
regulator-name = "vdd_3v3b";
regulator-always-on;
diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
index 3303c28..d38edfa 100644
--- a/arch/arm/boot/dts/am335x-sl50.dts
+++ b/arch/arm/boot/dts/am335x-sl50.dts
@@ -375,16 +375,19 @@
pinctrl-0 = <&uart4_pins>;
};
-#include "tps65217.dtsi"
-
&tps {
+ compatible = "ti,tps65217";
ti,pmic-shutdown-controller;
interrupt-parent = <&intc>;
interrupts = <7>; /* NNMI */
regulators {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
dcdc1_reg: regulator at 0 {
+ reg = <0>;
/* VDDS_DDR */
regulator-min-microvolt = <1500000>;
regulator-max-microvolt = <1500000>;
@@ -392,6 +395,7 @@
};
dcdc2_reg: regulator at 1 {
+ reg = <1>;
/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
regulator-name = "vdd_mpu";
regulator-min-microvolt = <925000>;
@@ -401,6 +405,7 @@
};
dcdc3_reg: regulator at 2 {
+ reg = <2>;
/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
regulator-name = "vdd_core";
regulator-min-microvolt = <925000>;
@@ -410,6 +415,7 @@
};
ldo1_reg: regulator at 3 {
+ reg = <3>;
/* VRTC / VIO / VDDS*/
regulator-always-on;
regulator-min-microvolt = <1800000>;
@@ -417,6 +423,7 @@
};
ldo2_reg: regulator at 4 {
+ reg = <4>;
/* VDD_3V3AUX */
regulator-always-on;
regulator-min-microvolt = <3300000>;
@@ -424,6 +431,7 @@
};
ldo3_reg: regulator at 5 {
+ reg = <5>;
/* VDD_1V8 */
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <1800000>;
@@ -431,6 +439,7 @@
};
ldo4_reg: regulator at 6 {
+ reg = <6>;
/* VDD_3V3A */
regulator-min-microvolt = <3300000>;
regulator-max-microvolt = <3300000>;
diff --git a/arch/arm/boot/dts/tps65217.dtsi b/arch/arm/boot/dts/tps65217.dtsi
deleted file mode 100644
index a632724..0000000
--- a/arch/arm/boot/dts/tps65217.dtsi
+++ /dev/null
@@ -1,56 +0,0 @@
-/*
- * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- */
-
-/*
- * Integrated Power Management Chip
- * http://www.ti.com/lit/ds/symlink/tps65217.pdf
- */
-
-&tps {
- compatible = "ti,tps65217";
-
- regulators {
- #address-cells = <1>;
- #size-cells = <0>;
-
- dcdc1_reg: regulator at 0 {
- reg = <0>;
- regulator-compatible = "dcdc1";
- };
-
- dcdc2_reg: regulator at 1 {
- reg = <1>;
- regulator-compatible = "dcdc2";
- };
-
- dcdc3_reg: regulator at 2 {
- reg = <2>;
- regulator-compatible = "dcdc3";
- };
-
- ldo1_reg: regulator at 3 {
- reg = <3>;
- regulator-compatible = "ldo1";
- };
-
- ldo2_reg: regulator at 4 {
- reg = <4>;
- regulator-compatible = "ldo2";
- };
-
- ldo3_reg: regulator at 5 {
- reg = <5>;
- regulator-compatible = "ldo3";
- };
-
- ldo4_reg: regulator at 6 {
- reg = <6>;
- regulator-compatible = "ldo4";
- };
- };
-};
--
2.1.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
2015-10-27 12:36 ` Heiko Schocher
@ 2015-11-12 14:01 ` Rob Herring
-1 siblings, 0 replies; 7+ messages in thread
From: Rob Herring @ 2015-11-12 14:01 UTC (permalink / raw)
To: Heiko Schocher
Cc: linux-kernel, linux-arm-kernel, Benoît Cousson, linux-omap,
devicetree, Kumar Gala, Ian Campbell, Sebastian Reichel,
Georg.Soffel, Pawel Moll, Mark Rutland, Russell King,
Mark Jackson, Tony Lindgren
On Tue, Oct 27, 2015 at 01:36:36PM +0100, Heiko Schocher wrote:
> remove tps65217.dtsi and adapt all boards, which
> used it.
>
> Signed-off-by: Heiko Schocher <hs@denx.de>
> Tested-by: Keerthy <j-keerthy@ti.com>
> Acked-by: Mark Brown <broonie@kernel.org>
Acked-by: Rob Herring <robh@kernel.org>
> ---
> Suggested by Mark Brown, see:
> https://lkml.org/lkml/2015/10/21/581
>
> Changes in v2:
> - accidentially removed tps65217.txt
> do not remove it, add Sebastian Reichel to
> cc, as he also deteted this.
> - add Acked-by from Mark Brown
> - add Tested-by from j-keerthy
>
> .../devicetree/bindings/regulator/tps65217.txt | 10 ++++
> arch/arm/boot/dts/am335x-bone-common.dtsi | 14 ++++--
> arch/arm/boot/dts/am335x-chilisom.dtsi | 14 +++++-
> arch/arm/boot/dts/am335x-nano.dts | 14 +++++-
> arch/arm/boot/dts/am335x-pepper.dts | 14 +++++-
> arch/arm/boot/dts/am335x-sl50.dts | 13 ++++-
> arch/arm/boot/dts/tps65217.dtsi | 56 ----------------------
> 7 files changed, 68 insertions(+), 67 deletions(-)
> delete mode 100644 arch/arm/boot/dts/tps65217.dtsi
>
> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
> index 4f05d20..d181096 100644
> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
> @@ -26,7 +26,11 @@ Example:
> ti,pmic-shutdown-controller;
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: dcdc1 {
> + reg = <0>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1800000>;
> regulator-boot-on;
> @@ -34,6 +38,7 @@ Example:
> };
>
> dcdc2_reg: dcdc2 {
> + reg = <1>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -41,6 +46,7 @@ Example:
> };
>
> dcdc3_reg: dcc3 {
> + reg = <2>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1500000>;
> regulator-boot-on;
> @@ -48,6 +54,7 @@ Example:
> };
>
> ldo1_reg: ldo1 {
> + reg = <3>;
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -55,6 +62,7 @@ Example:
> };
>
> ldo2_reg: ldo2 {
> + reg = <4>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -62,6 +70,7 @@ Example:
> };
>
> ldo3_reg: ldo3 {
> + reg = <5>;
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -69,6 +78,7 @@ Example:
> };
>
> ldo4_reg: ldo4 {
> + reg = <6>;
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
> index fec7834..0c4bde0 100644
> --- a/arch/arm/boot/dts/am335x-bone-common.dtsi
> +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
> @@ -285,10 +285,8 @@
> };
> };
>
> -
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> /*
> * Configure pmic to enter OFF-state instead of SLEEP-state ("RTC-only
> * mode") at poweroff. Most BeagleBone versions do not support RTC-only
> @@ -309,12 +307,17 @@
> ti,pmic-shutdown-controller;
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator@0 {
> + reg = <0>;
> regulator-name = "vdds_dpr";
> regulator-always-on;
> };
>
> dcdc2_reg: regulator@1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -324,6 +327,7 @@
> };
>
> dcdc3_reg: regulator@2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -333,21 +337,25 @@
> };
>
> ldo1_reg: regulator@3 {
> + reg = <3>;
> regulator-name = "vio,vrtc,vdds";
> regulator-always-on;
> };
>
> ldo2_reg: regulator@4 {
> + reg = <4>;
> regulator-name = "vdd_3v3aux";
> regulator-always-on;
> };
>
> ldo3_reg: regulator@5 {
> + reg = <5>;
> regulator-name = "vdd_1v8";
> regulator-always-on;
> };
>
> ldo4_reg: regulator@6 {
> + reg = <6>;
> regulator-name = "vdd_3v3a";
> regulator-always-on;
> };
> diff --git a/arch/arm/boot/dts/am335x-chilisom.dtsi b/arch/arm/boot/dts/am335x-chilisom.dtsi
> index 7e9a34d..9f5ee9a 100644
> --- a/arch/arm/boot/dts/am335x-chilisom.dtsi
> +++ b/arch/arm/boot/dts/am335x-chilisom.dtsi
> @@ -128,16 +128,21 @@
>
> };
>
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator@0 {
> + reg = <0>;
> regulator-name = "vdds_dpr";
> regulator-always-on;
> };
>
> dcdc2_reg: regulator@1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -147,6 +152,7 @@
> };
>
> dcdc3_reg: regulator@2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -156,24 +162,28 @@
> };
>
> ldo1_reg: regulator@3 {
> + reg = <3>;
> regulator-name = "vio,vrtc,vdds";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo2_reg: regulator@4 {
> + reg = <4>;
> regulator-name = "vdd_3v3aux";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo3_reg: regulator@5 {
> + reg = <5>;
> regulator-name = "vdd_1v8";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo4_reg: regulator@6 {
> + reg = <6>;
> regulator-name = "vdd_3v3d";
> regulator-boot-on;
> regulator-always-on;
> diff --git a/arch/arm/boot/dts/am335x-nano.dts b/arch/arm/boot/dts/am335x-nano.dts
> index 5ed4ca6..041adb8 100644
> --- a/arch/arm/boot/dts/am335x-nano.dts
> +++ b/arch/arm/boot/dts/am335x-nano.dts
> @@ -375,11 +375,15 @@
> wp-gpios = <&gpio3 18 0>;
> };
>
> -#include "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator@0 {
> + reg = <0>;
> /* +1.5V voltage with ±4% tolerance */
> regulator-min-microvolt = <1450000>;
> regulator-max-microvolt = <1550000>;
> @@ -388,6 +392,7 @@
> };
>
> dcdc2_reg: regulator@1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.1V with ±4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <915000>;
> @@ -397,6 +402,7 @@
> };
>
> dcdc3_reg: regulator@2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with ±4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <915000>;
> @@ -406,6 +412,7 @@
> };
>
> ldo1_reg: regulator@3 {
> + reg = <3>;
> /* +1.8V voltage with ±4% tolerance */
> regulator-min-microvolt = <1750000>;
> regulator-max-microvolt = <1870000>;
> @@ -414,6 +421,7 @@
> };
>
> ldo2_reg: regulator@4 {
> + reg = <4>;
> /* +3.3V voltage with ±4% tolerance */
> regulator-min-microvolt = <3175000>;
> regulator-max-microvolt = <3430000>;
> @@ -422,6 +430,7 @@
> };
>
> ldo3_reg: regulator@5 {
> + reg = <5>;
> /* +1.8V voltage with ±4% tolerance */
> regulator-min-microvolt = <1750000>;
> regulator-max-microvolt = <1870000>;
> @@ -430,6 +439,7 @@
> };
>
> ldo4_reg: regulator@6 {
> + reg = <6>;
> /* +3.3V voltage with ±4% tolerance */
> regulator-min-microvolt = <3175000>;
> regulator-max-microvolt = <3430000>;
> diff --git a/arch/arm/boot/dts/am335x-pepper.dts b/arch/arm/boot/dts/am335x-pepper.dts
> index 7106114..32924a9 100644
> --- a/arch/arm/boot/dts/am335x-pepper.dts
> +++ b/arch/arm/boot/dts/am335x-pepper.dts
> @@ -427,9 +427,9 @@
> vin-supply = <&vbat>;
> };
>
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> backlight {
> isel = <1>; /* ISET1 */
> fdim = <200>; /* TPS65217_BL_FDIM_200HZ */
> @@ -437,12 +437,17 @@
> };
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator@0 {
> + reg = <0>;
> /* VDD_1V8 system supply */
> regulator-always-on;
> };
>
> dcdc2_reg: regulator@1 {
> + reg = <1>;
> /* VDD_CORE voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -452,6 +457,7 @@
> };
>
> dcdc3_reg: regulator@2 {
> + reg = <2>;
> /* VDD_MPU voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -461,18 +467,21 @@
> };
>
> ldo1_reg: regulator@3 {
> + reg = <3>;
> /* VRTC 1.8V always-on supply */
> regulator-name = "vrtc,vdds";
> regulator-always-on;
> };
>
> ldo2_reg: regulator@4 {
> + reg = <4>;
> /* 3.3V rail */
> regulator-name = "vdd_3v3aux";
> regulator-always-on;
> };
>
> ldo3_reg: regulator@5 {
> + reg = <5>;
> /* VDD_3V3A 3.3V rail */
> regulator-name = "vdd_3v3a";
> regulator-min-microvolt = <3300000>;
> @@ -480,6 +489,7 @@
> };
>
> ldo4_reg: regulator@6 {
> + reg = <6>;
> /* VDD_3V3B 3.3V rail */
> regulator-name = "vdd_3v3b";
> regulator-always-on;
> diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
> index 3303c28..d38edfa 100644
> --- a/arch/arm/boot/dts/am335x-sl50.dts
> +++ b/arch/arm/boot/dts/am335x-sl50.dts
> @@ -375,16 +375,19 @@
> pinctrl-0 = <&uart4_pins>;
> };
>
> -#include "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> ti,pmic-shutdown-controller;
>
> interrupt-parent = <&intc>;
> interrupts = <7>; /* NNMI */
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator@0 {
> + reg = <0>;
> /* VDDS_DDR */
> regulator-min-microvolt = <1500000>;
> regulator-max-microvolt = <1500000>;
> @@ -392,6 +395,7 @@
> };
>
> dcdc2_reg: regulator@1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -401,6 +405,7 @@
> };
>
> dcdc3_reg: regulator@2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -410,6 +415,7 @@
> };
>
> ldo1_reg: regulator@3 {
> + reg = <3>;
> /* VRTC / VIO / VDDS*/
> regulator-always-on;
> regulator-min-microvolt = <1800000>;
> @@ -417,6 +423,7 @@
> };
>
> ldo2_reg: regulator@4 {
> + reg = <4>;
> /* VDD_3V3AUX */
> regulator-always-on;
> regulator-min-microvolt = <3300000>;
> @@ -424,6 +431,7 @@
> };
>
> ldo3_reg: regulator@5 {
> + reg = <5>;
> /* VDD_1V8 */
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> @@ -431,6 +439,7 @@
> };
>
> ldo4_reg: regulator@6 {
> + reg = <6>;
> /* VDD_3V3A */
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tps65217.dtsi b/arch/arm/boot/dts/tps65217.dtsi
> deleted file mode 100644
> index a632724..0000000
> --- a/arch/arm/boot/dts/tps65217.dtsi
> +++ /dev/null
> @@ -1,56 +0,0 @@
> -/*
> - * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
> - */
> -
> -/*
> - * Integrated Power Management Chip
> - * http://www.ti.com/lit/ds/symlink/tps65217.pdf
> - */
> -
> -&tps {
> - compatible = "ti,tps65217";
> -
> - regulators {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - dcdc1_reg: regulator@0 {
> - reg = <0>;
> - regulator-compatible = "dcdc1";
> - };
> -
> - dcdc2_reg: regulator@1 {
> - reg = <1>;
> - regulator-compatible = "dcdc2";
> - };
> -
> - dcdc3_reg: regulator@2 {
> - reg = <2>;
> - regulator-compatible = "dcdc3";
> - };
> -
> - ldo1_reg: regulator@3 {
> - reg = <3>;
> - regulator-compatible = "ldo1";
> - };
> -
> - ldo2_reg: regulator@4 {
> - reg = <4>;
> - regulator-compatible = "ldo2";
> - };
> -
> - ldo3_reg: regulator@5 {
> - reg = <5>;
> - regulator-compatible = "ldo3";
> - };
> -
> - ldo4_reg: regulator@6 {
> - reg = <6>;
> - regulator-compatible = "ldo4";
> - };
> - };
> -};
> --
> 2.1.0
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
@ 2015-11-12 14:01 ` Rob Herring
0 siblings, 0 replies; 7+ messages in thread
From: Rob Herring @ 2015-11-12 14:01 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Oct 27, 2015 at 01:36:36PM +0100, Heiko Schocher wrote:
> remove tps65217.dtsi and adapt all boards, which
> used it.
>
> Signed-off-by: Heiko Schocher <hs@denx.de>
> Tested-by: Keerthy <j-keerthy@ti.com>
> Acked-by: Mark Brown <broonie@kernel.org>
Acked-by: Rob Herring <robh@kernel.org>
> ---
> Suggested by Mark Brown, see:
> https://lkml.org/lkml/2015/10/21/581
>
> Changes in v2:
> - accidentially removed tps65217.txt
> do not remove it, add Sebastian Reichel to
> cc, as he also deteted this.
> - add Acked-by from Mark Brown
> - add Tested-by from j-keerthy
>
> .../devicetree/bindings/regulator/tps65217.txt | 10 ++++
> arch/arm/boot/dts/am335x-bone-common.dtsi | 14 ++++--
> arch/arm/boot/dts/am335x-chilisom.dtsi | 14 +++++-
> arch/arm/boot/dts/am335x-nano.dts | 14 +++++-
> arch/arm/boot/dts/am335x-pepper.dts | 14 +++++-
> arch/arm/boot/dts/am335x-sl50.dts | 13 ++++-
> arch/arm/boot/dts/tps65217.dtsi | 56 ----------------------
> 7 files changed, 68 insertions(+), 67 deletions(-)
> delete mode 100644 arch/arm/boot/dts/tps65217.dtsi
>
> diff --git a/Documentation/devicetree/bindings/regulator/tps65217.txt b/Documentation/devicetree/bindings/regulator/tps65217.txt
> index 4f05d20..d181096 100644
> --- a/Documentation/devicetree/bindings/regulator/tps65217.txt
> +++ b/Documentation/devicetree/bindings/regulator/tps65217.txt
> @@ -26,7 +26,11 @@ Example:
> ti,pmic-shutdown-controller;
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: dcdc1 {
> + reg = <0>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1800000>;
> regulator-boot-on;
> @@ -34,6 +38,7 @@ Example:
> };
>
> dcdc2_reg: dcdc2 {
> + reg = <1>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -41,6 +46,7 @@ Example:
> };
>
> dcdc3_reg: dcc3 {
> + reg = <2>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <1500000>;
> regulator-boot-on;
> @@ -48,6 +54,7 @@ Example:
> };
>
> ldo1_reg: ldo1 {
> + reg = <3>;
> regulator-min-microvolt = <1000000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -55,6 +62,7 @@ Example:
> };
>
> ldo2_reg: ldo2 {
> + reg = <4>;
> regulator-min-microvolt = <900000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -62,6 +70,7 @@ Example:
> };
>
> ldo3_reg: ldo3 {
> + reg = <5>;
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> @@ -69,6 +78,7 @@ Example:
> };
>
> ldo4_reg: ldo4 {
> + reg = <6>;
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <3300000>;
> regulator-boot-on;
> diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
> index fec7834..0c4bde0 100644
> --- a/arch/arm/boot/dts/am335x-bone-common.dtsi
> +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
> @@ -285,10 +285,8 @@
> };
> };
>
> -
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> /*
> * Configure pmic to enter OFF-state instead of SLEEP-state ("RTC-only
> * mode") at poweroff. Most BeagleBone versions do not support RTC-only
> @@ -309,12 +307,17 @@
> ti,pmic-shutdown-controller;
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator at 0 {
> + reg = <0>;
> regulator-name = "vdds_dpr";
> regulator-always-on;
> };
>
> dcdc2_reg: regulator at 1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -324,6 +327,7 @@
> };
>
> dcdc3_reg: regulator at 2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -333,21 +337,25 @@
> };
>
> ldo1_reg: regulator at 3 {
> + reg = <3>;
> regulator-name = "vio,vrtc,vdds";
> regulator-always-on;
> };
>
> ldo2_reg: regulator at 4 {
> + reg = <4>;
> regulator-name = "vdd_3v3aux";
> regulator-always-on;
> };
>
> ldo3_reg: regulator at 5 {
> + reg = <5>;
> regulator-name = "vdd_1v8";
> regulator-always-on;
> };
>
> ldo4_reg: regulator at 6 {
> + reg = <6>;
> regulator-name = "vdd_3v3a";
> regulator-always-on;
> };
> diff --git a/arch/arm/boot/dts/am335x-chilisom.dtsi b/arch/arm/boot/dts/am335x-chilisom.dtsi
> index 7e9a34d..9f5ee9a 100644
> --- a/arch/arm/boot/dts/am335x-chilisom.dtsi
> +++ b/arch/arm/boot/dts/am335x-chilisom.dtsi
> @@ -128,16 +128,21 @@
>
> };
>
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator at 0 {
> + reg = <0>;
> regulator-name = "vdds_dpr";
> regulator-always-on;
> };
>
> dcdc2_reg: regulator at 1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -147,6 +152,7 @@
> };
>
> dcdc3_reg: regulator at 2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -156,24 +162,28 @@
> };
>
> ldo1_reg: regulator at 3 {
> + reg = <3>;
> regulator-name = "vio,vrtc,vdds";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo2_reg: regulator at 4 {
> + reg = <4>;
> regulator-name = "vdd_3v3aux";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo3_reg: regulator at 5 {
> + reg = <5>;
> regulator-name = "vdd_1v8";
> regulator-boot-on;
> regulator-always-on;
> };
>
> ldo4_reg: regulator at 6 {
> + reg = <6>;
> regulator-name = "vdd_3v3d";
> regulator-boot-on;
> regulator-always-on;
> diff --git a/arch/arm/boot/dts/am335x-nano.dts b/arch/arm/boot/dts/am335x-nano.dts
> index 5ed4ca6..041adb8 100644
> --- a/arch/arm/boot/dts/am335x-nano.dts
> +++ b/arch/arm/boot/dts/am335x-nano.dts
> @@ -375,11 +375,15 @@
> wp-gpios = <&gpio3 18 0>;
> };
>
> -#include "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator at 0 {
> + reg = <0>;
> /* +1.5V voltage with ?4% tolerance */
> regulator-min-microvolt = <1450000>;
> regulator-max-microvolt = <1550000>;
> @@ -388,6 +392,7 @@
> };
>
> dcdc2_reg: regulator at 1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.1V with ?4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <915000>;
> @@ -397,6 +402,7 @@
> };
>
> dcdc3_reg: regulator at 2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with ?4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <915000>;
> @@ -406,6 +412,7 @@
> };
>
> ldo1_reg: regulator at 3 {
> + reg = <3>;
> /* +1.8V voltage with ?4% tolerance */
> regulator-min-microvolt = <1750000>;
> regulator-max-microvolt = <1870000>;
> @@ -414,6 +421,7 @@
> };
>
> ldo2_reg: regulator at 4 {
> + reg = <4>;
> /* +3.3V voltage with ?4% tolerance */
> regulator-min-microvolt = <3175000>;
> regulator-max-microvolt = <3430000>;
> @@ -422,6 +430,7 @@
> };
>
> ldo3_reg: regulator at 5 {
> + reg = <5>;
> /* +1.8V voltage with ?4% tolerance */
> regulator-min-microvolt = <1750000>;
> regulator-max-microvolt = <1870000>;
> @@ -430,6 +439,7 @@
> };
>
> ldo4_reg: regulator at 6 {
> + reg = <6>;
> /* +3.3V voltage with ?4% tolerance */
> regulator-min-microvolt = <3175000>;
> regulator-max-microvolt = <3430000>;
> diff --git a/arch/arm/boot/dts/am335x-pepper.dts b/arch/arm/boot/dts/am335x-pepper.dts
> index 7106114..32924a9 100644
> --- a/arch/arm/boot/dts/am335x-pepper.dts
> +++ b/arch/arm/boot/dts/am335x-pepper.dts
> @@ -427,9 +427,9 @@
> vin-supply = <&vbat>;
> };
>
> -/include/ "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> +
> backlight {
> isel = <1>; /* ISET1 */
> fdim = <200>; /* TPS65217_BL_FDIM_200HZ */
> @@ -437,12 +437,17 @@
> };
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator at 0 {
> + reg = <0>;
> /* VDD_1V8 system supply */
> regulator-always-on;
> };
>
> dcdc2_reg: regulator at 1 {
> + reg = <1>;
> /* VDD_CORE voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -452,6 +457,7 @@
> };
>
> dcdc3_reg: regulator at 2 {
> + reg = <2>;
> /* VDD_MPU voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -461,18 +467,21 @@
> };
>
> ldo1_reg: regulator at 3 {
> + reg = <3>;
> /* VRTC 1.8V always-on supply */
> regulator-name = "vrtc,vdds";
> regulator-always-on;
> };
>
> ldo2_reg: regulator at 4 {
> + reg = <4>;
> /* 3.3V rail */
> regulator-name = "vdd_3v3aux";
> regulator-always-on;
> };
>
> ldo3_reg: regulator at 5 {
> + reg = <5>;
> /* VDD_3V3A 3.3V rail */
> regulator-name = "vdd_3v3a";
> regulator-min-microvolt = <3300000>;
> @@ -480,6 +489,7 @@
> };
>
> ldo4_reg: regulator at 6 {
> + reg = <6>;
> /* VDD_3V3B 3.3V rail */
> regulator-name = "vdd_3v3b";
> regulator-always-on;
> diff --git a/arch/arm/boot/dts/am335x-sl50.dts b/arch/arm/boot/dts/am335x-sl50.dts
> index 3303c28..d38edfa 100644
> --- a/arch/arm/boot/dts/am335x-sl50.dts
> +++ b/arch/arm/boot/dts/am335x-sl50.dts
> @@ -375,16 +375,19 @@
> pinctrl-0 = <&uart4_pins>;
> };
>
> -#include "tps65217.dtsi"
> -
> &tps {
> + compatible = "ti,tps65217";
> ti,pmic-shutdown-controller;
>
> interrupt-parent = <&intc>;
> interrupts = <7>; /* NNMI */
>
> regulators {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> dcdc1_reg: regulator at 0 {
> + reg = <0>;
> /* VDDS_DDR */
> regulator-min-microvolt = <1500000>;
> regulator-max-microvolt = <1500000>;
> @@ -392,6 +395,7 @@
> };
>
> dcdc2_reg: regulator at 1 {
> + reg = <1>;
> /* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> regulator-name = "vdd_mpu";
> regulator-min-microvolt = <925000>;
> @@ -401,6 +405,7 @@
> };
>
> dcdc3_reg: regulator at 2 {
> + reg = <2>;
> /* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> regulator-name = "vdd_core";
> regulator-min-microvolt = <925000>;
> @@ -410,6 +415,7 @@
> };
>
> ldo1_reg: regulator at 3 {
> + reg = <3>;
> /* VRTC / VIO / VDDS*/
> regulator-always-on;
> regulator-min-microvolt = <1800000>;
> @@ -417,6 +423,7 @@
> };
>
> ldo2_reg: regulator at 4 {
> + reg = <4>;
> /* VDD_3V3AUX */
> regulator-always-on;
> regulator-min-microvolt = <3300000>;
> @@ -424,6 +431,7 @@
> };
>
> ldo3_reg: regulator at 5 {
> + reg = <5>;
> /* VDD_1V8 */
> regulator-min-microvolt = <1800000>;
> regulator-max-microvolt = <1800000>;
> @@ -431,6 +439,7 @@
> };
>
> ldo4_reg: regulator at 6 {
> + reg = <6>;
> /* VDD_3V3A */
> regulator-min-microvolt = <3300000>;
> regulator-max-microvolt = <3300000>;
> diff --git a/arch/arm/boot/dts/tps65217.dtsi b/arch/arm/boot/dts/tps65217.dtsi
> deleted file mode 100644
> index a632724..0000000
> --- a/arch/arm/boot/dts/tps65217.dtsi
> +++ /dev/null
> @@ -1,56 +0,0 @@
> -/*
> - * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
> - */
> -
> -/*
> - * Integrated Power Management Chip
> - * http://www.ti.com/lit/ds/symlink/tps65217.pdf
> - */
> -
> -&tps {
> - compatible = "ti,tps65217";
> -
> - regulators {
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> - dcdc1_reg: regulator at 0 {
> - reg = <0>;
> - regulator-compatible = "dcdc1";
> - };
> -
> - dcdc2_reg: regulator at 1 {
> - reg = <1>;
> - regulator-compatible = "dcdc2";
> - };
> -
> - dcdc3_reg: regulator at 2 {
> - reg = <2>;
> - regulator-compatible = "dcdc3";
> - };
> -
> - ldo1_reg: regulator at 3 {
> - reg = <3>;
> - regulator-compatible = "ldo1";
> - };
> -
> - ldo2_reg: regulator at 4 {
> - reg = <4>;
> - regulator-compatible = "ldo2";
> - };
> -
> - ldo3_reg: regulator at 5 {
> - reg = <5>;
> - regulator-compatible = "ldo3";
> - };
> -
> - ldo4_reg: regulator at 6 {
> - reg = <6>;
> - regulator-compatible = "ldo4";
> - };
> - };
> -};
> --
> 2.1.0
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
2015-11-12 14:01 ` Rob Herring
(?)
@ 2015-11-30 21:15 ` Tony Lindgren
-1 siblings, 0 replies; 7+ messages in thread
From: Tony Lindgren @ 2015-11-30 21:15 UTC (permalink / raw)
To: Rob Herring
Cc: Heiko Schocher, linux-kernel, linux-arm-kernel,
Benoît Cousson, linux-omap, devicetree, Kumar Gala,
Ian Campbell, Sebastian Reichel, Georg.Soffel, Pawel Moll,
Mark Rutland, Russell King, Mark Jackson
* Rob Herring <robh@kernel.org> [151112 06:02]:
> On Tue, Oct 27, 2015 at 01:36:36PM +0100, Heiko Schocher wrote:
> > remove tps65217.dtsi and adapt all boards, which
> > used it.
> >
> > Signed-off-by: Heiko Schocher <hs@denx.de>
> > Tested-by: Keerthy <j-keerthy@ti.com>
> > Acked-by: Mark Brown <broonie@kernel.org>
>
> Acked-by: Rob Herring <robh@kernel.org>
Applying into omap-for-v4.5/dt thanks.
Tony
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
@ 2015-11-30 21:15 ` Tony Lindgren
0 siblings, 0 replies; 7+ messages in thread
From: Tony Lindgren @ 2015-11-30 21:15 UTC (permalink / raw)
To: Rob Herring
Cc: Heiko Schocher, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Benoît Cousson, linux-omap-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, Kumar Gala, Ian Campbell,
Sebastian Reichel, Georg.Soffel-k21M0aUVSxZWk0Htik3J/w,
Pawel Moll, Mark Rutland, Russell King, Mark Jackson
* Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> [151112 06:02]:
> On Tue, Oct 27, 2015 at 01:36:36PM +0100, Heiko Schocher wrote:
> > remove tps65217.dtsi and adapt all boards, which
> > used it.
> >
> > Signed-off-by: Heiko Schocher <hs-ynQEQJNshbs@public.gmane.org>
> > Tested-by: Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>
> > Acked-by: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Applying into omap-for-v4.5/dt thanks.
Tony
--
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 [flat|nested] 7+ messages in thread
* [PATCH v2] regulator: tps65217: remove tps65217.dtsi file
@ 2015-11-30 21:15 ` Tony Lindgren
0 siblings, 0 replies; 7+ messages in thread
From: Tony Lindgren @ 2015-11-30 21:15 UTC (permalink / raw)
To: linux-arm-kernel
* Rob Herring <robh@kernel.org> [151112 06:02]:
> On Tue, Oct 27, 2015 at 01:36:36PM +0100, Heiko Schocher wrote:
> > remove tps65217.dtsi and adapt all boards, which
> > used it.
> >
> > Signed-off-by: Heiko Schocher <hs@denx.de>
> > Tested-by: Keerthy <j-keerthy@ti.com>
> > Acked-by: Mark Brown <broonie@kernel.org>
>
> Acked-by: Rob Herring <robh@kernel.org>
Applying into omap-for-v4.5/dt thanks.
Tony
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-11-30 21:15 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-27 12:36 [PATCH v2] regulator: tps65217: remove tps65217.dtsi file Heiko Schocher
2015-10-27 12:36 ` Heiko Schocher
2015-11-12 14:01 ` Rob Herring
2015-11-12 14:01 ` Rob Herring
2015-11-30 21:15 ` Tony Lindgren
2015-11-30 21:15 ` Tony Lindgren
2015-11-30 21:15 ` Tony Lindgren
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.