Re: [PATCH v2] regulator: tps65217: remove tps65217.dtsi file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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@xxxxxxx>
> Tested-by: Keerthy <j-keerthy@xxxxxx>
> Acked-by: Mark Brown <broonie@xxxxxxxxxx>

Acked-by: Rob Herring <robh@xxxxxxxxxx>

> ---
> 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
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux