RE: [PATCH V2 6/6] ARM: davinci: da850: add tps6507x regulator DT data

["Vishwanathrao Badarkhe, Manish" <manishv.b@ti.com>]

Hi Mark,

On Tue, Jan 29, 2013 at 13:08:54, Vishwanathrao Badarkhe, Manish wrote:
> Add tps6507x regulator device tree data to da850-evm by adding regulator consumers with tightened constraints and regulator-name.TPS6507x regulator handle can be obtained by using this regulator name.
> Regulator constraints are added as per da850 board file.
> 
> Regulator names are given as per maximum output voltage the regulator is capable to provide.
> for e.g. regulator name for dcdc1 is "VDCDC1_3.3V".
> Also, add tps6507x PMIC I2C slave device under I2C0 node.
> 
> Tested on da850-evm.
> 
> Signed-off-by: Vishwanathrao Badarkhe, Manish <manishv.b@ti.com>
> ---
> Changes since V1:
>  - No change
> 
> :100644 100644 c9ed683... 58e6961... M	arch/arm/boot/dts/da850-evm.dts
>  arch/arm/boot/dts/da850-evm.dts |   63 +++++++++++++++++++++++++++++++++++++++
>  1 files changed, 63 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts index c9ed683..58e6961 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -31,6 +31,10 @@
>  			status = "okay";
>  			pinctrl-names = "default";
>  			pinctrl-0 = <&i2c0_pins>;
> +
> +			tps: tps@48 {
> +				reg = <0x48>;
> +			};
>  		};
>  	};
>  	nand_cs3@62000000 {
> @@ -38,4 +42,63 @@
>  		pinctrl-names = "default";
>  		pinctrl-0 = <&nand_cs3_pins>;
>  	};
> +	vbat: fixedregulator@0 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vbat";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-boot-on;
> +	};
> +};
> +
> +/include/ "tps6507x.dtsi"
> +
> +&tps {
> +        vdcdc1_2-supply = <&vbat>;
> +        vdcdc3-supply = <&vbat>;
> +        vldo1_2-supply = <&vbat>;
> +
> +        regulators {
> +                vdcdc1_reg: regulator@0 {
> +                        regulator-name = "VDCDC1_3.3V";
> +                        regulator-min-microvolt = <3150000>;
> +                        regulator-max-microvolt = <3450000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +
> +                vdcdc2_reg: regulator@1 {
> +                        regulator-name = "VDCDC2_3.3V";
> +                        regulator-min-microvolt = <1710000>;
> +                        regulator-max-microvolt = <3450000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                        ti,defdcdc_default = <1>;
> +                };
> +
> +                vdcdc3_reg: regulator@2 {
> +                        regulator-name = "VDCDC3_1.2V";
> +                        regulator-min-microvolt = <950000>;
> +                        regulator-max-microvolt = <1350000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                        ti,defdcdc_default = <1>;
> +                };
> +
> +                ldo1_reg: regulator@3 {
> +                        regulator-name = "LDO1_1.8V";
> +                        regulator-min-microvolt = <1710000>;
> +                        regulator-max-microvolt = <1890000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +
> +                ldo2_reg: regulator@4 {
> +                        regulator-name = "LDO2_1.2V";
> +                        regulator-min-microvolt = <1140000>;
> +                        regulator-max-microvolt = <1320000>;
> +                        regulator-always-on;
> +                        regulator-boot-on;
> +                };
> +        };
>  };
> --
> 1.7.4.1
> 
> 

If there are not any comments on this patch, Could you please accept 
this patch?

Regards, 
Manish