* [PATCH 0/3] ARM: DTS: misc. OMAP3 fixes for proper wakeup
@ 2013-05-31 22:45 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: linux-arm-kernel
Benoit,
Here are handful of DT updates while working on getting PM functionality
working for DT boot on OMAP3.
Kevin
Kevin Hilman (3):
ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
wakeup
ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
arch/arm/boot/dts/omap3-beagle-xm.dts | 39 +++++++++++++++++++++++++++++++++++
arch/arm/boot/dts/omap3-beagle.dts | 37 +++++++++++++++++++++++++++++++++
arch/arm/boot/dts/omap3-overo.dtsi | 14 +++++++++++++
arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++
4 files changed, 107 insertions(+)
--
1.8.2
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
2013-05-31 22:45 ` Kevin Hilman
@ 2013-05-31 22:45 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: Benoit Cousson, linux-omap; +Cc: linux-arm-kernel
Ensure the console uart (UART3) on these boards is mux'd correctly, and
IO ring wakeup is enabled.
This is needed for serial console wakeups when using DT boot.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
3 files changed, 40 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index 3046d1f..ca067b0 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -126,3 +126,17 @@
mode = <3>;
power = <50>;
};
+
+&omap3_pmx_core {
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
+};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index 6eec699..b004372 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -95,6 +95,13 @@
0x1ae 0x10b /* USBB2_ULPITLL_CLK_MUXMODE.usbb1_ulpiphy_dat7 INPUT | PULLDOWN */
>;
};
+
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
};
&i2c1 {
@@ -142,3 +149,8 @@
*/
ti,pulldowns = <0x03a1c4>;
};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index a626c50..d63ed39 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -77,3 +77,17 @@
mode = <3>;
power = <50>;
};
+
+&omap3_pmx_core {
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
+};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
@ 2013-05-31 22:45 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: linux-arm-kernel
Ensure the console uart (UART3) on these boards is mux'd correctly, and
IO ring wakeup is enabled.
This is needed for serial console wakeups when using DT boot.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
3 files changed, 40 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index 3046d1f..ca067b0 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -126,3 +126,17 @@
mode = <3>;
power = <50>;
};
+
+&omap3_pmx_core {
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
+};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index 6eec699..b004372 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -95,6 +95,13 @@
0x1ae 0x10b /* USBB2_ULPITLL_CLK_MUXMODE.usbb1_ulpiphy_dat7 INPUT | PULLDOWN */
>;
};
+
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
};
&i2c1 {
@@ -142,3 +149,8 @@
*/
ti,pulldowns = <0x03a1c4>;
};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index a626c50..d63ed39 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -77,3 +77,17 @@
mode = <3>;
power = <50>;
};
+
+&omap3_pmx_core {
+ uart3_pins: pinmux_uart3_pins {
+ pinctrl-single,pins = <
+ 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
+ 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
+ >;
+ };
+};
+
+&uart3 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&uart3_pins>;
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* Re: [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
2013-05-31 22:45 ` Kevin Hilman
@ 2013-06-03 9:36 ` Florian Vaussard
-1 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:36 UTC (permalink / raw)
To: Kevin Hilman; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> Ensure the console uart (UART3) on these boards is mux'd correctly, and
> IO ring wakeup is enabled.
>
> This is needed for serial console wakeups when using DT boot.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
> arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
> arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
> 3 files changed, 40 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index 3046d1f..ca067b0 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -126,3 +126,17 @@
> mode = <3>;
> power = <50>;
> };
> +
> +&omap3_pmx_core {
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
If you go through Benoit's tree, could you use the newly introduced
constants [1]? They are defined
in include/dt-bindings/pinctrl/omap.h. The above will look:
0x16e (PIN_INPUT | WAKEUP_EVENT | MUX_MODE0) /*
uart3_rx_irrx.uart3_rx_irrx */
0x170 (PIN_OUTPUT | MUX_MODE0) /* uart3_tx_irtx.uart3_tx_irtx */
> + >;
> + };
> +};
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index 6eec699..b004372 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -95,6 +95,13 @@
> 0x1ae 0x10b /* USBB2_ULPITLL_CLK_MUXMODE.usbb1_ulpiphy_dat7 INPUT | PULLDOWN */
> >;
> };
> +
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
Same here
> + >;
> + };
> };
>
> &i2c1 {
> @@ -142,3 +149,8 @@
> */
> ti,pulldowns = <0x03a1c4>;
> };
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index a626c50..d63ed39 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -77,3 +77,17 @@
> mode = <3>;
> power = <50>;
> };
> +
> +&omap3_pmx_core {
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
Same here
> + >;
> + };
> +};
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
>
Regards,
Florian
[1] http://thread.gmane.org/gmane.linux.drivers.devicetree/36951
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
@ 2013-06-03 9:36 ` Florian Vaussard
0 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:36 UTC (permalink / raw)
To: linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> Ensure the console uart (UART3) on these boards is mux'd correctly, and
> IO ring wakeup is enabled.
>
> This is needed for serial console wakeups when using DT boot.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
> arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
> arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
> 3 files changed, 40 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index 3046d1f..ca067b0 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -126,3 +126,17 @@
> mode = <3>;
> power = <50>;
> };
> +
> +&omap3_pmx_core {
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
If you go through Benoit's tree, could you use the newly introduced
constants [1]? They are defined
in include/dt-bindings/pinctrl/omap.h. The above will look:
0x16e (PIN_INPUT | WAKEUP_EVENT | MUX_MODE0) /*
uart3_rx_irrx.uart3_rx_irrx */
0x170 (PIN_OUTPUT | MUX_MODE0) /* uart3_tx_irtx.uart3_tx_irtx */
> + >;
> + };
> +};
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index 6eec699..b004372 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -95,6 +95,13 @@
> 0x1ae 0x10b /* USBB2_ULPITLL_CLK_MUXMODE.usbb1_ulpiphy_dat7 INPUT | PULLDOWN */
> >;
> };
> +
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
Same here
> + >;
> + };
> };
>
> &i2c1 {
> @@ -142,3 +149,8 @@
> */
> ti,pulldowns = <0x03a1c4>;
> };
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index a626c50..d63ed39 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -77,3 +77,17 @@
> mode = <3>;
> power = <50>;
> };
> +
> +&omap3_pmx_core {
> + uart3_pins: pinmux_uart3_pins {
> + pinctrl-single,pins = <
> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
Same here
> + >;
> + };
> +};
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&uart3_pins>;
> +};
>
Regards,
Florian
[1] http://thread.gmane.org/gmane.linux.drivers.devicetree/36951
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
2013-06-03 9:36 ` Florian Vaussard
@ 2013-06-03 21:26 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-03 21:26 UTC (permalink / raw)
To: florian.vaussard; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
Florian Vaussard <florian.vaussard@epfl.ch> writes:
> Hello Kevin,
>
> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> Ensure the console uart (UART3) on these boards is mux'd correctly, and
>> IO ring wakeup is enabled.
>>
>> This is needed for serial console wakeups when using DT boot.
>>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
>> arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
>> arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
>> 3 files changed, 40 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
>> index 3046d1f..ca067b0 100644
>> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
>> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
>> @@ -126,3 +126,17 @@
>> mode = <3>;
>> power = <50>;
>> };
>> +
>> +&omap3_pmx_core {
>> + uart3_pins: pinmux_uart3_pins {
>> + pinctrl-single,pins = <
>> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
>> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
>
> If you go through Benoit's tree, could you use the newly introduced
> constants [1]? They are defined
> in include/dt-bindings/pinctrl/omap.h. The above will look:
>
> 0x16e (PIN_INPUT | WAKEUP_EVENT | MUX_MODE0)
> /* uart3_rx_irrx.uart3_rx_irrx */
> 0x170 (PIN_OUTPUT | MUX_MODE0) /* uart3_tx_irtx.uart3_tx_irtx */
Excellent, thanks for the pointer. I hadn't yet read through your
pre-processor series, but I really like the readability improvements.
I'll respin these on top of your changes.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 1/3] ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
@ 2013-06-03 21:26 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-03 21:26 UTC (permalink / raw)
To: linux-arm-kernel
Florian Vaussard <florian.vaussard@epfl.ch> writes:
> Hello Kevin,
>
> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> Ensure the console uart (UART3) on these boards is mux'd correctly, and
>> IO ring wakeup is enabled.
>>
>> This is needed for serial console wakeups when using DT boot.
>>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> arch/arm/boot/dts/omap3-beagle-xm.dts | 14 ++++++++++++++
>> arch/arm/boot/dts/omap3-beagle.dts | 12 ++++++++++++
>> arch/arm/boot/dts/omap3-overo.dtsi | 14 ++++++++++++++
>> 3 files changed, 40 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
>> index 3046d1f..ca067b0 100644
>> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
>> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
>> @@ -126,3 +126,17 @@
>> mode = <3>;
>> power = <50>;
>> };
>> +
>> +&omap3_pmx_core {
>> + uart3_pins: pinmux_uart3_pins {
>> + pinctrl-single,pins = <
>> + 0x16e 0x4100 /* uart3_rx_irrx.uart3_rx_irrx WAKEUP | INPUT | MODE0 */
>> + 0x170 0x0 /* uart3_tx_irtx.uart3_tx_irtx OUTPUT | MODE0 */
>
> If you go through Benoit's tree, could you use the newly introduced
> constants [1]? They are defined
> in include/dt-bindings/pinctrl/omap.h. The above will look:
>
> 0x16e (PIN_INPUT | WAKEUP_EVENT | MUX_MODE0)
> /* uart3_rx_irrx.uart3_rx_irrx */
> 0x170 (PIN_OUTPUT | MUX_MODE0) /* uart3_tx_irtx.uart3_tx_irtx */
Excellent, thanks for the pointer. I hadn't yet read through your
pre-processor series, but I really like the readability improvements.
I'll respin these on top of your changes.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH 2/3] ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
2013-05-31 22:45 ` Kevin Hilman
@ 2013-05-31 22:45 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: Benoit Cousson, linux-omap; +Cc: linux-arm-kernel
Using the gpio-keys bindings, configure the user button on Beagle
boards. Since the user button is enabled as a wakeup source, also
ensure the GPIO pin is mux'd correctly and has IO ring wakeups enabled,
so it can also wakeup from off mode.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/omap3-beagle-xm.dts | 25 +++++++++++++++++++++++++
arch/arm/boot/dts/omap3-beagle.dts | 25 +++++++++++++++++++++++++
2 files changed, 50 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ca067b0..c93c36f 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -57,6 +57,26 @@
ti,mcbsp = <&mcbsp2>;
ti,codec = <&twl_audio>;
};
+
+ gpio_keys {
+ compatible = "gpio-keys";
+
+ user {
+ label = "user";
+ gpios = <&gpio1 4 0>;
+ linux,code = <0x114>;
+ gpio-key,wakeup;
+ };
+
+ };
+};
+
+&omap3_pmx_wkup {
+ gpio1_pins: pinmux_gpio1_pins {
+ pinctrl-single,pins = <
+ 0x0e 0x4104 /* sys_boot2.gpio_4 WAKEUP | INPUT | MODE4 */
+ >;
+ };
};
&i2c1 {
@@ -140,3 +160,8 @@
pinctrl-names = "default";
pinctrl-0 = <&uart3_pins>;
};
+
+&gpio1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&gpio1_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index b004372..c9794cb 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -71,6 +71,26 @@
reset-supply = <&hsusb2_reset>;
vcc-supply = <&hsusb2_power>;
};
+
+ gpio_keys {
+ compatible = "gpio-keys";
+
+ user {
+ label = "user";
+ gpios = <&gpio1 7 0>;
+ linux,code = <0x114>;
+ gpio-key,wakeup;
+ };
+
+ };
+};
+
+&omap3_pmx_wkup {
+ gpio1_pins: pinmux_gpio1_pins {
+ pinctrl-single,pins = <
+ 0x14 0x4104 /* sys_boot5.gpio_7 WAKEUP | INPUT | MODE4 */
+ >;
+ };
};
&omap3_pmx_core {
@@ -154,3 +174,8 @@
pinctrl-names = "default";
pinctrl-0 = <&uart3_pins>;
};
+
+&gpio1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&gpio1_pins>;
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* [PATCH 2/3] ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
@ 2013-05-31 22:45 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: linux-arm-kernel
Using the gpio-keys bindings, configure the user button on Beagle
boards. Since the user button is enabled as a wakeup source, also
ensure the GPIO pin is mux'd correctly and has IO ring wakeups enabled,
so it can also wakeup from off mode.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/omap3-beagle-xm.dts | 25 +++++++++++++++++++++++++
arch/arm/boot/dts/omap3-beagle.dts | 25 +++++++++++++++++++++++++
2 files changed, 50 insertions(+)
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ca067b0..c93c36f 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -57,6 +57,26 @@
ti,mcbsp = <&mcbsp2>;
ti,codec = <&twl_audio>;
};
+
+ gpio_keys {
+ compatible = "gpio-keys";
+
+ user {
+ label = "user";
+ gpios = <&gpio1 4 0>;
+ linux,code = <0x114>;
+ gpio-key,wakeup;
+ };
+
+ };
+};
+
+&omap3_pmx_wkup {
+ gpio1_pins: pinmux_gpio1_pins {
+ pinctrl-single,pins = <
+ 0x0e 0x4104 /* sys_boot2.gpio_4 WAKEUP | INPUT | MODE4 */
+ >;
+ };
};
&i2c1 {
@@ -140,3 +160,8 @@
pinctrl-names = "default";
pinctrl-0 = <&uart3_pins>;
};
+
+&gpio1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&gpio1_pins>;
+};
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index b004372..c9794cb 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -71,6 +71,26 @@
reset-supply = <&hsusb2_reset>;
vcc-supply = <&hsusb2_power>;
};
+
+ gpio_keys {
+ compatible = "gpio-keys";
+
+ user {
+ label = "user";
+ gpios = <&gpio1 7 0>;
+ linux,code = <0x114>;
+ gpio-key,wakeup;
+ };
+
+ };
+};
+
+&omap3_pmx_wkup {
+ gpio1_pins: pinmux_gpio1_pins {
+ pinctrl-single,pins = <
+ 0x14 0x4104 /* sys_boot5.gpio_7 WAKEUP | INPUT | MODE4 */
+ >;
+ };
};
&omap3_pmx_core {
@@ -154,3 +174,8 @@
pinctrl-names = "default";
pinctrl-0 = <&uart3_pins>;
};
+
+&gpio1 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&gpio1_pins>;
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* Re: [PATCH 2/3] ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
2013-05-31 22:45 ` Kevin Hilman
@ 2013-06-03 9:40 ` Florian Vaussard
-1 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:40 UTC (permalink / raw)
To: Kevin Hilman; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> Using the gpio-keys bindings, configure the user button on Beagle
> boards. Since the user button is enabled as a wakeup source, also
> ensure the GPIO pin is mux'd correctly and has IO ring wakeups enabled,
> so it can also wakeup from off mode.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/omap3-beagle-xm.dts | 25 +++++++++++++++++++++++++
> arch/arm/boot/dts/omap3-beagle.dts | 25 +++++++++++++++++++++++++
> 2 files changed, 50 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ca067b0..c93c36f 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -57,6 +57,26 @@
> ti,mcbsp = <&mcbsp2>;
> ti,codec = <&twl_audio>;
> };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> +
> + user {
> + label = "user";
> + gpios = <&gpio1 4 0>;
Now you can also use GPIO constants (include/dt-bindings/gpio/gpio.h),
like this:
gpios = <&gpio1 4 GPIO_ACTIVE_HIGH>;
> + linux,code = <0x114>;
> + gpio-key,wakeup;
> + };
> +
> + };
> +};
> +
> +&omap3_pmx_wkup {
> + gpio1_pins: pinmux_gpio1_pins {
> + pinctrl-single,pins = <
> + 0x0e 0x4104 /* sys_boot2.gpio_4 WAKEUP | INPUT | MODE4 */
As for patch 1, you can use pinctrl constants
> + >;
> + };
> };
>
> &i2c1 {
> @@ -140,3 +160,8 @@
> pinctrl-names = "default";
> pinctrl-0 = <&uart3_pins>;
> };
> +
> +&gpio1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio1_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index b004372..c9794cb 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -71,6 +71,26 @@
> reset-supply = <&hsusb2_reset>;
> vcc-supply = <&hsusb2_power>;
> };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> +
> + user {
> + label = "user";
> + gpios = <&gpio1 7 0>;
GPIO constant
gpios = <&gpio7 7 GPIO_ACTIVE_HIGH>;
> + linux,code = <0x114>;
> + gpio-key,wakeup;
> + };
> +
> + };
> +};
> +
> +&omap3_pmx_wkup {
> + gpio1_pins: pinmux_gpio1_pins {
> + pinctrl-single,pins = <
> + 0x14 0x4104 /* sys_boot5.gpio_7 WAKEUP | INPUT | MODE4 */
Same for pinctrl constant
> + >;
> + };
> };
>
> &omap3_pmx_core {
> @@ -154,3 +174,8 @@
> pinctrl-names = "default";
> pinctrl-0 = <&uart3_pins>;
> };
> +
> +&gpio1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio1_pins>;
> +};
>
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 2/3] ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
@ 2013-06-03 9:40 ` Florian Vaussard
0 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:40 UTC (permalink / raw)
To: linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> Using the gpio-keys bindings, configure the user button on Beagle
> boards. Since the user button is enabled as a wakeup source, also
> ensure the GPIO pin is mux'd correctly and has IO ring wakeups enabled,
> so it can also wakeup from off mode.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/omap3-beagle-xm.dts | 25 +++++++++++++++++++++++++
> arch/arm/boot/dts/omap3-beagle.dts | 25 +++++++++++++++++++++++++
> 2 files changed, 50 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ca067b0..c93c36f 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -57,6 +57,26 @@
> ti,mcbsp = <&mcbsp2>;
> ti,codec = <&twl_audio>;
> };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> +
> + user {
> + label = "user";
> + gpios = <&gpio1 4 0>;
Now you can also use GPIO constants (include/dt-bindings/gpio/gpio.h),
like this:
gpios = <&gpio1 4 GPIO_ACTIVE_HIGH>;
> + linux,code = <0x114>;
> + gpio-key,wakeup;
> + };
> +
> + };
> +};
> +
> +&omap3_pmx_wkup {
> + gpio1_pins: pinmux_gpio1_pins {
> + pinctrl-single,pins = <
> + 0x0e 0x4104 /* sys_boot2.gpio_4 WAKEUP | INPUT | MODE4 */
As for patch 1, you can use pinctrl constants
> + >;
> + };
> };
>
> &i2c1 {
> @@ -140,3 +160,8 @@
> pinctrl-names = "default";
> pinctrl-0 = <&uart3_pins>;
> };
> +
> +&gpio1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio1_pins>;
> +};
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index b004372..c9794cb 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -71,6 +71,26 @@
> reset-supply = <&hsusb2_reset>;
> vcc-supply = <&hsusb2_power>;
> };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> +
> + user {
> + label = "user";
> + gpios = <&gpio1 7 0>;
GPIO constant
gpios = <&gpio7 7 GPIO_ACTIVE_HIGH>;
> + linux,code = <0x114>;
> + gpio-key,wakeup;
> + };
> +
> + };
> +};
> +
> +&omap3_pmx_wkup {
> + gpio1_pins: pinmux_gpio1_pins {
> + pinctrl-single,pins = <
> + 0x14 0x4104 /* sys_boot5.gpio_7 WAKEUP | INPUT | MODE4 */
Same for pinctrl constant
> + >;
> + };
> };
>
> &omap3_pmx_core {
> @@ -154,3 +174,8 @@
> pinctrl-names = "default";
> pinctrl-0 = <&uart3_pins>;
> };
> +
> +&gpio1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&gpio1_pins>;
> +};
>
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-05-31 22:45 ` Kevin Hilman
@ 2013-05-31 22:45 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: Benoit Cousson, linux-omap; +Cc: linux-arm-kernel
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
SYS_NIRQ pin in the twl4030 DTS file.
Boards that hook up the twl4030 differently can easily override this
in their board-specific DTS file.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
index b3034da..c7fe9f6 100644
--- a/arch/arm/boot/dts/twl4030.dtsi
+++ b/arch/arm/boot/dts/twl4030.dtsi
@@ -13,6 +13,8 @@
compatible = "ti,twl4030";
interrupt-controller;
#interrupt-cells = <1>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
rtc {
compatible = "ti,twl4030-rtc";
@@ -92,3 +94,18 @@
#pwm-cells = <2>;
};
};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here. Boards that hook up
+ * the twl4030 differently can easily override this in their
+ * board-specific DTS file.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
+ >;
+ };
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-05-31 22:45 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-05-31 22:45 UTC (permalink / raw)
To: linux-arm-kernel
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
SYS_NIRQ pin in the twl4030 DTS file.
Boards that hook up the twl4030 differently can easily override this
in their board-specific DTS file.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
index b3034da..c7fe9f6 100644
--- a/arch/arm/boot/dts/twl4030.dtsi
+++ b/arch/arm/boot/dts/twl4030.dtsi
@@ -13,6 +13,8 @@
compatible = "ti,twl4030";
interrupt-controller;
#interrupt-cells = <1>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
rtc {
compatible = "ti,twl4030-rtc";
@@ -92,3 +94,18 @@
#pwm-cells = <2>;
};
};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here. Boards that hook up
+ * the twl4030 differently can easily override this in their
+ * board-specific DTS file.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
+ >;
+ };
+};
--
1.8.2
^ permalink raw reply related [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-05-31 22:45 ` Kevin Hilman
@ 2013-06-03 9:48 ` Florian Vaussard
-1 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:48 UTC (permalink / raw)
To: Kevin Hilman; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> SYS_NIRQ pin in the twl4030 DTS file.
>
> Boards that hook up the twl4030 differently can easily override this
> in their board-specific DTS file.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> index b3034da..c7fe9f6 100644
> --- a/arch/arm/boot/dts/twl4030.dtsi
> +++ b/arch/arm/boot/dts/twl4030.dtsi
> @@ -13,6 +13,8 @@
> compatible = "ti,twl4030";
> interrupt-controller;
> #interrupt-cells = <1>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
>
> rtc {
> compatible = "ti,twl4030-rtc";
> @@ -92,3 +94,18 @@
> #pwm-cells = <2>;
> };
> };
> +
> +&omap3_pmx_core {
Here you are making the TWL4030 to explicitly depend on OMAP3. Could
this chip be
used by another target? If the case, I would put this piece of code in
the files
including twl4030. Otherwise it's ok here.
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here. Boards that hook up
> + * the twl4030 differently can easily override this in their
> + * board-specific DTS file.
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
By using pinctrl constants:
0x1b0 (PIN_INPUT_PULLUP | WAKEUP_EN | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
>
I will test this afternoon on an Overo.
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-03 9:48 ` Florian Vaussard
0 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-03 9:48 UTC (permalink / raw)
To: linux-arm-kernel
Hello Kevin,
On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> SYS_NIRQ pin in the twl4030 DTS file.
>
> Boards that hook up the twl4030 differently can easily override this
> in their board-specific DTS file.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> index b3034da..c7fe9f6 100644
> --- a/arch/arm/boot/dts/twl4030.dtsi
> +++ b/arch/arm/boot/dts/twl4030.dtsi
> @@ -13,6 +13,8 @@
> compatible = "ti,twl4030";
> interrupt-controller;
> #interrupt-cells = <1>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
>
> rtc {
> compatible = "ti,twl4030-rtc";
> @@ -92,3 +94,18 @@
> #pwm-cells = <2>;
> };
> };
> +
> +&omap3_pmx_core {
Here you are making the TWL4030 to explicitly depend on OMAP3. Could
this chip be
used by another target? If the case, I would put this piece of code in
the files
including twl4030. Otherwise it's ok here.
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here. Boards that hook up
> + * the twl4030 differently can easily override this in their
> + * board-specific DTS file.
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
By using pinctrl constants:
0x1b0 (PIN_INPUT_PULLUP | WAKEUP_EN | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
>
I will test this afternoon on an Overo.
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-03 9:48 ` Florian Vaussard
@ 2013-06-03 22:19 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-03 22:19 UTC (permalink / raw)
To: florian.vaussard; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
Florian Vaussard <florian.vaussard@epfl.ch> writes:
> Hello Kevin,
>
> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>> SYS_NIRQ pin in the twl4030 DTS file.
>>
>> Boards that hook up the twl4030 differently can easily override this
>> in their board-specific DTS file.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>> 1 file changed, 17 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>> index b3034da..c7fe9f6 100644
>> --- a/arch/arm/boot/dts/twl4030.dtsi
>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>> @@ -13,6 +13,8 @@
>> compatible = "ti,twl4030";
>> interrupt-controller;
>> #interrupt-cells = <1>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&twl4030_pins>;
>>
>> rtc {
>> compatible = "ti,twl4030-rtc";
>> @@ -92,3 +94,18 @@
>> #pwm-cells = <2>;
>> };
>> };
>> +
>> +&omap3_pmx_core {
>
> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> this chip be used by another target? If the case, I would put this
> piece of code in the files including twl4030. Otherwise it's ok here.
I talked this over with Tony before posting. In theorry, it could be
used on other platforms. In practice, it's only used on OMAP. We can
easily move this to board files if needed later, but thought it would be
simpler to put it here. I'll let Benoit decide.
>> + /*
>> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
>> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
>> + * defaults for the SYS_NIRQ pin here. Boards that hook up
>> + * the twl4030 differently can easily override this in their
>> + * board-specific DTS file.
>> + */
>> + twl4030_pins: pinmux_twl4030_pins {
>> + pinctrl-single,pins = <
>> + 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
>
> By using pinctrl constants:
>
> 0x1b0 (PIN_INPUT_PULLUP | WAKEUP_EN | MUX_MODE0) /* sys_nirq.sys_nirq */
Yup.
>> + >;
>> + };
>> +};
>>
>
> I will test this afternoon on an Overo.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-03 22:19 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-03 22:19 UTC (permalink / raw)
To: linux-arm-kernel
Florian Vaussard <florian.vaussard@epfl.ch> writes:
> Hello Kevin,
>
> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>> SYS_NIRQ pin in the twl4030 DTS file.
>>
>> Boards that hook up the twl4030 differently can easily override this
>> in their board-specific DTS file.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>> 1 file changed, 17 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>> index b3034da..c7fe9f6 100644
>> --- a/arch/arm/boot/dts/twl4030.dtsi
>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>> @@ -13,6 +13,8 @@
>> compatible = "ti,twl4030";
>> interrupt-controller;
>> #interrupt-cells = <1>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&twl4030_pins>;
>>
>> rtc {
>> compatible = "ti,twl4030-rtc";
>> @@ -92,3 +94,18 @@
>> #pwm-cells = <2>;
>> };
>> };
>> +
>> +&omap3_pmx_core {
>
> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> this chip be used by another target? If the case, I would put this
> piece of code in the files including twl4030. Otherwise it's ok here.
I talked this over with Tony before posting. In theorry, it could be
used on other platforms. In practice, it's only used on OMAP. We can
easily move this to board files if needed later, but thought it would be
simpler to put it here. I'll let Benoit decide.
>> + /*
>> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
>> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
>> + * defaults for the SYS_NIRQ pin here. Boards that hook up
>> + * the twl4030 differently can easily override this in their
>> + * board-specific DTS file.
>> + */
>> + twl4030_pins: pinmux_twl4030_pins {
>> + pinctrl-single,pins = <
>> + 0x1b0 0x4118 /* sys_nirq.sys_nirq WAKEUP | INPUT_PULLUP | MODE 0 */
>
> By using pinctrl constants:
>
> 0x1b0 (PIN_INPUT_PULLUP | WAKEUP_EN | MUX_MODE0) /* sys_nirq.sys_nirq */
Yup.
>> + >;
>> + };
>> +};
>>
>
> I will test this afternoon on an Overo.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-03 22:19 ` Kevin Hilman
@ 2013-06-03 22:39 ` Jason Cooper
-1 siblings, 0 replies; 52+ messages in thread
From: Jason Cooper @ 2013-06-03 22:39 UTC (permalink / raw)
To: Kevin Hilman
Cc: florian.vaussard, linux-omap, Benoit Cousson, linux-arm-kernel
On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
> > Hello Kevin,
> >
> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> >> SYS_NIRQ pin in the twl4030 DTS file.
> >>
> >> Boards that hook up the twl4030 differently can easily override this
> >> in their board-specific DTS file.
> >>
> >> This allows RTC wake from off-mode to work again on OMAP3-based
> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> >> 3530/Overo, 3730/Overo-STORM.
> >>
> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> >> ---
> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> >> 1 file changed, 17 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> >> index b3034da..c7fe9f6 100644
> >> --- a/arch/arm/boot/dts/twl4030.dtsi
> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
> >> @@ -13,6 +13,8 @@
> >> compatible = "ti,twl4030";
> >> interrupt-controller;
> >> #interrupt-cells = <1>;
> >> + pinctrl-names = "default";
> >> + pinctrl-0 = <&twl4030_pins>;
> >>
> >> rtc {
> >> compatible = "ti,twl4030-rtc";
> >> @@ -92,3 +94,18 @@
> >> #pwm-cells = <2>;
> >> };
> >> };
> >> +
> >> +&omap3_pmx_core {
> >
> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> > this chip be used by another target? If the case, I would put this
> > piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
What about omap boards that don't use the twl4030? eg I'm currently
tinkering with a board that uses a tps6507x...
thx,
Jason.
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-03 22:39 ` Jason Cooper
0 siblings, 0 replies; 52+ messages in thread
From: Jason Cooper @ 2013-06-03 22:39 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
> > Hello Kevin,
> >
> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> >> SYS_NIRQ pin in the twl4030 DTS file.
> >>
> >> Boards that hook up the twl4030 differently can easily override this
> >> in their board-specific DTS file.
> >>
> >> This allows RTC wake from off-mode to work again on OMAP3-based
> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> >> 3530/Overo, 3730/Overo-STORM.
> >>
> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> >> ---
> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> >> 1 file changed, 17 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> >> index b3034da..c7fe9f6 100644
> >> --- a/arch/arm/boot/dts/twl4030.dtsi
> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
> >> @@ -13,6 +13,8 @@
> >> compatible = "ti,twl4030";
> >> interrupt-controller;
> >> #interrupt-cells = <1>;
> >> + pinctrl-names = "default";
> >> + pinctrl-0 = <&twl4030_pins>;
> >>
> >> rtc {
> >> compatible = "ti,twl4030-rtc";
> >> @@ -92,3 +94,18 @@
> >> #pwm-cells = <2>;
> >> };
> >> };
> >> +
> >> +&omap3_pmx_core {
> >
> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> > this chip be used by another target? If the case, I would put this
> > piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
What about omap boards that don't use the twl4030? eg I'm currently
tinkering with a board that uses a tps6507x...
thx,
Jason.
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-03 22:39 ` Jason Cooper
@ 2013-06-04 17:58 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-04 17:58 UTC (permalink / raw)
To: Jason Cooper
Cc: florian.vaussard, linux-omap, Benoit Cousson, linux-arm-kernel
Jason Cooper <jason@lakedaemon.net> writes:
> On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
>> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>>
>> > Hello Kevin,
>> >
>> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>> >> SYS_NIRQ pin in the twl4030 DTS file.
>> >>
>> >> Boards that hook up the twl4030 differently can easily override this
>> >> in their board-specific DTS file.
>> >>
>> >> This allows RTC wake from off-mode to work again on OMAP3-based
>> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> >> 3530/Overo, 3730/Overo-STORM.
>> >>
>> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> >> ---
>> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>> >> 1 file changed, 17 insertions(+)
>> >>
>> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>> >> index b3034da..c7fe9f6 100644
>> >> --- a/arch/arm/boot/dts/twl4030.dtsi
>> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
>> >> @@ -13,6 +13,8 @@
>> >> compatible = "ti,twl4030";
>> >> interrupt-controller;
>> >> #interrupt-cells = <1>;
>> >> + pinctrl-names = "default";
>> >> + pinctrl-0 = <&twl4030_pins>;
>> >>
>> >> rtc {
>> >> compatible = "ti,twl4030-rtc";
>> >> @@ -92,3 +94,18 @@
>> >> #pwm-cells = <2>;
>> >> };
>> >> };
>> >> +
>> >> +&omap3_pmx_core {
>> >
>> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> > this chip be used by another target? If the case, I would put this
>> > piece of code in the files including twl4030. Otherwise it's ok here.
>>
>> I talked this over with Tony before posting. In theorry, it could be
>> used on other platforms. In practice, it's only used on OMAP. We can
>> easily move this to board files if needed later, but thought it would be
>> simpler to put it here. I'll let Benoit decide.
>
> What about omap boards that don't use the twl4030? eg I'm currently
> tinkering with a board that uses a tps6507x...
Then you'll be inclding tps6507x.dtsi instead of twl4030.dtsi and won't
be affected by this patch at all.
Your board file should setup the muxing (using this as an example) for
how your board wires the TPS chip to the OMAP.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-04 17:58 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-04 17:58 UTC (permalink / raw)
To: linux-arm-kernel
Jason Cooper <jason@lakedaemon.net> writes:
> On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
>> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>>
>> > Hello Kevin,
>> >
>> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>> >> SYS_NIRQ pin in the twl4030 DTS file.
>> >>
>> >> Boards that hook up the twl4030 differently can easily override this
>> >> in their board-specific DTS file.
>> >>
>> >> This allows RTC wake from off-mode to work again on OMAP3-based
>> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> >> 3530/Overo, 3730/Overo-STORM.
>> >>
>> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> >> ---
>> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>> >> 1 file changed, 17 insertions(+)
>> >>
>> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>> >> index b3034da..c7fe9f6 100644
>> >> --- a/arch/arm/boot/dts/twl4030.dtsi
>> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
>> >> @@ -13,6 +13,8 @@
>> >> compatible = "ti,twl4030";
>> >> interrupt-controller;
>> >> #interrupt-cells = <1>;
>> >> + pinctrl-names = "default";
>> >> + pinctrl-0 = <&twl4030_pins>;
>> >>
>> >> rtc {
>> >> compatible = "ti,twl4030-rtc";
>> >> @@ -92,3 +94,18 @@
>> >> #pwm-cells = <2>;
>> >> };
>> >> };
>> >> +
>> >> +&omap3_pmx_core {
>> >
>> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> > this chip be used by another target? If the case, I would put this
>> > piece of code in the files including twl4030. Otherwise it's ok here.
>>
>> I talked this over with Tony before posting. In theorry, it could be
>> used on other platforms. In practice, it's only used on OMAP. We can
>> easily move this to board files if needed later, but thought it would be
>> simpler to put it here. I'll let Benoit decide.
>
> What about omap boards that don't use the twl4030? eg I'm currently
> tinkering with a board that uses a tps6507x...
Then you'll be inclding tps6507x.dtsi instead of twl4030.dtsi and won't
be affected by this patch at all.
Your board file should setup the muxing (using this as an example) for
how your board wires the TPS chip to the OMAP.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-04 17:58 ` Kevin Hilman
@ 2013-06-04 18:05 ` Jason Cooper
-1 siblings, 0 replies; 52+ messages in thread
From: Jason Cooper @ 2013-06-04 18:05 UTC (permalink / raw)
To: Kevin Hilman
Cc: florian.vaussard, linux-omap, Benoit Cousson, linux-arm-kernel
On Tue, Jun 04, 2013 at 10:58:46AM -0700, Kevin Hilman wrote:
> Jason Cooper <jason@lakedaemon.net> writes:
>
> > On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
> >> Florian Vaussard <florian.vaussard@epfl.ch> writes:
> >>
> >> > Hello Kevin,
> >> >
> >> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> >> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> >> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> >> >> SYS_NIRQ pin in the twl4030 DTS file.
> >> >>
> >> >> Boards that hook up the twl4030 differently can easily override this
> >> >> in their board-specific DTS file.
> >> >>
> >> >> This allows RTC wake from off-mode to work again on OMAP3-based
> >> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> >> >> 3530/Overo, 3730/Overo-STORM.
> >> >>
> >> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> >> >> ---
> >> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> >> >> 1 file changed, 17 insertions(+)
> >> >>
> >> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> >> >> index b3034da..c7fe9f6 100644
> >> >> --- a/arch/arm/boot/dts/twl4030.dtsi
> >> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
> >> >> @@ -13,6 +13,8 @@
> >> >> compatible = "ti,twl4030";
> >> >> interrupt-controller;
> >> >> #interrupt-cells = <1>;
> >> >> + pinctrl-names = "default";
> >> >> + pinctrl-0 = <&twl4030_pins>;
> >> >>
> >> >> rtc {
> >> >> compatible = "ti,twl4030-rtc";
> >> >> @@ -92,3 +94,18 @@
> >> >> #pwm-cells = <2>;
> >> >> };
> >> >> };
> >> >> +
> >> >> +&omap3_pmx_core {
> >> >
> >> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> >> > this chip be used by another target? If the case, I would put this
> >> > piece of code in the files including twl4030. Otherwise it's ok here.
> >>
> >> I talked this over with Tony before posting. In theorry, it could be
> >> used on other platforms. In practice, it's only used on OMAP. We can
> >> easily move this to board files if needed later, but thought it would be
> >> simpler to put it here. I'll let Benoit decide.
> >
> > What about omap boards that don't use the twl4030? eg I'm currently
> > tinkering with a board that uses a tps6507x...
>
> Then you'll be inclding tps6507x.dtsi instead of twl4030.dtsi and won't
> be affected by this patch at all.
Gah! I saw omap3_pmx_core being modified and missed that the mod was in
twl4030.dtsi. Of course, you are right.
> Your board file should setup the muxing (using this as an example) for
> how your board wires the TPS chip to the OMAP.
Yes, that's the fun part, I'm trying to get a vanilla kernel to boot on
the Seagate Wireless Plus. They didn't exactly hand out schematics ;-)
thx,
Jason.
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-04 18:05 ` Jason Cooper
0 siblings, 0 replies; 52+ messages in thread
From: Jason Cooper @ 2013-06-04 18:05 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jun 04, 2013 at 10:58:46AM -0700, Kevin Hilman wrote:
> Jason Cooper <jason@lakedaemon.net> writes:
>
> > On Mon, Jun 03, 2013 at 03:19:25PM -0700, Kevin Hilman wrote:
> >> Florian Vaussard <florian.vaussard@epfl.ch> writes:
> >>
> >> > Hello Kevin,
> >> >
> >> > On 06/01/2013 12:45 AM, Kevin Hilman wrote:
> >> >> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> >> >> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
> >> >> SYS_NIRQ pin in the twl4030 DTS file.
> >> >>
> >> >> Boards that hook up the twl4030 differently can easily override this
> >> >> in their board-specific DTS file.
> >> >>
> >> >> This allows RTC wake from off-mode to work again on OMAP3-based
> >> >> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> >> >> 3530/Overo, 3730/Overo-STORM.
> >> >>
> >> >> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> >> >> ---
> >> >> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
> >> >> 1 file changed, 17 insertions(+)
> >> >>
> >> >> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
> >> >> index b3034da..c7fe9f6 100644
> >> >> --- a/arch/arm/boot/dts/twl4030.dtsi
> >> >> +++ b/arch/arm/boot/dts/twl4030.dtsi
> >> >> @@ -13,6 +13,8 @@
> >> >> compatible = "ti,twl4030";
> >> >> interrupt-controller;
> >> >> #interrupt-cells = <1>;
> >> >> + pinctrl-names = "default";
> >> >> + pinctrl-0 = <&twl4030_pins>;
> >> >>
> >> >> rtc {
> >> >> compatible = "ti,twl4030-rtc";
> >> >> @@ -92,3 +94,18 @@
> >> >> #pwm-cells = <2>;
> >> >> };
> >> >> };
> >> >> +
> >> >> +&omap3_pmx_core {
> >> >
> >> > Here you are making the TWL4030 to explicitly depend on OMAP3. Could
> >> > this chip be used by another target? If the case, I would put this
> >> > piece of code in the files including twl4030. Otherwise it's ok here.
> >>
> >> I talked this over with Tony before posting. In theorry, it could be
> >> used on other platforms. In practice, it's only used on OMAP. We can
> >> easily move this to board files if needed later, but thought it would be
> >> simpler to put it here. I'll let Benoit decide.
> >
> > What about omap boards that don't use the twl4030? eg I'm currently
> > tinkering with a board that uses a tps6507x...
>
> Then you'll be inclding tps6507x.dtsi instead of twl4030.dtsi and won't
> be affected by this patch at all.
Gah! I saw omap3_pmx_core being modified and missed that the mod was in
twl4030.dtsi. Of course, you are right.
> Your board file should setup the muxing (using this as an example) for
> how your board wires the TPS chip to the OMAP.
Yes, that's the fun part, I'm trying to get a vanilla kernel to boot on
the Seagate Wireless Plus. They didn't exactly hand out schematics ;-)
thx,
Jason.
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-03 22:19 ` Kevin Hilman
@ 2013-06-05 7:41 ` Florian Vaussard
-1 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-05 7:41 UTC (permalink / raw)
To: Kevin Hilman; +Cc: Benoit Cousson, linux-omap, linux-arm-kernel
On 06/04/2013 12:19 AM, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
>> Hello Kevin,
>>
>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>
>>> Boards that hook up the twl4030 differently can easily override this
>>> in their board-specific DTS file.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> ---
>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>> 1 file changed, 17 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>> index b3034da..c7fe9f6 100644
>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>> @@ -13,6 +13,8 @@
>>> compatible = "ti,twl4030";
>>> interrupt-controller;
>>> #interrupt-cells = <1>;
>>> + pinctrl-names = "default";
>>> + pinctrl-0 = <&twl4030_pins>;
>>>
>>> rtc {
>>> compatible = "ti,twl4030-rtc";
>>> @@ -92,3 +94,18 @@
>>> #pwm-cells = <2>;
>>> };
>>> };
>>> +
>>> +&omap3_pmx_core {
>>
>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> this chip be used by another target? If the case, I would put this
>> piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
>
Ok, so I am fine with your approach. If need arises later, we can have
a .dtsi file in-between to make the glue layer and avoid duplication inside
board files.
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-05 7:41 ` Florian Vaussard
0 siblings, 0 replies; 52+ messages in thread
From: Florian Vaussard @ 2013-06-05 7:41 UTC (permalink / raw)
To: linux-arm-kernel
On 06/04/2013 12:19 AM, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
>> Hello Kevin,
>>
>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>
>>> Boards that hook up the twl4030 differently can easily override this
>>> in their board-specific DTS file.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> ---
>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>> 1 file changed, 17 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>> index b3034da..c7fe9f6 100644
>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>> @@ -13,6 +13,8 @@
>>> compatible = "ti,twl4030";
>>> interrupt-controller;
>>> #interrupt-cells = <1>;
>>> + pinctrl-names = "default";
>>> + pinctrl-0 = <&twl4030_pins>;
>>>
>>> rtc {
>>> compatible = "ti,twl4030-rtc";
>>> @@ -92,3 +94,18 @@
>>> #pwm-cells = <2>;
>>> };
>>> };
>>> +
>>> +&omap3_pmx_core {
>>
>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> this chip be used by another target? If the case, I would put this
>> piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
>
Ok, so I am fine with your approach. If need arises later, we can have
a .dtsi file in-between to make the glue layer and avoid duplication inside
board files.
Regards,
Florian
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-03 22:19 ` Kevin Hilman
@ 2013-06-05 13:52 ` Benoit Cousson
-1 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-05 13:52 UTC (permalink / raw)
To: Kevin Hilman; +Cc: florian.vaussard, linux-omap, linux-arm-kernel
Salut Kevin,
On 06/04/2013 12:19 AM, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
>> Hello Kevin,
>>
>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>
>>> Boards that hook up the twl4030 differently can easily override this
>>> in their board-specific DTS file.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> ---
>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>> 1 file changed, 17 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>> index b3034da..c7fe9f6 100644
>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>> @@ -13,6 +13,8 @@
>>> compatible = "ti,twl4030";
>>> interrupt-controller;
>>> #interrupt-cells = <1>;
>>> + pinctrl-names = "default";
>>> + pinctrl-0 = <&twl4030_pins>;
>>>
>>> rtc {
>>> compatible = "ti,twl4030-rtc";
>>> @@ -92,3 +94,18 @@
>>> #pwm-cells = <2>;
>>> };
>>> };
>>> +
>>> +&omap3_pmx_core {
>>
>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> this chip be used by another target? If the case, I would put this
>> piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
Yeah, I don't like that either. Some twl variants are used on other
platform. And these variants might include the original twl4030 as a
base. None of them are in mainline for sure, but anyway adding SoC stuff
in external IC file is not a good practice.
I don't want people in the future using that as an excuse for similar hacks.
The easiest way is to create an extra file including the platform free
twl4030 file and adding the OMAP3 specific config on top of that.
Every OMAP3 boards will the used that file. Otherwise you can hack each
boards.
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-05 13:52 ` Benoit Cousson
0 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-05 13:52 UTC (permalink / raw)
To: linux-arm-kernel
Salut Kevin,
On 06/04/2013 12:19 AM, Kevin Hilman wrote:
> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>
>> Hello Kevin,
>>
>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>
>>> Boards that hook up the twl4030 differently can easily override this
>>> in their board-specific DTS file.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> ---
>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>> 1 file changed, 17 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>> index b3034da..c7fe9f6 100644
>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>> @@ -13,6 +13,8 @@
>>> compatible = "ti,twl4030";
>>> interrupt-controller;
>>> #interrupt-cells = <1>;
>>> + pinctrl-names = "default";
>>> + pinctrl-0 = <&twl4030_pins>;
>>>
>>> rtc {
>>> compatible = "ti,twl4030-rtc";
>>> @@ -92,3 +94,18 @@
>>> #pwm-cells = <2>;
>>> };
>>> };
>>> +
>>> +&omap3_pmx_core {
>>
>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>> this chip be used by another target? If the case, I would put this
>> piece of code in the files including twl4030. Otherwise it's ok here.
>
> I talked this over with Tony before posting. In theorry, it could be
> used on other platforms. In practice, it's only used on OMAP. We can
> easily move this to board files if needed later, but thought it would be
> simpler to put it here. I'll let Benoit decide.
Yeah, I don't like that either. Some twl variants are used on other
platform. And these variants might include the original twl4030 as a
base. None of them are in mainline for sure, but anyway adding SoC stuff
in external IC file is not a good practice.
I don't want people in the future using that as an excuse for similar hacks.
The easiest way is to create an extra file including the platform free
twl4030 file and adding the OMAP3 specific config on top of that.
Every OMAP3 boards will the used that file. Otherwise you can hack each
boards.
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-05 13:52 ` Benoit Cousson
@ 2013-06-05 14:26 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-05 14:26 UTC (permalink / raw)
To: Benoit Cousson; +Cc: florian.vaussard, linux-omap, linux-arm-kernel
Benoit Cousson <b-cousson@ti.com> writes:
> Salut Kevin,
>
> On 06/04/2013 12:19 AM, Kevin Hilman wrote:
>> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>>
>>> Hello Kevin,
>>>
>>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>>
>>>> Boards that hook up the twl4030 differently can easily override this
>>>> in their board-specific DTS file.
>>>>
>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>> 3530/Overo, 3730/Overo-STORM.
>>>>
>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>> ---
>>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>>> 1 file changed, 17 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>>> index b3034da..c7fe9f6 100644
>>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>>> @@ -13,6 +13,8 @@
>>>> compatible = "ti,twl4030";
>>>> interrupt-controller;
>>>> #interrupt-cells = <1>;
>>>> + pinctrl-names = "default";
>>>> + pinctrl-0 = <&twl4030_pins>;
>>>>
>>>> rtc {
>>>> compatible = "ti,twl4030-rtc";
>>>> @@ -92,3 +94,18 @@
>>>> #pwm-cells = <2>;
>>>> };
>>>> };
>>>> +
>>>> +&omap3_pmx_core {
>>>
>>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>>> this chip be used by another target? If the case, I would put this
>>> piece of code in the files including twl4030. Otherwise it's ok here.
>>
>> I talked this over with Tony before posting. In theorry, it could be
>> used on other platforms. In practice, it's only used on OMAP. We can
>> easily move this to board files if needed later, but thought it would be
>> simpler to put it here. I'll let Benoit decide.
>
> Yeah, I don't like that either. Some twl variants are used on other
> platform. And these variants might include the original twl4030 as a
> base. None of them are in mainline for sure, but anyway adding SoC stuff
> in external IC file is not a good practice.
>
> I don't want people in the future using that as an excuse for similar hacks.
>
> The easiest way is to create an extra file including the platform free
> twl4030 file and adding the OMAP3 specific config on top of that.
> Every OMAP3 boards will the used that file. Otherwise you can hack each
> boards.
Yeah, somehow I knew you would say that. I'll respin.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-05 14:26 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-05 14:26 UTC (permalink / raw)
To: linux-arm-kernel
Benoit Cousson <b-cousson@ti.com> writes:
> Salut Kevin,
>
> On 06/04/2013 12:19 AM, Kevin Hilman wrote:
>> Florian Vaussard <florian.vaussard@epfl.ch> writes:
>>
>>> Hello Kevin,
>>>
>>> On 06/01/2013 12:45 AM, Kevin Hilman wrote:
>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>> SYS_NIRQ line on OMAP. Therefore, configure the defaults for the
>>>> SYS_NIRQ pin in the twl4030 DTS file.
>>>>
>>>> Boards that hook up the twl4030 differently can easily override this
>>>> in their board-specific DTS file.
>>>>
>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>> 3530/Overo, 3730/Overo-STORM.
>>>>
>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>> ---
>>>> arch/arm/boot/dts/twl4030.dtsi | 17 +++++++++++++++++
>>>> 1 file changed, 17 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi
>>>> index b3034da..c7fe9f6 100644
>>>> --- a/arch/arm/boot/dts/twl4030.dtsi
>>>> +++ b/arch/arm/boot/dts/twl4030.dtsi
>>>> @@ -13,6 +13,8 @@
>>>> compatible = "ti,twl4030";
>>>> interrupt-controller;
>>>> #interrupt-cells = <1>;
>>>> + pinctrl-names = "default";
>>>> + pinctrl-0 = <&twl4030_pins>;
>>>>
>>>> rtc {
>>>> compatible = "ti,twl4030-rtc";
>>>> @@ -92,3 +94,18 @@
>>>> #pwm-cells = <2>;
>>>> };
>>>> };
>>>> +
>>>> +&omap3_pmx_core {
>>>
>>> Here you are making the TWL4030 to explicitly depend on OMAP3. Could
>>> this chip be used by another target? If the case, I would put this
>>> piece of code in the files including twl4030. Otherwise it's ok here.
>>
>> I talked this over with Tony before posting. In theorry, it could be
>> used on other platforms. In practice, it's only used on OMAP. We can
>> easily move this to board files if needed later, but thought it would be
>> simpler to put it here. I'll let Benoit decide.
>
> Yeah, I don't like that either. Some twl variants are used on other
> platform. And these variants might include the original twl4030 as a
> base. None of them are in mainline for sure, but anyway adding SoC stuff
> in external IC file is not a good practice.
>
> I don't want people in the future using that as an excuse for similar hacks.
>
> The easiest way is to create an extra file including the platform free
> twl4030 file and adding the OMAP3 specific config on top of that.
> Every OMAP3 boards will the used that file. Otherwise you can hack each
> boards.
Yeah, somehow I knew you would say that. I'll respin.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-05-31 22:45 ` Kevin Hilman
@ 2013-06-06 23:27 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-06 23:27 UTC (permalink / raw)
To: Benoit Cousson, linux-omap; +Cc: linux-arm-kernel, Florian Vaussard
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Add another DTS include file
(twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
to include.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Special thanks to Florian Vaussard for suggesting use of preprocessor
feature.
Cc: Florian Vaussard <florian.vaussard@epfl.ch>
Cc: Benoit Cousson <b-cousson@ti.com>
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
v2: Updated to change board files instead of twl4030 file.
arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
arch/arm/boot/dts/omap3-beagle.dts | 1 +
arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
arch/arm/boot/dts/omap3-evm.dts | 1 +
arch/arm/boot/dts/omap3-igep.dtsi | 1 +
arch/arm/boot/dts/omap3-overo.dtsi | 1 +
arch/arm/boot/dts/omap3430-sdp.dts | 1 +
arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
8 files changed, 24 insertions(+)
create mode 100644 arch/arm/boot/dts/twl4030_omap3_mux.dtsi
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ad17b6b..1afb563 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -96,6 +96,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index eb7e02a..d5810d7 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -135,6 +135,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
index 5be71b1..166b90c 100644
--- a/arch/arm/boot/dts/omap3-devkit8000.dts
+++ b/arch/arm/boot/dts/omap3-devkit8000.dts
@@ -81,6 +81,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
index baa24bb..ad7ab86 100644
--- a/arch/arm/boot/dts/omap3-evm.dts
+++ b/arch/arm/boot/dts/omap3-evm.dts
@@ -45,6 +45,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
index 5224c29..3107dc3 100644
--- a/arch/arm/boot/dts/omap3-igep.dtsi
+++ b/arch/arm/boot/dts/omap3-igep.dtsi
@@ -87,6 +87,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index 144a5a2..bb55907 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -50,6 +50,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
/* i2c2 pins are used for gpio */
&i2c2 {
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index 2a725a0..df3b918 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -29,6 +29,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
new file mode 100644
index 0000000..1bd0f9f
--- /dev/null
+++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
@@ -0,0 +1,17 @@
+&twl {
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
+};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
+ >;
+ };
+};
--
1.8.3
^ permalink raw reply related [flat|nested] 52+ messages in thread* [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-06 23:27 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-06 23:27 UTC (permalink / raw)
To: linux-arm-kernel
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Add another DTS include file
(twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
to include.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Special thanks to Florian Vaussard for suggesting use of preprocessor
feature.
Cc: Florian Vaussard <florian.vaussard@epfl.ch>
Cc: Benoit Cousson <b-cousson@ti.com>
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
v2: Updated to change board files instead of twl4030 file.
arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
arch/arm/boot/dts/omap3-beagle.dts | 1 +
arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
arch/arm/boot/dts/omap3-evm.dts | 1 +
arch/arm/boot/dts/omap3-igep.dtsi | 1 +
arch/arm/boot/dts/omap3-overo.dtsi | 1 +
arch/arm/boot/dts/omap3430-sdp.dts | 1 +
arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
8 files changed, 24 insertions(+)
create mode 100644 arch/arm/boot/dts/twl4030_omap3_mux.dtsi
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ad17b6b..1afb563 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -96,6 +96,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index eb7e02a..d5810d7 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -135,6 +135,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
index 5be71b1..166b90c 100644
--- a/arch/arm/boot/dts/omap3-devkit8000.dts
+++ b/arch/arm/boot/dts/omap3-devkit8000.dts
@@ -81,6 +81,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
index baa24bb..ad7ab86 100644
--- a/arch/arm/boot/dts/omap3-evm.dts
+++ b/arch/arm/boot/dts/omap3-evm.dts
@@ -45,6 +45,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
index 5224c29..3107dc3 100644
--- a/arch/arm/boot/dts/omap3-igep.dtsi
+++ b/arch/arm/boot/dts/omap3-igep.dtsi
@@ -87,6 +87,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index 144a5a2..bb55907 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -50,6 +50,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
/* i2c2 pins are used for gpio */
&i2c2 {
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index 2a725a0..df3b918 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -29,6 +29,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3_mux.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
new file mode 100644
index 0000000..1bd0f9f
--- /dev/null
+++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
@@ -0,0 +1,17 @@
+&twl {
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
+};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
+ >;
+ };
+};
--
1.8.3
^ permalink raw reply related [flat|nested] 52+ messages in thread* Re: [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-06 23:27 ` Kevin Hilman
@ 2013-06-06 23:42 ` Nishanth Menon
-1 siblings, 0 replies; 52+ messages in thread
From: Nishanth Menon @ 2013-06-06 23:42 UTC (permalink / raw)
To: Kevin Hilman
Cc: Benoit Cousson, linux-omap, linux-arm-kernel, Florian Vaussard
On 16:27-20130606, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Add another DTS include file
> (twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
> to include.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Special thanks to Florian Vaussard for suggesting use of preprocessor
> feature.
>
> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
> Cc: Benoit Cousson <b-cousson@ti.com>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> v2: Updated to change board files instead of twl4030 file.
>
> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
> arch/arm/boot/dts/omap3-beagle.dts | 1 +
> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
> arch/arm/boot/dts/omap3-evm.dts | 1 +
> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
> arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
minor comment:
can we do twl4030_omap3 instead? That way, I can add in twl4030 VP
specific parameters which are OMAP3 specific?
one additional minor comment below:
> 8 files changed, 24 insertions(+)
> create mode 100644 arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ad17b6b..1afb563 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -96,6 +96,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index eb7e02a..d5810d7 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -135,6 +135,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
> index 5be71b1..166b90c 100644
> --- a/arch/arm/boot/dts/omap3-devkit8000.dts
> +++ b/arch/arm/boot/dts/omap3-devkit8000.dts
> @@ -81,6 +81,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
> index baa24bb..ad7ab86 100644
> --- a/arch/arm/boot/dts/omap3-evm.dts
> +++ b/arch/arm/boot/dts/omap3-evm.dts
> @@ -45,6 +45,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
> index 5224c29..3107dc3 100644
> --- a/arch/arm/boot/dts/omap3-igep.dtsi
> +++ b/arch/arm/boot/dts/omap3-igep.dtsi
> @@ -87,6 +87,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index 144a5a2..bb55907 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -50,6 +50,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> /* i2c2 pins are used for gpio */
> &i2c2 {
> diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
> index 2a725a0..df3b918 100644
> --- a/arch/arm/boot/dts/omap3430-sdp.dts
> +++ b/arch/arm/boot/dts/omap3430-sdp.dts
> @@ -29,6 +29,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
> new file mode 100644
> index 0000000..1bd0f9f
> --- /dev/null
> +++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
> @@ -0,0 +1,17 @@
some license text?
> +&twl {
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
> +};
> +
> +&omap3_pmx_core {
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here.
^^^ tab?
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-06 23:42 ` Nishanth Menon
0 siblings, 0 replies; 52+ messages in thread
From: Nishanth Menon @ 2013-06-06 23:42 UTC (permalink / raw)
To: linux-arm-kernel
On 16:27-20130606, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Add another DTS include file
> (twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
> to include.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Special thanks to Florian Vaussard for suggesting use of preprocessor
> feature.
>
> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
> Cc: Benoit Cousson <b-cousson@ti.com>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> ---
> v2: Updated to change board files instead of twl4030 file.
>
> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
> arch/arm/boot/dts/omap3-beagle.dts | 1 +
> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
> arch/arm/boot/dts/omap3-evm.dts | 1 +
> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
> arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
minor comment:
can we do twl4030_omap3 instead? That way, I can add in twl4030 VP
specific parameters which are OMAP3 specific?
one additional minor comment below:
> 8 files changed, 24 insertions(+)
> create mode 100644 arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ad17b6b..1afb563 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -96,6 +96,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index eb7e02a..d5810d7 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -135,6 +135,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
> index 5be71b1..166b90c 100644
> --- a/arch/arm/boot/dts/omap3-devkit8000.dts
> +++ b/arch/arm/boot/dts/omap3-devkit8000.dts
> @@ -81,6 +81,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
> index baa24bb..ad7ab86 100644
> --- a/arch/arm/boot/dts/omap3-evm.dts
> +++ b/arch/arm/boot/dts/omap3-evm.dts
> @@ -45,6 +45,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
> index 5224c29..3107dc3 100644
> --- a/arch/arm/boot/dts/omap3-igep.dtsi
> +++ b/arch/arm/boot/dts/omap3-igep.dtsi
> @@ -87,6 +87,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index 144a5a2..bb55907 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -50,6 +50,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> /* i2c2 pins are used for gpio */
> &i2c2 {
> diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
> index 2a725a0..df3b918 100644
> --- a/arch/arm/boot/dts/omap3430-sdp.dts
> +++ b/arch/arm/boot/dts/omap3430-sdp.dts
> @@ -29,6 +29,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3_mux.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
> new file mode 100644
> index 0000000..1bd0f9f
> --- /dev/null
> +++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
> @@ -0,0 +1,17 @@
some license text?
> +&twl {
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
> +};
> +
> +&omap3_pmx_core {
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here.
^^^ tab?
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-06 23:42 ` Nishanth Menon
@ 2013-06-07 16:59 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-07 16:59 UTC (permalink / raw)
To: Nishanth Menon
Cc: Benoit Cousson, linux-omap, linux-arm-kernel, Florian Vaussard
Nishanth Menon <nm@ti.com> writes:
> On 16:27-20130606, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Add another DTS include file
>> (twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
>> to include.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>> feature.
>>
>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>> Cc: Benoit Cousson <b-cousson@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> v2: Updated to change board files instead of twl4030 file.
>>
>> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
>> arch/arm/boot/dts/omap3-beagle.dts | 1 +
>> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
>> arch/arm/boot/dts/omap3-evm.dts | 1 +
>> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
>> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
>> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
>> arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
> minor comment:
> can we do twl4030_omap3 instead? That way, I can add in twl4030 VP
> specific parameters which are OMAP3 specific?
Good idea, I'll rename it to twl4030_omap3.dtsi
[...]
>> diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>> new file mode 100644
>> index 0000000..1bd0f9f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>> @@ -0,0 +1,17 @@
> some license text?
oops
>> +&twl {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&twl4030_pins>;
>> +};
>> +
>> +&omap3_pmx_core {
>> + /*
>> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
>> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
>> + * defaults for the SYS_NIRQ pin here.
> ^^^ tab?
good catch.
Thanks for the review, will make the changes and resubmit.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH v2 3/3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-07 16:59 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-07 16:59 UTC (permalink / raw)
To: linux-arm-kernel
Nishanth Menon <nm@ti.com> writes:
> On 16:27-20130606, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Add another DTS include file
>> (twl4030_omap3_mux.dtsi) for boards that hook up the twl4030 this way
>> to include.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>> feature.
>>
>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>> Cc: Benoit Cousson <b-cousson@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> ---
>> v2: Updated to change board files instead of twl4030 file.
>>
>> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
>> arch/arm/boot/dts/omap3-beagle.dts | 1 +
>> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
>> arch/arm/boot/dts/omap3-evm.dts | 1 +
>> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
>> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
>> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
>> arch/arm/boot/dts/twl4030_omap3_mux.dtsi | 17 +++++++++++++++++
> minor comment:
> can we do twl4030_omap3 instead? That way, I can add in twl4030 VP
> specific parameters which are OMAP3 specific?
Good idea, I'll rename it to twl4030_omap3.dtsi
[...]
>> diff --git a/arch/arm/boot/dts/twl4030_omap3_mux.dtsi b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>> new file mode 100644
>> index 0000000..1bd0f9f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/twl4030_omap3_mux.dtsi
>> @@ -0,0 +1,17 @@
> some license text?
oops
>> +&twl {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&twl4030_pins>;
>> +};
>> +
>> +&omap3_pmx_core {
>> + /*
>> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
>> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
>> + * defaults for the SYS_NIRQ pin here.
> ^^^ tab?
good catch.
Thanks for the review, will make the changes and resubmit.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-05-31 22:45 ` Kevin Hilman
@ 2013-06-07 18:31 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-07 18:31 UTC (permalink / raw)
To: Benoit Cousson, linux-omap
Cc: linux-arm-kernel, Florian Vaussard, Nishanth Menon
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Add another DTS include file
(twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
to include.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Special thanks to Florian Vaussard for suggesting use of preprocessor
feature.
Cc: Florian Vaussard <florian.vaussard@epfl.ch>
Cc: Benoit Cousson <b-cousson@ti.com>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
v3: renamed to twl4030_omap3 (suggested by Nishanth), add GPL, fix whitespace
v2: Updated to change board files instead of twl4030 file.
arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
arch/arm/boot/dts/omap3-beagle.dts | 1 +
arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
arch/arm/boot/dts/omap3-evm.dts | 1 +
arch/arm/boot/dts/omap3-igep.dtsi | 1 +
arch/arm/boot/dts/omap3-overo.dtsi | 1 +
arch/arm/boot/dts/omap3430-sdp.dts | 1 +
arch/arm/boot/dts/twl4030_omap3.dtsi | 25 +++++++++++++++++++++++++
8 files changed, 32 insertions(+)
create mode 100644 arch/arm/boot/dts/twl4030_omap3.dtsi
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ad17b6b..afdb164 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -96,6 +96,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index eb7e02a..dfd8310 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -135,6 +135,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
index 5be71b1..af32eff 100644
--- a/arch/arm/boot/dts/omap3-devkit8000.dts
+++ b/arch/arm/boot/dts/omap3-devkit8000.dts
@@ -81,6 +81,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
index baa24bb..7d4329d 100644
--- a/arch/arm/boot/dts/omap3-evm.dts
+++ b/arch/arm/boot/dts/omap3-evm.dts
@@ -45,6 +45,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
index 5224c29..bc48b11 100644
--- a/arch/arm/boot/dts/omap3-igep.dtsi
+++ b/arch/arm/boot/dts/omap3-igep.dtsi
@@ -87,6 +87,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index 144a5a2..4bc78896 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -50,6 +50,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
/* i2c2 pins are used for gpio */
&i2c2 {
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index 2a725a0..c4a1c0a 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -29,6 +29,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/twl4030_omap3.dtsi b/arch/arm/boot/dts/twl4030_omap3.dtsi
new file mode 100644
index 0000000..c353ef0
--- /dev/null
+++ b/arch/arm/boot/dts/twl4030_omap3.dtsi
@@ -0,0 +1,25 @@
+/*
+ * Copyright (C) 2013 Linaro, Ltd.
+ *
+ * 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 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
+};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
+ >;
+ };
+};
--
1.8.3
^ permalink raw reply related [flat|nested] 52+ messages in thread* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-07 18:31 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-07 18:31 UTC (permalink / raw)
To: linux-arm-kernel
On most OMAP3 platforms, the twl4030 IRQ line is connected to the
SYS_NIRQ line on OMAP. Add another DTS include file
(twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
to include.
This allows RTC wake from off-mode to work again on OMAP3-based
platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
3530/Overo, 3730/Overo-STORM.
Special thanks to Florian Vaussard for suggesting use of preprocessor
feature.
Cc: Florian Vaussard <florian.vaussard@epfl.ch>
Cc: Benoit Cousson <b-cousson@ti.com>
Cc: Nishanth Menon <nm@ti.com>
Signed-off-by: Kevin Hilman <khilman@linaro.org>
---
v3: renamed to twl4030_omap3 (suggested by Nishanth), add GPL, fix whitespace
v2: Updated to change board files instead of twl4030 file.
arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
arch/arm/boot/dts/omap3-beagle.dts | 1 +
arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
arch/arm/boot/dts/omap3-evm.dts | 1 +
arch/arm/boot/dts/omap3-igep.dtsi | 1 +
arch/arm/boot/dts/omap3-overo.dtsi | 1 +
arch/arm/boot/dts/omap3430-sdp.dts | 1 +
arch/arm/boot/dts/twl4030_omap3.dtsi | 25 +++++++++++++++++++++++++
8 files changed, 32 insertions(+)
create mode 100644 arch/arm/boot/dts/twl4030_omap3.dtsi
diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
index ad17b6b..afdb164 100644
--- a/arch/arm/boot/dts/omap3-beagle-xm.dts
+++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
@@ -96,6 +96,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
index eb7e02a..dfd8310 100644
--- a/arch/arm/boot/dts/omap3-beagle.dts
+++ b/arch/arm/boot/dts/omap3-beagle.dts
@@ -135,6 +135,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
index 5be71b1..af32eff 100644
--- a/arch/arm/boot/dts/omap3-devkit8000.dts
+++ b/arch/arm/boot/dts/omap3-devkit8000.dts
@@ -81,6 +81,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
index baa24bb..7d4329d 100644
--- a/arch/arm/boot/dts/omap3-evm.dts
+++ b/arch/arm/boot/dts/omap3-evm.dts
@@ -45,6 +45,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
index 5224c29..bc48b11 100644
--- a/arch/arm/boot/dts/omap3-igep.dtsi
+++ b/arch/arm/boot/dts/omap3-igep.dtsi
@@ -87,6 +87,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&i2c2 {
clock-frequency = <400000>;
diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
index 144a5a2..4bc78896 100644
--- a/arch/arm/boot/dts/omap3-overo.dtsi
+++ b/arch/arm/boot/dts/omap3-overo.dtsi
@@ -50,6 +50,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
/* i2c2 pins are used for gpio */
&i2c2 {
diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
index 2a725a0..c4a1c0a 100644
--- a/arch/arm/boot/dts/omap3430-sdp.dts
+++ b/arch/arm/boot/dts/omap3430-sdp.dts
@@ -29,6 +29,7 @@
};
#include "twl4030.dtsi"
+#include "twl4030_omap3.dtsi"
&mmc1 {
vmmc-supply = <&vmmc1>;
diff --git a/arch/arm/boot/dts/twl4030_omap3.dtsi b/arch/arm/boot/dts/twl4030_omap3.dtsi
new file mode 100644
index 0000000..c353ef0
--- /dev/null
+++ b/arch/arm/boot/dts/twl4030_omap3.dtsi
@@ -0,0 +1,25 @@
+/*
+ * Copyright (C) 2013 Linaro, Ltd.
+ *
+ * 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 {
+ pinctrl-names = "default";
+ pinctrl-0 = <&twl4030_pins>;
+};
+
+&omap3_pmx_core {
+ /*
+ * On most OMAP3 platforms, the twl4030 IRQ line is connected
+ * to the SYS_NIRQ line on OMAP. Therefore, configure the
+ * defaults for the SYS_NIRQ pin here.
+ */
+ twl4030_pins: pinmux_twl4030_pins {
+ pinctrl-single,pins = <
+ 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
+ >;
+ };
+};
--
1.8.3
^ permalink raw reply related [flat|nested] 52+ messages in thread* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-07 18:31 ` Kevin Hilman
@ 2013-06-07 19:31 ` Nishanth Menon
-1 siblings, 0 replies; 52+ messages in thread
From: Nishanth Menon @ 2013-06-07 19:31 UTC (permalink / raw)
To: Kevin Hilman
Cc: Benoit Cousson, linux-omap, linux-arm-kernel, Florian Vaussard
On 11:31-20130607, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Add another DTS include file
> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
> to include.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Special thanks to Florian Vaussard for suggesting use of preprocessor
> feature.
>
> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
> Cc: Benoit Cousson <b-cousson@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
Thanks,
Reviewed-by: Nishanth Menon <nm@ti.com>
> ---
> v3: renamed to twl4030_omap3 (suggested by Nishanth), add GPL, fix whitespace
> v2: Updated to change board files instead of twl4030 file.
>
> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
> arch/arm/boot/dts/omap3-beagle.dts | 1 +
> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
> arch/arm/boot/dts/omap3-evm.dts | 1 +
> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
> arch/arm/boot/dts/twl4030_omap3.dtsi | 25 +++++++++++++++++++++++++
> 8 files changed, 32 insertions(+)
> create mode 100644 arch/arm/boot/dts/twl4030_omap3.dtsi
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ad17b6b..afdb164 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -96,6 +96,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index eb7e02a..dfd8310 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -135,6 +135,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
> index 5be71b1..af32eff 100644
> --- a/arch/arm/boot/dts/omap3-devkit8000.dts
> +++ b/arch/arm/boot/dts/omap3-devkit8000.dts
> @@ -81,6 +81,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
> index baa24bb..7d4329d 100644
> --- a/arch/arm/boot/dts/omap3-evm.dts
> +++ b/arch/arm/boot/dts/omap3-evm.dts
> @@ -45,6 +45,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
> index 5224c29..bc48b11 100644
> --- a/arch/arm/boot/dts/omap3-igep.dtsi
> +++ b/arch/arm/boot/dts/omap3-igep.dtsi
> @@ -87,6 +87,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index 144a5a2..4bc78896 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -50,6 +50,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> /* i2c2 pins are used for gpio */
> &i2c2 {
> diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
> index 2a725a0..c4a1c0a 100644
> --- a/arch/arm/boot/dts/omap3430-sdp.dts
> +++ b/arch/arm/boot/dts/omap3430-sdp.dts
> @@ -29,6 +29,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/twl4030_omap3.dtsi b/arch/arm/boot/dts/twl4030_omap3.dtsi
> new file mode 100644
> index 0000000..c353ef0
> --- /dev/null
> +++ b/arch/arm/boot/dts/twl4030_omap3.dtsi
> @@ -0,0 +1,25 @@
> +/*
> + * Copyright (C) 2013 Linaro, Ltd.
> + *
> + * 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 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
> +};
> +
> +&omap3_pmx_core {
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here.
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
> --
> 1.8.3
>
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 52+ messages in thread* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-07 19:31 ` Nishanth Menon
0 siblings, 0 replies; 52+ messages in thread
From: Nishanth Menon @ 2013-06-07 19:31 UTC (permalink / raw)
To: linux-arm-kernel
On 11:31-20130607, Kevin Hilman wrote:
> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
> SYS_NIRQ line on OMAP. Add another DTS include file
> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
> to include.
>
> This allows RTC wake from off-mode to work again on OMAP3-based
> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
> 3530/Overo, 3730/Overo-STORM.
>
> Special thanks to Florian Vaussard for suggesting use of preprocessor
> feature.
>
> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
> Cc: Benoit Cousson <b-cousson@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
> Signed-off-by: Kevin Hilman <khilman@linaro.org>
Thanks,
Reviewed-by: Nishanth Menon <nm@ti.com>
> ---
> v3: renamed to twl4030_omap3 (suggested by Nishanth), add GPL, fix whitespace
> v2: Updated to change board files instead of twl4030 file.
>
> arch/arm/boot/dts/omap3-beagle-xm.dts | 1 +
> arch/arm/boot/dts/omap3-beagle.dts | 1 +
> arch/arm/boot/dts/omap3-devkit8000.dts | 1 +
> arch/arm/boot/dts/omap3-evm.dts | 1 +
> arch/arm/boot/dts/omap3-igep.dtsi | 1 +
> arch/arm/boot/dts/omap3-overo.dtsi | 1 +
> arch/arm/boot/dts/omap3430-sdp.dts | 1 +
> arch/arm/boot/dts/twl4030_omap3.dtsi | 25 +++++++++++++++++++++++++
> 8 files changed, 32 insertions(+)
> create mode 100644 arch/arm/boot/dts/twl4030_omap3.dtsi
>
> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> index ad17b6b..afdb164 100644
> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> @@ -96,6 +96,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts
> index eb7e02a..dfd8310 100644
> --- a/arch/arm/boot/dts/omap3-beagle.dts
> +++ b/arch/arm/boot/dts/omap3-beagle.dts
> @@ -135,6 +135,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-devkit8000.dts b/arch/arm/boot/dts/omap3-devkit8000.dts
> index 5be71b1..af32eff 100644
> --- a/arch/arm/boot/dts/omap3-devkit8000.dts
> +++ b/arch/arm/boot/dts/omap3-devkit8000.dts
> @@ -81,6 +81,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/omap3-evm.dts b/arch/arm/boot/dts/omap3-evm.dts
> index baa24bb..7d4329d 100644
> --- a/arch/arm/boot/dts/omap3-evm.dts
> +++ b/arch/arm/boot/dts/omap3-evm.dts
> @@ -45,6 +45,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-igep.dtsi b/arch/arm/boot/dts/omap3-igep.dtsi
> index 5224c29..bc48b11 100644
> --- a/arch/arm/boot/dts/omap3-igep.dtsi
> +++ b/arch/arm/boot/dts/omap3-igep.dtsi
> @@ -87,6 +87,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &i2c2 {
> clock-frequency = <400000>;
> diff --git a/arch/arm/boot/dts/omap3-overo.dtsi b/arch/arm/boot/dts/omap3-overo.dtsi
> index 144a5a2..4bc78896 100644
> --- a/arch/arm/boot/dts/omap3-overo.dtsi
> +++ b/arch/arm/boot/dts/omap3-overo.dtsi
> @@ -50,6 +50,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> /* i2c2 pins are used for gpio */
> &i2c2 {
> diff --git a/arch/arm/boot/dts/omap3430-sdp.dts b/arch/arm/boot/dts/omap3430-sdp.dts
> index 2a725a0..c4a1c0a 100644
> --- a/arch/arm/boot/dts/omap3430-sdp.dts
> +++ b/arch/arm/boot/dts/omap3430-sdp.dts
> @@ -29,6 +29,7 @@
> };
>
> #include "twl4030.dtsi"
> +#include "twl4030_omap3.dtsi"
>
> &mmc1 {
> vmmc-supply = <&vmmc1>;
> diff --git a/arch/arm/boot/dts/twl4030_omap3.dtsi b/arch/arm/boot/dts/twl4030_omap3.dtsi
> new file mode 100644
> index 0000000..c353ef0
> --- /dev/null
> +++ b/arch/arm/boot/dts/twl4030_omap3.dtsi
> @@ -0,0 +1,25 @@
> +/*
> + * Copyright (C) 2013 Linaro, Ltd.
> + *
> + * 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 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&twl4030_pins>;
> +};
> +
> +&omap3_pmx_core {
> + /*
> + * On most OMAP3 platforms, the twl4030 IRQ line is connected
> + * to the SYS_NIRQ line on OMAP. Therefore, configure the
> + * defaults for the SYS_NIRQ pin here.
> + */
> + twl4030_pins: pinmux_twl4030_pins {
> + pinctrl-single,pins = <
> + 0x1b0 (PIN_INPUT_PULLUP | PIN_OFF_WAKEUPENABLE | MUX_MODE0) /* sys_nirq.sys_nirq */
> + >;
> + };
> +};
> --
> 1.8.3
>
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 52+ messages in thread* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-07 19:31 ` Nishanth Menon
@ 2013-06-10 9:22 ` Benoit Cousson
-1 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-10 9:22 UTC (permalink / raw)
To: Kevin Hilman
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
Hi Kevin,
On 06/07/2013 09:31 PM, Nishanth Menon wrote:
> On 11:31-20130607, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Add another DTS include file
>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>> to include.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>> feature.
>>
>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>> Cc: Benoit Cousson <b-cousson@ti.com>
>> Cc: Nishanth Menon <nm@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> Thanks,
> Reviewed-by: Nishanth Menon <nm@ti.com>
Thanks, I've just applied it in for_3.11/dts.
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-10 9:22 ` Benoit Cousson
0 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-10 9:22 UTC (permalink / raw)
To: linux-arm-kernel
Hi Kevin,
On 06/07/2013 09:31 PM, Nishanth Menon wrote:
> On 11:31-20130607, Kevin Hilman wrote:
>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>> SYS_NIRQ line on OMAP. Add another DTS include file
>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>> to include.
>>
>> This allows RTC wake from off-mode to work again on OMAP3-based
>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>> 3530/Overo, 3730/Overo-STORM.
>>
>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>> feature.
>>
>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>> Cc: Benoit Cousson <b-cousson@ti.com>
>> Cc: Nishanth Menon <nm@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
> Thanks,
> Reviewed-by: Nishanth Menon <nm@ti.com>
Thanks, I've just applied it in for_3.11/dts.
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-10 9:22 ` Benoit Cousson
@ 2013-06-10 16:40 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-10 16:40 UTC (permalink / raw)
To: Benoit Cousson
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
Benoit Cousson <b-cousson@ti.com> writes:
> Hi Kevin,
>
> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>> On 11:31-20130607, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>> to include.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>> feature.
>>>
>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>> Cc: Nishanth Menon <nm@ti.com>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> Thanks,
>> Reviewed-by: Nishanth Menon <nm@ti.com>
>
> Thanks, I've just applied it in for_3.11/dts.
Thanks,
Will you apply the first 2 from the series too?
ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-10 16:40 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-10 16:40 UTC (permalink / raw)
To: linux-arm-kernel
Benoit Cousson <b-cousson@ti.com> writes:
> Hi Kevin,
>
> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>> On 11:31-20130607, Kevin Hilman wrote:
>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>> to include.
>>>
>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>> 3530/Overo, 3730/Overo-STORM.
>>>
>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>> feature.
>>>
>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>> Cc: Nishanth Menon <nm@ti.com>
>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>> Thanks,
>> Reviewed-by: Nishanth Menon <nm@ti.com>
>
> Thanks, I've just applied it in for_3.11/dts.
Thanks,
Will you apply the first 2 from the series too?
ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
Thanks,
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-10 16:40 ` Kevin Hilman
@ 2013-06-11 11:33 ` Cousson, Benoit
-1 siblings, 0 replies; 52+ messages in thread
From: Cousson, Benoit @ 2013-06-11 11:33 UTC (permalink / raw)
To: Kevin Hilman
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
On 6/10/2013 6:40 PM, Kevin Hilman wrote:
> Benoit Cousson <b-cousson@ti.com> writes:
>
>> Hi Kevin,
>>
>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>> On 11:31-20130607, Kevin Hilman wrote:
>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>> to include.
>>>>
>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>> 3530/Overo, 3730/Overo-STORM.
>>>>
>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>> feature.
>>>>
>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>> Cc: Nishanth Menon <nm@ti.com>
>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> Thanks,
>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>
>> Thanks, I've just applied it in for_3.11/dts.
>
> Thanks,
>
> Will you apply the first 2 from the series too?
>
> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
Thanks for the reminder, I kind of forgot these ones :-)
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-11 11:33 ` Cousson, Benoit
0 siblings, 0 replies; 52+ messages in thread
From: Cousson, Benoit @ 2013-06-11 11:33 UTC (permalink / raw)
To: linux-arm-kernel
On 6/10/2013 6:40 PM, Kevin Hilman wrote:
> Benoit Cousson <b-cousson@ti.com> writes:
>
>> Hi Kevin,
>>
>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>> On 11:31-20130607, Kevin Hilman wrote:
>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>> to include.
>>>>
>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>> 3530/Overo, 3730/Overo-STORM.
>>>>
>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>> feature.
>>>>
>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>> Cc: Nishanth Menon <nm@ti.com>
>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>> Thanks,
>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>
>> Thanks, I've just applied it in for_3.11/dts.
>
> Thanks,
>
> Will you apply the first 2 from the series too?
>
> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable wakeup
Thanks for the reminder, I kind of forgot these ones :-)
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-11 11:33 ` Cousson, Benoit
@ 2013-06-11 11:43 ` Benoit Cousson
-1 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-11 11:43 UTC (permalink / raw)
To: Kevin Hilman
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>> Benoit Cousson <b-cousson@ti.com> writes:
>>
>>> Hi Kevin,
>>>
>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>> to include.
>>>>>
>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>
>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>> feature.
>>>>>
>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>> Thanks,
>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>
>>> Thanks, I've just applied it in for_3.11/dts.
>>
>> Thanks,
>>
>> Will you apply the first 2 from the series too?
>>
>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>> wakeup
>
> Thanks for the reminder, I kind of forgot these ones :-)
BTW, do you have any v2 for these ones? This version does not use the
macros.
Moreover I cannot apply them on top of Florian's series :-(
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-11 11:43 ` Benoit Cousson
0 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-11 11:43 UTC (permalink / raw)
To: linux-arm-kernel
On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>> Benoit Cousson <b-cousson@ti.com> writes:
>>
>>> Hi Kevin,
>>>
>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>> to include.
>>>>>
>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>
>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>> feature.
>>>>>
>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>> Thanks,
>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>
>>> Thanks, I've just applied it in for_3.11/dts.
>>
>> Thanks,
>>
>> Will you apply the first 2 from the series too?
>>
>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>> wakeup
>
> Thanks for the reminder, I kind of forgot these ones :-)
BTW, do you have any v2 for these ones? This version does not use the
macros.
Moreover I cannot apply them on top of Florian's series :-(
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-11 11:43 ` Benoit Cousson
@ 2013-06-11 14:30 ` Kevin Hilman
-1 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-11 14:30 UTC (permalink / raw)
To: Benoit Cousson
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
Benoit Cousson <b-cousson@ti.com> writes:
> On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
>> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>>> Benoit Cousson <b-cousson@ti.com> writes:
>>>
>>>> Hi Kevin,
>>>>
>>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>>> to include.
>>>>>>
>>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>>
>>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>>> feature.
>>>>>>
>>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>>> Thanks,
>>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>>
>>>> Thanks, I've just applied it in for_3.11/dts.
>>>
>>> Thanks,
>>>
>>> Will you apply the first 2 from the series too?
>>>
>>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>>> wakeup
>>
>> Thanks for the reminder, I kind of forgot these ones :-)
>
> BTW, do you have any v2 for these ones? This version does not use the
> macros.
> Moreover I cannot apply them on top of Florian's series :-(
Oops, I thought I reposted those too. All three are here:
git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.11/dt
which is based on top of your for_3.11/dts branch.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-11 14:30 ` Kevin Hilman
0 siblings, 0 replies; 52+ messages in thread
From: Kevin Hilman @ 2013-06-11 14:30 UTC (permalink / raw)
To: linux-arm-kernel
Benoit Cousson <b-cousson@ti.com> writes:
> On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
>> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>>> Benoit Cousson <b-cousson@ti.com> writes:
>>>
>>>> Hi Kevin,
>>>>
>>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>>> to include.
>>>>>>
>>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>>
>>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>>> feature.
>>>>>>
>>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>>> Thanks,
>>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>>
>>>> Thanks, I've just applied it in for_3.11/dts.
>>>
>>> Thanks,
>>>
>>> Will you apply the first 2 from the series too?
>>>
>>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>>> wakeup
>>
>> Thanks for the reminder, I kind of forgot these ones :-)
>
> BTW, do you have any v2 for these ones? This version does not use the
> macros.
> Moreover I cannot apply them on top of Florian's series :-(
Oops, I thought I reposted those too. All three are here:
git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.11/dt
which is based on top of your for_3.11/dts branch.
Kevin
^ permalink raw reply [flat|nested] 52+ messages in thread
* Re: [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
2013-06-11 14:30 ` Kevin Hilman
@ 2013-06-19 1:25 ` Benoit Cousson
-1 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-19 1:25 UTC (permalink / raw)
To: Kevin Hilman
Cc: Nishanth Menon, linux-omap, linux-arm-kernel, Florian Vaussard
On 06/11/2013 09:30 AM, Kevin Hilman wrote:
> Benoit Cousson <b-cousson@ti.com> writes:
>
>> On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
>>> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>>>> Benoit Cousson <b-cousson@ti.com> writes:
>>>>
>>>>> Hi Kevin,
>>>>>
>>>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>>>> to include.
>>>>>>>
>>>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>>>
>>>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>>>> feature.
>>>>>>>
>>>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>>>> Thanks,
>>>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>>>
>>>>> Thanks, I've just applied it in for_3.11/dts.
>>>>
>>>> Thanks,
>>>>
>>>> Will you apply the first 2 from the series too?
>>>>
>>>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>>>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>>>> wakeup
>>>
>>> Thanks for the reminder, I kind of forgot these ones :-)
>>
>> BTW, do you have any v2 for these ones? This version does not use the
>> macros.
>> Moreover I cannot apply them on top of Florian's series :-(
>
> Oops, I thought I reposted those too. All three are here:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.11/dt
>
> which is based on top of your for_3.11/dts branch.
Thanks, I've just applied it.
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread
* [PATCH v3] ARM: DTS: TWL4030: fix mux and wakeup for SYS_NIRQ line
@ 2013-06-19 1:25 ` Benoit Cousson
0 siblings, 0 replies; 52+ messages in thread
From: Benoit Cousson @ 2013-06-19 1:25 UTC (permalink / raw)
To: linux-arm-kernel
On 06/11/2013 09:30 AM, Kevin Hilman wrote:
> Benoit Cousson <b-cousson@ti.com> writes:
>
>> On 06/11/2013 01:33 PM, Cousson, Benoit wrote:
>>> On 6/10/2013 6:40 PM, Kevin Hilman wrote:
>>>> Benoit Cousson <b-cousson@ti.com> writes:
>>>>
>>>>> Hi Kevin,
>>>>>
>>>>> On 06/07/2013 09:31 PM, Nishanth Menon wrote:
>>>>>> On 11:31-20130607, Kevin Hilman wrote:
>>>>>>> On most OMAP3 platforms, the twl4030 IRQ line is connected to the
>>>>>>> SYS_NIRQ line on OMAP. Add another DTS include file
>>>>>>> (twl4030_omap3.dtsi) for boards that hook up the twl4030 this way
>>>>>>> to include.
>>>>>>>
>>>>>>> This allows RTC wake from off-mode to work again on OMAP3-based
>>>>>>> platforms with twl4030. Tested on 3530/Beagle, 3730/Beagle-xM,
>>>>>>> 3530/Overo, 3730/Overo-STORM.
>>>>>>>
>>>>>>> Special thanks to Florian Vaussard for suggesting use of preprocessor
>>>>>>> feature.
>>>>>>>
>>>>>>> Cc: Florian Vaussard <florian.vaussard@epfl.ch>
>>>>>>> Cc: Benoit Cousson <b-cousson@ti.com>
>>>>>>> Cc: Nishanth Menon <nm@ti.com>
>>>>>>> Signed-off-by: Kevin Hilman <khilman@linaro.org>
>>>>>> Thanks,
>>>>>> Reviewed-by: Nishanth Menon <nm@ti.com>
>>>>>
>>>>> Thanks, I've just applied it in for_3.11/dts.
>>>>
>>>> Thanks,
>>>>
>>>> Will you apply the first 2 from the series too?
>>>>
>>>> ARM: DTS: OMAP3: beagle/overo: mux console UART, enable wakeup
>>>> ARM: DTS: OMAP3: beagle: enable user button via gpio_keys, enable
>>>> wakeup
>>>
>>> Thanks for the reminder, I kind of forgot these ones :-)
>>
>> BTW, do you have any v2 for these ones? This version does not use the
>> macros.
>> Moreover I cannot apply them on top of Florian's series :-(
>
> Oops, I thought I reposted those too. All three are here:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.11/dt
>
> which is based on top of your for_3.11/dts branch.
Thanks, I've just applied it.
Regards,
Benoit
^ permalink raw reply [flat|nested] 52+ messages in thread