Re: [PATCH] arm: dts: twl6030: typical connection to omap4 as a separatedtsi file

From: Ruslan Bilovol
Date: Wed Jul 24 2013 - 09:10:27 EST


Hi Uri,

On 07/24/2013 10:05 AM, Uri Yosef wrote:
Hi Ruslan,

Our OMAP4 SOM connection to PMIC is similar to Blaze SDP4430.


Thank you for confirmation, I will add this in next patch series.

Regards,
Ruslan

Regards,
Uri Yosef



On Mon, Jul 22, 2013 at 7:24 PM, Ruslan Bilovol<ruslan.bilovol@xxxxxx>wrote:

Hi Nishanth,

On Fri, Jul 19, 2013 at 5:10 PM, Nishanth Menon<nm@xxxxxx> wrote:
On 07/19/2013 07:10 AM, Ruslan Bilovol wrote:
The OMAP4 SoC family uses specially-designed
PMIC (power management IC) companion chip for power
management needs: TWL6030/TWL6032.
Therefore there is a typical connection of PMIC to OMAP4
so we can figure it out into separate .dtsi file
and do not duplicate over board-specific files.

Tested on OMAP4 SDP board and compile-tested for Panda board

Signed-off-by: Ruslan Bilovol<ruslan.bilovol@xxxxxx>
---
arch/arm/boot/dts/omap4-panda-common.dtsi | 21 +---------------
arch/arm/boot/dts/omap4-sdp.dts | 21 +---------------
arch/arm/boot/dts/twl6030_omap4.dtsi | 38
+++++++++++++++++++++++++++++
arch/arm/boot/dts/omap4-var-som.dts ?
There is no muxing for TWL6030 in that file at all, so I didn't add it.
Moreover, that file seems to be not maintained since added (just
generic fixes or framework changes that came to all other files)
Second thing - I do not have schematics and HW for verification.
But if somebody from Variscite can confirm that omap4 is typically
connected to PMIC, I'm happy to add it as separate patch.
So I CCed Uri Yosef who is original author of the omap4-var-som.dts

Regards,
Ruslan

3 files changed, 40 insertions(+), 40 deletions(-)
create mode 100644 arch/arm/boot/dts/twl6030_omap4.dtsi

diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi
b/arch/arm/boot/dts/omap4-panda-common.dtsi
index faa95b5..d796302 100644
--- a/arch/arm/boot/dts/omap4-panda-common.dtsi
+++ b/arch/arm/boot/dts/omap4-panda-common.dtsi
@@ -109,23 +109,9 @@
};
};

-&omap4_pmx_wkup {
- pinctrl-names = "default";
- pinctrl-0 =<
-&twl6030_wkup_pins
->;
-
- twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
- pinctrl-single,pins =<
- 0x14 (PIN_OUTPUT | MUX_MODE2) /*
fref_clk0_out.sys_drm_msecure */
->;
- };
-};
-
&omap4_pmx_core {
pinctrl-names = "default";
pinctrl-0 =<
-&twl6030_pins
&twl6040_pins
&mcpdm_pins
&mcbsp1_pins
@@ -134,12 +120,6 @@
&hsusbb1_pins
>;

- twl6030_pins: pinmux_twl6030_pins {
- pinctrl-single,pins =<
- 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
/* sys_nirq1.sys_nirq1 */
->;
- };
-
twl6040_pins: pinmux_twl6040_pins {
pinctrl-single,pins =<
0xe0 (PIN_OUTPUT | MUX_MODE3) /*
hdq_sio.gpio_127 */
@@ -265,6 +245,7 @@
};

#include "twl6030.dtsi"
+#include "twl6030_omap4.dtsi"

&i2c2 {
pinctrl-names = "default";
diff --git a/arch/arm/boot/dts/omap4-sdp.dts
b/arch/arm/boot/dts/omap4-sdp.dts
index 7951b4e..91030c3 100644
--- a/arch/arm/boot/dts/omap4-sdp.dts
+++ b/arch/arm/boot/dts/omap4-sdp.dts
@@ -142,23 +142,9 @@
};
};

-&omap4_pmx_wkup {
- pinctrl-names = "default";
- pinctrl-0 =<
-&twl6030_wkup_pins
->;
-
- twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
- pinctrl-single,pins =<
- 0x14 (PIN_OUTPUT | MUX_MODE2) /*
fref_clk0_out.sys_drm_msecure */
->;
- };
-};
-
&omap4_pmx_core {
pinctrl-names = "default";
pinctrl-0 =<
-&twl6030_pins
&twl6040_pins
&mcpdm_pins
&dmic_pins
@@ -193,12 +179,6 @@
>;
};

- twl6030_pins: pinmux_twl6030_pins {
- pinctrl-single,pins =<
- 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
/* sys_nirq1.sys_nirq1 */
->;
- };
-
twl6040_pins: pinmux_twl6040_pins {
pinctrl-single,pins =<
0xe0 (PIN_OUTPUT | MUX_MODE3) /*
hdq_sio.gpio_127 */
@@ -337,6 +317,7 @@
};

#include "twl6030.dtsi"
+#include "twl6030_omap4.dtsi"

&i2c2 {
pinctrl-names = "default";
diff --git a/arch/arm/boot/dts/twl6030_omap4.dtsi
b/arch/arm/boot/dts/twl6030_omap4.dtsi
new file mode 100644
index 0000000..a4fa570
--- /dev/null
+++ b/arch/arm/boot/dts/twl6030_omap4.dtsi
@@ -0,0 +1,38 @@
+/*
+ * Copyright (C) 2013 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.
+ */
+
+&twl {
+ /*
+ * On most OMAP4 platforms, the twl6030 IRQ line is connected
+ * to the SYS_NIRQ1 line on OMAP and the twl6030 MSECURE line is
+ * connected to the fref_clk0_out.sys_drm_msecure line.
+ * Therefore, configure the defaults for the SYS_NIRQ1 and
+ * fref_clk0_out.sys_drm_msecure pins here.
+ */
+ pinctrl-names = "default";
+ pinctrl-0 =<
+&twl6030_pins
+&twl6030_wkup_pins
+>;
+};
+
+&omap4_pmx_wkup {
+ twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
+ pinctrl-single,pins =<
+ 0x14 (PIN_OUTPUT | MUX_MODE2) /*
fref_clk0_out.sys_drm_msecure */
+>;
+ };
+};
+
+&omap4_pmx_core {
+ twl6030_pins: pinmux_twl6030_pins {
+ pinctrl-single,pins =<
+ 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
/* sys_nirq1.sys_nirq1 */
+>;
+ };
+};


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