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

2013-02-07 Thread Vishwanathrao Badarkhe, Manish
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 
> ---
> Changes since V1:
>  - No change
> 
> :100644 100644 c9ed683... 58e6961... March/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@6200 {
> @@ -38,4 +42,63 @@
>   pinctrl-names = "default";
>   pinctrl-0 = <&nand_cs3_pins>;
>   };
> + vbat: fixedregulator@0 {
> + compatible = "regulator-fixed";
> + regulator-name = "vbat";
> + regulator-min-microvolt = <500>;
> + regulator-max-microvolt = <500>;
> + 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 = <315>;
> +regulator-max-microvolt = <345>;
> +regulator-always-on;
> +regulator-boot-on;
> +};
> +
> +vdcdc2_reg: regulator@1 {
> +regulator-name = "VDCDC2_3.3V";
> +regulator-min-microvolt = <171>;
> +regulator-max-microvolt = <345>;
> +regulator-always-on;
> +regulator-boot-on;
> +ti,defdcdc_default = <1>;
> +};
> +
> +vdcdc3_reg: regulator@2 {
> +regulator-name = "VDCDC3_1.2V";
> +regulator-min-microvolt = <95>;
> +regulator-max-microvolt = <135>;
> +regulator-always-on;
> +regulator-boot-on;
> +ti,defdcdc_default = <1>;
> +};
> +
> +ldo1_reg: regulator@3 {
> +regulator-name = "LDO1_1.8V";
> +regulator-min-microvolt = <171>;
> +regulator-max-microvolt = <189>;
> +regulator-always-on;
> +regulator-boot-on;
> +};
> +
> +ldo2_reg: regulator@4 {
> +regulator-name = "LDO2_1.2V";
> +regulator-min-microvolt = <114>;
> +regulator-max-microvolt = <132>;
> +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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


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

2013-01-28 Thread Vishwanathrao Badarkhe, Manish
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 
---
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@6200 {
@@ -38,4 +42,63 @@
pinctrl-names = "default";
pinctrl-0 = <&nand_cs3_pins>;
};
+   vbat: fixedregulator@0 {
+   compatible = "regulator-fixed";
+   regulator-name = "vbat";
+   regulator-min-microvolt = <500>;
+   regulator-max-microvolt = <500>;
+   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 = <315>;
+regulator-max-microvolt = <345>;
+regulator-always-on;
+regulator-boot-on;
+};
+
+vdcdc2_reg: regulator@1 {
+regulator-name = "VDCDC2_3.3V";
+regulator-min-microvolt = <171>;
+regulator-max-microvolt = <345>;
+regulator-always-on;
+regulator-boot-on;
+ti,defdcdc_default = <1>;
+};
+
+vdcdc3_reg: regulator@2 {
+regulator-name = "VDCDC3_1.2V";
+regulator-min-microvolt = <95>;
+regulator-max-microvolt = <135>;
+regulator-always-on;
+regulator-boot-on;
+ti,defdcdc_default = <1>;
+};
+
+ldo1_reg: regulator@3 {
+regulator-name = "LDO1_1.8V";
+regulator-min-microvolt = <171>;
+regulator-max-microvolt = <189>;
+regulator-always-on;
+regulator-boot-on;
+};
+
+ldo2_reg: regulator@4 {
+regulator-name = "LDO2_1.2V";
+regulator-min-microvolt = <114>;
+regulator-max-microvolt = <132>;
+regulator-always-on;
+regulator-boot-on;
+};
+};
 };
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/