Re: [PATCH] ARM: DTS: DRA7: Add TPS659038 PMIC nodes

2013-10-04 Thread Keerthy 

Hi Benoit,

Thanks a lot for fixing it.

On Thursday 03 October 2013 07:09 PM, Benoit Cousson wrote:

Hi Keerthy,

On 11/09/2013 07:30, Keerthy wrote:

On Tuesday 10 September 2013 12:51 AM, Nishanth Menon wrote:

On 08/26/2013 12:36 AM, Keerthy wrote:

The Patch adds nodes for TPS659038 PMIC for DRA7 boards.

It is based on top of:
http://comments.gmane.org/gmane.linux.ports.arm.omap/102459.

Documentation:Documentation/devicetree/bindings/mfd/palmas.txt
 Documentation/devicetree/bindings/regulator/palmas-pmic.txt

Boot Tested on DRA7 d1 Board.

Signed-off-by: Keerthy j-keer...@ti.com
---
   arch/arm/boot/dts/dra7-evm.dts |  118

   1 file changed, 118 insertions(+)

Index: linux/arch/arm/boot/dts/dra7-evm.dts
===
--- linux.orig/arch/arm/boot/dts/dra7-evm.dts2013-08-26
09:57:52.496173554 +0530
+++ linux/arch/arm/boot/dts/dra7-evm.dts2013-08-26
10:38:44.995414695 +0530
@@ -93,6 +93,119 @@
   pinctrl-names = default;
   pinctrl-0 = i2c1_pins;
   clock-frequency = 40;
+
+tps659038: tps659038@58 {
+compatible = ti,tps659038;
+reg = 0x58;
+
+tps659038_pmic {
+compatible = ti,tps659038-pmic;
+
+regulators {
+smps123_reg: smps123 {
+/* VDD_MPU */
+regulator-name = smps123;
+regulator-min-microvolt =  85;
+regulator-max-microvolt = 125;
+regulator-always-on;
+regulator-boot-on;
+};
+
+smps45_reg: smps45 {
+/* VDD_DSPEVE */
+regulator-name = smps45;
+regulator-min-microvolt =  85;
+regulator-max-microvolt = 115;
+regulator-boot-on;
+};
+
+smps6_reg: smps6 {
+/* VDD_GPU - over VDD_SMPS6 */
+regulator-name = smps6;
+regulator-min-microvolt = 85;
+regulator-max-microvolt = 1250;
+regulator-boot-on;
+};
+
+smps7_reg: smps7 {
+/* CORE_VDD */
+regulator-name = smps7;
+regulator-min-microvolt = 85;
+regulator-max-microvolt = 103;
+regulator-always-on;
+regulator-boot-on;
+};
+
+smps8_reg: smps8 {
+/* VDD_IVAHD */
+regulator-name = smps8;
+regulator-min-microvolt =  85;
+regulator-max-microvolt = 125;
+regulator-boot-on;
+};
+
+smps9_reg: smps9 {
+/* VDDS1V8 */
+regulator-name = smps9;
+regulator-min-microvolt = 180;
+regulator-max-microvolt = 180;
+regulator-always-on;
+regulator-boot-on;
+};
+
+ldo1_reg: ldo1 {
+/* LDO1_OUT -- SDIO  */
+regulator-name = ldo1;
+regulator-min-microvolt = 180;
+regulator-max-microvolt = 330;
+regulator-boot-on;
+};
+
+ldo2_reg: ldo2 {
+/* VDD_RTCIO */
+/* LDO2 - VDDSHV5, LDO2 also goes to
CAN_PHY_3V3 */
+regulator-name = ldo2;
+regulator-min-microvolt = 330;
+regulator-max-microvolt = 330;
+regulator-boot-on;
+};
+
+ldo3_reg: ldo3 {
+/* VDDA_1V8_PHY */
+regulator-name = ldo3;
+regulator-min-microvolt = 180;
+regulator-max-microvolt = 180;
+regulator-boot-on;
+};
+
+ldo9_reg: ldo9 {
+/* VDD_RTC */
+regulator-name = ldo9;
+regulator-min-microvolt = 105;
+regulator-max-microvolt = 105;
+regulator-boot-on;
+};
+
+ldoln_reg: ldoln {
+/* VDDA_1V8_PLL */
+regulator-name = ldoln;
+regulator-min-microvolt = 180;
+regulator-max-microvolt = 180;
+regulator-always-on;
+

Re: [PATCH] ARM: DTS: DRA7: Add TPS659038 PMIC nodes

2013-10-03 Thread Benoit Cousson 
Hi Keerthy,

On 11/09/2013 07:30, Keerthy wrote:
 On Tuesday 10 September 2013 12:51 AM, Nishanth Menon wrote:
 On 08/26/2013 12:36 AM, Keerthy wrote:
 The Patch adds nodes for TPS659038 PMIC for DRA7 boards.

 It is based on top of:
 http://comments.gmane.org/gmane.linux.ports.arm.omap/102459.

 Documentation:Documentation/devicetree/bindings/mfd/palmas.txt
 Documentation/devicetree/bindings/regulator/palmas-pmic.txt

 Boot Tested on DRA7 d1 Board.

 Signed-off-by: Keerthy j-keer...@ti.com
 ---
   arch/arm/boot/dts/dra7-evm.dts |  118 
 
   1 file changed, 118 insertions(+)

 Index: linux/arch/arm/boot/dts/dra7-evm.dts
 ===
 --- linux.orig/arch/arm/boot/dts/dra7-evm.dts2013-08-26 
 09:57:52.496173554 +0530
 +++ linux/arch/arm/boot/dts/dra7-evm.dts2013-08-26 
 10:38:44.995414695 +0530
 @@ -93,6 +93,119 @@
   pinctrl-names = default;
   pinctrl-0 = i2c1_pins;
   clock-frequency = 40;
 +
 +tps659038: tps659038@58 {
 +compatible = ti,tps659038;
 +reg = 0x58;
 +
 +tps659038_pmic {
 +compatible = ti,tps659038-pmic;
 +
 +regulators {
 +smps123_reg: smps123 {
 +/* VDD_MPU */
 +regulator-name = smps123;
 +regulator-min-microvolt =  85;
 +regulator-max-microvolt = 125;
 +regulator-always-on;
 +regulator-boot-on;
 +};
 +
 +smps45_reg: smps45 {
 +/* VDD_DSPEVE */
 +regulator-name = smps45;
 +regulator-min-microvolt =  85;
 +regulator-max-microvolt = 115;
 +regulator-boot-on;
 +};
 +
 +smps6_reg: smps6 {
 +/* VDD_GPU - over VDD_SMPS6 */
 +regulator-name = smps6;
 +regulator-min-microvolt = 85;
 +regulator-max-microvolt = 1250;
 +regulator-boot-on;
 +};
 +
 +smps7_reg: smps7 {
 +/* CORE_VDD */
 +regulator-name = smps7;
 +regulator-min-microvolt = 85;
 +regulator-max-microvolt = 103;
 +regulator-always-on;
 +regulator-boot-on;
 +};
 +
 +smps8_reg: smps8 {
 +/* VDD_IVAHD */
 +regulator-name = smps8;
 +regulator-min-microvolt =  85;
 +regulator-max-microvolt = 125;
 +regulator-boot-on;
 +};
 +
 +smps9_reg: smps9 {
 +/* VDDS1V8 */
 +regulator-name = smps9;
 +regulator-min-microvolt = 180;
 +regulator-max-microvolt = 180;
 +regulator-always-on;
 +regulator-boot-on;
 +};
 +
 +ldo1_reg: ldo1 {
 +/* LDO1_OUT -- SDIO  */
 +regulator-name = ldo1;
 +regulator-min-microvolt = 180;
 +regulator-max-microvolt = 330;
 +regulator-boot-on;
 +};
 +
 +ldo2_reg: ldo2 {
 +/* VDD_RTCIO */
 +/* LDO2 - VDDSHV5, LDO2 also goes to 
 CAN_PHY_3V3 */
 +regulator-name = ldo2;
 +regulator-min-microvolt = 330;
 +regulator-max-microvolt = 330;
 +regulator-boot-on;
 +};
 +
 +ldo3_reg: ldo3 {
 +/* VDDA_1V8_PHY */
 +regulator-name = ldo3;
 +regulator-min-microvolt = 180;
 +regulator-max-microvolt = 180;
 +regulator-boot-on;
 +};
 +
 +ldo9_reg: ldo9 {
 +/* VDD_RTC */
 +regulator-name = ldo9;
 +regulator-min-microvolt = 105;
 +regulator-max-microvolt = 105;
 +regulator-boot-on;
 +};
 +
 +ldoln_reg: ldoln {
 +/* VDDA_1V8_PLL */
 +regulator-name = ldoln;
 +regulator-min-microvolt = 180;
 +regulator-max-microvolt = 180;
 +

Re: [PATCH] ARM: DTS: DRA7: Add TPS659038 PMIC nodes

2013-09-10 Thread Keerthy 

On Tuesday 10 September 2013 12:51 AM, Nishanth Menon wrote:

On 08/26/2013 12:36 AM, Keerthy wrote:

The Patch adds nodes for TPS659038 PMIC for DRA7 boards.

It is based on top of:
http://comments.gmane.org/gmane.linux.ports.arm.omap/102459.

Documentation:  Documentation/devicetree/bindings/mfd/palmas.txt
Documentation/devicetree/bindings/regulator/palmas-pmic.txt 


Boot Tested on DRA7 d1 Board.

Signed-off-by: Keerthy j-keer...@ti.com
---
  arch/arm/boot/dts/dra7-evm.dts |  118 
  1 file changed, 118 insertions(+)

Index: linux/arch/arm/boot/dts/dra7-evm.dts
===
--- linux.orig/arch/arm/boot/dts/dra7-evm.dts   2013-08-26 09:57:52.496173554 
+0530
+++ linux/arch/arm/boot/dts/dra7-evm.dts2013-08-26 10:38:44.995414695 
+0530
@@ -93,6 +93,119 @@
pinctrl-names = default;
pinctrl-0 = i2c1_pins;
clock-frequency = 40;
+
+   tps659038: tps659038@58 {
+   compatible = ti,tps659038;
+   reg = 0x58;
+
+   tps659038_pmic {
+   compatible = ti,tps659038-pmic;
+
+   regulators {
+   smps123_reg: smps123 {
+   /* VDD_MPU */
+   regulator-name = smps123;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
125;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   smps45_reg: smps45 {
+   /* VDD_DSPEVE */
+   regulator-name = smps45;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
115;
+   regulator-boot-on;
+   };
+
+   smps6_reg: smps6 {
+   /* VDD_GPU - over VDD_SMPS6 */
+   regulator-name = smps6;
+   regulator-min-microvolt = 
85;
+   regulator-max-microvolt = 
1250;
+   regulator-boot-on;
+   };
+
+   smps7_reg: smps7 {
+   /* CORE_VDD */
+   regulator-name = smps7;
+   regulator-min-microvolt = 
85;
+   regulator-max-microvolt = 
103;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   smps8_reg: smps8 {
+   /* VDD_IVAHD */
+   regulator-name = smps8;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
125;
+   regulator-boot-on;
+   };
+
+   smps9_reg: smps9 {
+   /* VDDS1V8 */
+   regulator-name = smps9;
+   regulator-min-microvolt = 
180;
+   regulator-max-microvolt = 
180;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   ldo1_reg: ldo1 {
+   /* LDO1_OUT -- SDIO  */
+   regulator-name = ldo1;
+   regulator-min-microvolt = 
180;
+   regulator-max-microvolt = 
330;
+   regulator-boot-on;
+   };
+
+   ldo2_reg: ldo2 {
+   /* VDD_RTCIO */
+   /* LDO2 - VDDSHV5, LDO2 also 
goes to CAN_PHY_3V3 */
+

Re: [PATCH] ARM: DTS: DRA7: Add TPS659038 PMIC nodes

2013-09-09 Thread Nishanth Menon 
On 08/26/2013 12:36 AM, Keerthy wrote:
 The Patch adds nodes for TPS659038 PMIC for DRA7 boards.
 
 It is based on top of:
 http://comments.gmane.org/gmane.linux.ports.arm.omap/102459.
 
 Documentation:Documentation/devicetree/bindings/mfd/palmas.txt
   Documentation/devicetree/bindings/regulator/palmas-pmic.txt 
 
 
 Boot Tested on DRA7 d1 Board.
 
 Signed-off-by: Keerthy j-keer...@ti.com
 ---
  arch/arm/boot/dts/dra7-evm.dts |  118 
 
  1 file changed, 118 insertions(+)
 
 Index: linux/arch/arm/boot/dts/dra7-evm.dts
 ===
 --- linux.orig/arch/arm/boot/dts/dra7-evm.dts 2013-08-26 09:57:52.496173554 
 +0530
 +++ linux/arch/arm/boot/dts/dra7-evm.dts  2013-08-26 10:38:44.995414695 
 +0530
 @@ -93,6 +93,119 @@
   pinctrl-names = default;
   pinctrl-0 = i2c1_pins;
   clock-frequency = 40;
 +
 + tps659038: tps659038@58 {
 + compatible = ti,tps659038;
 + reg = 0x58;
 +
 + tps659038_pmic {
 + compatible = ti,tps659038-pmic;
 +
 + regulators {
 + smps123_reg: smps123 {
 + /* VDD_MPU */
 + regulator-name = smps123;
 + regulator-min-microvolt =  
 85;
 + regulator-max-microvolt = 
 125;
 + regulator-always-on;
 + regulator-boot-on;
 + };
 +
 + smps45_reg: smps45 {
 + /* VDD_DSPEVE */
 + regulator-name = smps45;
 + regulator-min-microvolt =  
 85;
 + regulator-max-microvolt = 
 115;
 + regulator-boot-on;
 + };
 +
 + smps6_reg: smps6 {
 + /* VDD_GPU - over VDD_SMPS6 */
 + regulator-name = smps6;
 + regulator-min-microvolt = 
 85;
 + regulator-max-microvolt = 
 1250;
 + regulator-boot-on;
 + };
 +
 + smps7_reg: smps7 {
 + /* CORE_VDD */
 + regulator-name = smps7;
 + regulator-min-microvolt = 
 85;
 + regulator-max-microvolt = 
 103;
 + regulator-always-on;
 + regulator-boot-on;
 + };
 +
 + smps8_reg: smps8 {
 + /* VDD_IVAHD */
 + regulator-name = smps8;
 + regulator-min-microvolt =  
 85;
 + regulator-max-microvolt = 
 125;
 + regulator-boot-on;
 + };
 +
 + smps9_reg: smps9 {
 + /* VDDS1V8 */
 + regulator-name = smps9;
 + regulator-min-microvolt = 
 180;
 + regulator-max-microvolt = 
 180;
 + regulator-always-on;
 + regulator-boot-on;
 + };
 +
 + ldo1_reg: ldo1 {
 + /* LDO1_OUT -- SDIO  */
 + regulator-name = ldo1;
 + regulator-min-microvolt = 
 180;
 + regulator-max-microvolt = 
 330;
 + regulator-boot-on;
 + };
 +
 + ldo2_reg: ldo2 {
 + /* VDD_RTCIO */
 + /* LDO2 - VDDSHV5, LDO2 also 
 goes to CAN_PHY_3V3 */
 + regulator-name = ldo2;
 + regulator-min-microvolt

Re: [PATCH] ARM: DTS: DRA7: Add TPS659038 PMIC nodes

2013-09-03 Thread Keerthy 

Benoit,


On Monday 26 August 2013 11:06 AM, Keerthy wrote:

The Patch adds nodes for TPS659038 PMIC for DRA7 boards.

It is based on top of:
http://comments.gmane.org/gmane.linux.ports.arm.omap/102459.

Documentation:  Documentation/devicetree/bindings/mfd/palmas.txt
Documentation/devicetree/bindings/regulator/palmas-pmic.txt 


Boot Tested on DRA7 d1 Board.


Any comments on this patch?


Signed-off-by: Keerthy j-keer...@ti.com
---
  arch/arm/boot/dts/dra7-evm.dts |  118 
  1 file changed, 118 insertions(+)

Index: linux/arch/arm/boot/dts/dra7-evm.dts
===
--- linux.orig/arch/arm/boot/dts/dra7-evm.dts   2013-08-26 09:57:52.496173554 
+0530
+++ linux/arch/arm/boot/dts/dra7-evm.dts2013-08-26 10:38:44.995414695 
+0530
@@ -93,6 +93,119 @@
pinctrl-names = default;
pinctrl-0 = i2c1_pins;
clock-frequency = 40;
+
+   tps659038: tps659038@58 {
+   compatible = ti,tps659038;
+   reg = 0x58;
+
+   tps659038_pmic {
+   compatible = ti,tps659038-pmic;
+
+   regulators {
+   smps123_reg: smps123 {
+   /* VDD_MPU */
+   regulator-name = smps123;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
125;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   smps45_reg: smps45 {
+   /* VDD_DSPEVE */
+   regulator-name = smps45;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
115;
+   regulator-boot-on;
+   };
+
+   smps6_reg: smps6 {
+   /* VDD_GPU - over VDD_SMPS6 */
+   regulator-name = smps6;
+   regulator-min-microvolt = 
85;
+   regulator-max-microvolt = 
1250;
+   regulator-boot-on;
+   };
+
+   smps7_reg: smps7 {
+   /* CORE_VDD */
+   regulator-name = smps7;
+   regulator-min-microvolt = 
85;
+   regulator-max-microvolt = 
103;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   smps8_reg: smps8 {
+   /* VDD_IVAHD */
+   regulator-name = smps8;
+   regulator-min-microvolt =  
85;
+   regulator-max-microvolt = 
125;
+   regulator-boot-on;
+   };
+
+   smps9_reg: smps9 {
+   /* VDDS1V8 */
+   regulator-name = smps9;
+   regulator-min-microvolt = 
180;
+   regulator-max-microvolt = 
180;
+   regulator-always-on;
+   regulator-boot-on;
+   };
+
+   ldo1_reg: ldo1 {
+   /* LDO1_OUT -- SDIO  */
+   regulator-name = ldo1;
+   regulator-min-microvolt = 
180;
+   regulator-max-microvolt = 
330;
+   regulator-boot-on;
+   };
+
+   ldo2_reg: ldo2 {
+   /* VDD_RTCIO */
+   /* LDO2 - VDDSHV5, LDO2 also 
goes to CAN_PHY_3V3 */
+   regulator-name = ldo2;