[PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Keerthy]

The patch series adds i2c1 and tps65917 and related nodes.
The patch series is boot tested on DRA72-EVM.

Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.

Keerthy J (2):
  ARM: dts: dra72-evm: Enable I2C1 node
  ARM: dts: dra72-evm: Add tps65917 PMIC node

 arch/arm/boot/dts/dra72-evm.dts |  120 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 120 insertions(+)

-- 
1.7.9.5

[PATCH 1/2] ARM: dts: dra72-evm: Enable I2C1 node

[Keerthy]

From: Keerthy J <j-keerthy@ti.com>

I2C1 bus is used for the following peripherals
	P8 connector (MLB)
	TLV320AIC3106 Audio codec
	J15 LCD header
	24WC256 eeprom
	TMP102AIDRLT temperature sensor
	PCF8575 GPIO expander
	PCA9306 i2c voltage translator -> Goes to P9 for comm interface
	P2 expansion connector
	TPS65917 PMIC

The slowest speed of all the peripherals seems to be 400KHz.

Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/dra72-evm.dts |   16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
index 5147023..1e12862 100644
--- a/arch/arm/boot/dts/dra72-evm.dts
+++ b/arch/arm/boot/dts/dra72-evm.dts
@@ -19,6 +19,22 @@
 	};
 };
 
+&dra7_pmx_core {
+	i2c1_pins: pinmux_i2c1_pins {
+		pinctrl-single,pins = <
+			0x400 (PIN_INPUT | MUX_MODE0) /* i2c1_sda.i2c1_sda */
+			0x404 (PIN_INPUT | MUX_MODE0) /* i2c1_scl.i2c1_scl */
+		>;
+	};
+};
+
+&i2c1 {
+	status = "okay";
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c1_pins>;
+	clock-frequency = <400000>;
+};
+
 &uart1 {
 	status = "okay";
 };
-- 
1.7.9.5

Re: [PATCH 1/2] ARM: dts: dra72-evm: Enable I2C1 node

[Keerthy]

On Monday 28 July 2014 11:48 AM, Keerthy wrote:
> From: Keerthy J <j-keerthy@ti.com>
>
> I2C1 bus is used for the following peripherals
> 	P8 connector (MLB)
> 	TLV320AIC3106 Audio codec
> 	J15 LCD header
> 	24WC256 eeprom
> 	TMP102AIDRLT temperature sensor
> 	PCF8575 GPIO expander
> 	PCA9306 i2c voltage translator -> Goes to P9 for comm interface
> 	P2 expansion connector
> 	TPS65917 PMIC
>
> The slowest speed of all the peripherals seems to be 400KHz.

Correcting the 'to' list.

>
> Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>   arch/arm/boot/dts/dra72-evm.dts |   16 ++++++++++++++++
>   1 file changed, 16 insertions(+)
>
> diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
> index 5147023..1e12862 100644
> --- a/arch/arm/boot/dts/dra72-evm.dts
> +++ b/arch/arm/boot/dts/dra72-evm.dts
> @@ -19,6 +19,22 @@
>   	};
>   };
>   
> +&dra7_pmx_core {
> +	i2c1_pins: pinmux_i2c1_pins {
> +		pinctrl-single,pins = <
> +			0x400 (PIN_INPUT | MUX_MODE0) /* i2c1_sda.i2c1_sda */
> +			0x404 (PIN_INPUT | MUX_MODE0) /* i2c1_scl.i2c1_scl */
> +		>;
> +	};
> +};
> +
> +&i2c1 {
> +	status = "okay";
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&i2c1_pins>;
> +	clock-frequency = <400000>;
> +};
> +
>   &uart1 {
>   	status = "okay";
>   };

[PATCH 2/2] ARM: dts: dra72-evm: Add tps65917 PMIC node

[Keerthy]

From: Keerthy J <j-keerthy@ti.com>

DRA72x-evm uses TPS65917 PMIC. Add the node.

NOTE: LDO2 is actually unused, but the usage if any is expected to be
between 1.8 to 3.3v IO voltage. So define the node.

NOTE: Interrupt used is crossbar number based.

Tested-by: Lokesh Vutla <lokeshvutla@ti.com> 
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/dra72-evm.dts |  104 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 104 insertions(+)

diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
index 1e12862..4107428 100644
--- a/arch/arm/boot/dts/dra72-evm.dts
+++ b/arch/arm/boot/dts/dra72-evm.dts
@@ -33,6 +33,110 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&i2c1_pins>;
 	clock-frequency = <400000>;
+
+	tps65917: tps65917@58 {
+		compatible = "ti,tps65917";
+		reg = <0x58>;
+
+		interrupts = <GIC_SPI 2 IRQ_TYPE_NONE>;  /* IRQ_SYS_1N */
+		interrupt-parent = <&gic>;
+		interrupt-controller;
+		#interrupt-cells = <2>;
+
+		ti,system-power-controller;
+
+		tps65917_pmic {
+			compatible = "ti,tps65917-pmic";
+
+			regulators {
+				smps1_reg: smps1 {
+					/* VDD_MPU */
+					regulator-name = "smps1";
+					regulator-min-microvolt = <850000>;
+					regulator-max-microvolt = <1250000>;
+					regulator-always-on;
+					regulator-boot-on;
+				};
+
+				smps2_reg: smps2 {
+					/* VDD_CORE */
+					regulator-name = "smps2";
+					regulator-min-microvolt = <850000>;
+					regulator-max-microvolt = <1030000>;
+					regulator-boot-on;
+					regulator-always-on;
+				};
+
+				smps3_reg: smps3 {
+					/* VDD_GPU IVA DSPEVE */
+					regulator-name = "smps3";
+					regulator-min-microvolt = <850000>;
+					regulator-max-microvolt = <1250000>;
+					regulator-boot-on;
+					regulator-always-on;
+				};
+
+				smps4_reg: smps4 {
+					/* VDDS1V8 */
+					regulator-name = "smps4";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <1800000>;
+					regulator-always-on;
+					regulator-boot-on;
+				};
+
+				smps5_reg: smps5 {
+					/* VDD_DDR */
+					regulator-name = "smps5";
+					regulator-min-microvolt = <1350000>;
+					regulator-max-microvolt = <1350000>;
+					regulator-boot-on;
+					regulator-always-on;
+				};
+
+				ldo1_reg: ldo1 {
+					/* LDO1_OUT --> SDIO  */
+					regulator-name = "ldo1";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <3300000>;
+					regulator-boot-on;
+				};
+
+				ldo2_reg: ldo2 {
+					/* LDO2_OUT --> TP1017 (UNUSED)  */
+					regulator-name = "ldo2";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <3300000>;
+				};
+
+				ldo3_reg: ldo3 {
+					/* VDDA_1V8_PHY */
+					regulator-name = "ldo3";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <1800000>;
+					regulator-boot-on;
+					regulator-always-on;
+				};
+
+				ldo5_reg: ldo5 {
+					/* VDDA_1V8_PLL */
+					regulator-name = "ldo5";
+					regulator-min-microvolt = <1800000>;
+					regulator-max-microvolt = <1800000>;
+					regulator-always-on;
+					regulator-boot-on;
+				};
+
+				ldo4_reg: ldo4 {
+					/* VDDA_3V_USB: VDDA_USBHS33 */
+					regulator-name = "ldo4";
+					regulator-min-microvolt = <3300000>;
+					regulator-max-microvolt = <3300000>;
+					regulator-boot-on;
+				};
+			};
+		};
+	};
 };
 
 &uart1 {
-- 
1.7.9.5

Re: [PATCH 2/2] ARM: dts: dra72-evm: Add tps65917 PMIC node

[Keerthy]

On Monday 28 July 2014 11:48 AM, Keerthy wrote:
> From: Keerthy J <j-keerthy@ti.com>
>
> DRA72x-evm uses TPS65917 PMIC. Add the node.
>
> NOTE: LDO2 is actually unused, but the usage if any is expected to be
> between 1.8 to 3.3v IO voltage. So define the node.
>
> NOTE: Interrupt used is crossbar number based.

Correcting the 'to' list.

> Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>   arch/arm/boot/dts/dra72-evm.dts |  104 +++++++++++++++++++++++++++++++++++++++
>   1 file changed, 104 insertions(+)
>
> diff --git a/arch/arm/boot/dts/dra72-evm.dts b/arch/arm/boot/dts/dra72-evm.dts
> index 1e12862..4107428 100644
> --- a/arch/arm/boot/dts/dra72-evm.dts
> +++ b/arch/arm/boot/dts/dra72-evm.dts
> @@ -33,6 +33,110 @@
>   	pinctrl-names = "default";
>   	pinctrl-0 = <&i2c1_pins>;
>   	clock-frequency = <400000>;
> +
> +	tps65917: tps65917@58 {
> +		compatible = "ti,tps65917";
> +		reg = <0x58>;
> +
> +		interrupts = <GIC_SPI 2 IRQ_TYPE_NONE>;  /* IRQ_SYS_1N */
> +		interrupt-parent = <&gic>;
> +		interrupt-controller;
> +		#interrupt-cells = <2>;
> +
> +		ti,system-power-controller;
> +
> +		tps65917_pmic {
> +			compatible = "ti,tps65917-pmic";
> +
> +			regulators {
> +				smps1_reg: smps1 {
> +					/* VDD_MPU */
> +					regulator-name = "smps1";
> +					regulator-min-microvolt = <850000>;
> +					regulator-max-microvolt = <1250000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				smps2_reg: smps2 {
> +					/* VDD_CORE */
> +					regulator-name = "smps2";
> +					regulator-min-microvolt = <850000>;
> +					regulator-max-microvolt = <1030000>;
> +					regulator-boot-on;
> +					regulator-always-on;
> +				};
> +
> +				smps3_reg: smps3 {
> +					/* VDD_GPU IVA DSPEVE */
> +					regulator-name = "smps3";
> +					regulator-min-microvolt = <850000>;
> +					regulator-max-microvolt = <1250000>;
> +					regulator-boot-on;
> +					regulator-always-on;
> +				};
> +
> +				smps4_reg: smps4 {
> +					/* VDDS1V8 */
> +					regulator-name = "smps4";
> +					regulator-min-microvolt = <1800000>;
> +					regulator-max-microvolt = <1800000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				smps5_reg: smps5 {
> +					/* VDD_DDR */
> +					regulator-name = "smps5";
> +					regulator-min-microvolt = <1350000>;
> +					regulator-max-microvolt = <1350000>;
> +					regulator-boot-on;
> +					regulator-always-on;
> +				};
> +
> +				ldo1_reg: ldo1 {
> +					/* LDO1_OUT --> SDIO  */
> +					regulator-name = "ldo1";
> +					regulator-min-microvolt = <1800000>;
> +					regulator-max-microvolt = <3300000>;
> +					regulator-boot-on;
> +				};
> +
> +				ldo2_reg: ldo2 {
> +					/* LDO2_OUT --> TP1017 (UNUSED)  */
> +					regulator-name = "ldo2";
> +					regulator-min-microvolt = <1800000>;
> +					regulator-max-microvolt = <3300000>;
> +				};
> +
> +				ldo3_reg: ldo3 {
> +					/* VDDA_1V8_PHY */
> +					regulator-name = "ldo3";
> +					regulator-min-microvolt = <1800000>;
> +					regulator-max-microvolt = <1800000>;
> +					regulator-boot-on;
> +					regulator-always-on;
> +				};
> +
> +				ldo5_reg: ldo5 {
> +					/* VDDA_1V8_PLL */
> +					regulator-name = "ldo5";
> +					regulator-min-microvolt = <1800000>;
> +					regulator-max-microvolt = <1800000>;
> +					regulator-always-on;
> +					regulator-boot-on;
> +				};
> +
> +				ldo4_reg: ldo4 {
> +					/* VDDA_3V_USB: VDDA_USBHS33 */
> +					regulator-name = "ldo4";
> +					regulator-min-microvolt = <3300000>;
> +					regulator-max-microvolt = <3300000>;
> +					regulator-boot-on;
> +				};
> +			};
> +		};
> +	};
>   };
>   
>   &uart1 {

Re: [PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Lokesh Vutla]

On Monday 28 July 2014 11:48 AM, Keerthy wrote:
> The patch series adds i2c1 and tps65917 and related nodes.
> The patch series is boot tested on DRA72-EVM.
> 
> Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.
For booting, this is dependent on recent hwmod fix patch series posted by me:
http://www.mail-archive.com/linux-omap@vger.kernel.org/msg108001.html

Boot log for this series can be found here:
http://paste.ubuntu.com/7881545/

Thanks and regards,
Lokesh
> 
> Keerthy J (2):
>   ARM: dts: dra72-evm: Enable I2C1 node
>   ARM: dts: dra72-evm: Add tps65917 PMIC node
> 
>  arch/arm/boot/dts/dra72-evm.dts |  120 +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 120 insertions(+)
> 

Re: [PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Keerthy]

On Monday 28 July 2014 12:00 PM, Lokesh Vutla wrote:
> On Monday 28 July 2014 11:48 AM, Keerthy wrote:
>> The patch series adds i2c1 and tps65917 and related nodes.
>> The patch series is boot tested on DRA72-EVM.
>>
>> Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.

Correcting the 'to' list.

> For booting, this is dependent on recent hwmod fix patch series posted by me:
> http://www.mail-archive.com/linux-omap@vger.kernel.org/msg108001.html
>
> Boot log for this series can be found here:
> http://paste.ubuntu.com/7881545/

Thanks Lokesh!

> Thanks and regards,
> Lokesh
>> Keerthy J (2):
>>    ARM: dts: dra72-evm: Enable I2C1 node
>>    ARM: dts: dra72-evm: Add tps65917 PMIC node
>>
>>   arch/arm/boot/dts/dra72-evm.dts |  120 +++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 120 insertions(+)
>>

Re: [PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Keerthy]

On Monday 28 July 2014 02:52 PM, Keerthy wrote:
> On Monday 28 July 2014 12:00 PM, Lokesh Vutla wrote:
>> On Monday 28 July 2014 11:48 AM, Keerthy wrote:
>>> The patch series adds i2c1 and tps65917 and related nodes.
>>> The patch series is boot tested on DRA72-EVM.
>>>
>>> Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.
>
> Correcting the 'to' list.

Hi Tony,

If there are no comments could you please pull these two patches?

Regards,
Keerthy
>
>> For booting, this is dependent on recent hwmod fix patch series 
>> posted by me:
>> http://www.mail-archive.com/linux-omap@vger.kernel.org/msg108001.html
>>
>> Boot log for this series can be found here:
>> http://paste.ubuntu.com/7881545/
>
> Thanks Lokesh!
>
>> Thanks and regards,
>> Lokesh
>>> Keerthy J (2):
>>>    ARM: dts: dra72-evm: Enable I2C1 node
>>>    ARM: dts: dra72-evm: Add tps65917 PMIC node
>>>
>>>   arch/arm/boot/dts/dra72-evm.dts |  120 
>>> +++++++++++++++++++++++++++++++++++++++
>>>   1 file changed, 120 insertions(+)
>>>
>

Re: [PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Nishanth Menon]

On 18:05-20140812, Keerthy wrote:
> On Monday 28 July 2014 02:52 PM, Keerthy wrote:
> >On Monday 28 July 2014 12:00 PM, Lokesh Vutla wrote:
> >>On Monday 28 July 2014 11:48 AM, Keerthy wrote:
> >>>The patch series adds i2c1 and tps65917 and related nodes.
> >>>The patch series is boot tested on DRA72-EVM.
> >>>
> >>>Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.
> >
> >Correcting the 'to' list.
> 
> Hi Tony,
> 
> If there are no comments could you please pull these two patches?

With 3.17-rc4, I tested this series:
dra72-evm-before: http://paste.ubuntu.org.cn/1496342
dra72-evm-after: http://paste.ubuntu.org.cn/1496341

Patches:
https://patchwork.kernel.org/patch/4632391/
https://patchwork.kernel.org/patch/4632421/

Could we pick these with 3.18?

Please free to add:

Tested-by: Nishanth Menon <nm@ti.com>
-- 
Regards,
Nishanth Menon

Re: [PATCH 0/2] ARM: dts: dra72-evm: Add i2c1 and tps65917 nodes

[Tony Lindgren]

* Nishanth Menon <nm@ti.com> [140908 13:05]:
> On 18:05-20140812, Keerthy wrote:
> > On Monday 28 July 2014 02:52 PM, Keerthy wrote:
> > >On Monday 28 July 2014 12:00 PM, Lokesh Vutla wrote:
> > >>On Monday 28 July 2014 11:48 AM, Keerthy wrote:
> > >>>The patch series adds i2c1 and tps65917 and related nodes.
> > >>>The patch series is boot tested on DRA72-EVM.
> > >>>
> > >>>Thanks to Lokesh <lokeshvutla@ti.com> for testing the series.
> > >
> > >Correcting the 'to' list.
> > 
> > Hi Tony,
> > 
> > If there are no comments could you please pull these two patches?
> 
> With 3.17-rc4, I tested this series:
> dra72-evm-before: http://paste.ubuntu.org.cn/1496342
> dra72-evm-after: http://paste.ubuntu.org.cn/1496341
> 
> Patches:
> https://patchwork.kernel.org/patch/4632391/
> https://patchwork.kernel.org/patch/4632421/
> 
> Could we pick these with 3.18?
> 
> Please free to add:
> 
> Tested-by: Nishanth Menon <nm@ti.com>

OK applying into omap-for-v3.18/dt thanks.

Regards,

Tony