[<prev] [next>] [day] [month] [year] [list]
Message-ID: <51EFD235.4060704@ti.com>
Date: Wed, 24 Jul 2013 16:10:13 +0300
From: Ruslan Bilovol <ruslan.bilovol@...com>
To: Uri Yosef <uri.y@...iscite.com>
CC: Nishanth Menon <nm@...com>, <benoit.cousson@...aro.org>,
Tony Lindgren <tony@...mide.com>, <devicetree@...r.kernel.org>,
<linux-omap@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm: dts: twl6030: typical connection to omap4 as a separate
dtsi file
Hi Uri,
On 07/24/2013 10:05 AM, Uri Yosef wrote:
> Hi Ruslan,
>
> Our OMAP4 SOM connection to PMIC is similar to Blaze SDP4430.
Thank you for confirmation, I will add this in next patch series.
Regards,
Ruslan
> Regards,
> Uri Yosef
>
>
>
> On Mon, Jul 22, 2013 at 7:24 PM, Ruslan Bilovol<ruslan.bilovol@...com>wrote:
>
>> Hi Nishanth,
>>
>> On Fri, Jul 19, 2013 at 5:10 PM, Nishanth Menon<nm@...com> wrote:
>>> On 07/19/2013 07:10 AM, Ruslan Bilovol wrote:
>>>> The OMAP4 SoC family uses specially-designed
>>>> PMIC (power management IC) companion chip for power
>>>> management needs: TWL6030/TWL6032.
>>>> Therefore there is a typical connection of PMIC to OMAP4
>>>> so we can figure it out into separate .dtsi file
>>>> and do not duplicate over board-specific files.
>>>>
>>>> Tested on OMAP4 SDP board and compile-tested for Panda board
>>>>
>>>> Signed-off-by: Ruslan Bilovol<ruslan.bilovol@...com>
>>>> ---
>>>> arch/arm/boot/dts/omap4-panda-common.dtsi | 21 +---------------
>>>> arch/arm/boot/dts/omap4-sdp.dts | 21 +---------------
>>>> arch/arm/boot/dts/twl6030_omap4.dtsi | 38
>>>> +++++++++++++++++++++++++++++
>>> arch/arm/boot/dts/omap4-var-som.dts ?
>> There is no muxing for TWL6030 in that file at all, so I didn't add it.
>> Moreover, that file seems to be not maintained since added (just
>> generic fixes or framework changes that came to all other files)
>> Second thing - I do not have schematics and HW for verification.
>> But if somebody from Variscite can confirm that omap4 is typically
>> connected to PMIC, I'm happy to add it as separate patch.
>> So I CCed Uri Yosef who is original author of the omap4-var-som.dts
>>
>> Regards,
>> Ruslan
>>
>>>> 3 files changed, 40 insertions(+), 40 deletions(-)
>>>> create mode 100644 arch/arm/boot/dts/twl6030_omap4.dtsi
>>>>
>>>> diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi
>>>> b/arch/arm/boot/dts/omap4-panda-common.dtsi
>>>> index faa95b5..d796302 100644
>>>> --- a/arch/arm/boot/dts/omap4-panda-common.dtsi
>>>> +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi
>>>> @@ -109,23 +109,9 @@
>>>> };
>>>> };
>>>>
>>>> -&omap4_pmx_wkup {
>>>> - pinctrl-names = "default";
>>>> - pinctrl-0 =<
>>>> -&twl6030_wkup_pins
>>>> ->;
>>>> -
>>>> - twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
>>>> - pinctrl-single,pins =<
>>>> - 0x14 (PIN_OUTPUT | MUX_MODE2) /*
>>>> fref_clk0_out.sys_drm_msecure */
>>>> ->;
>>>> - };
>>>> -};
>>>> -
>>>> &omap4_pmx_core {
>>>> pinctrl-names = "default";
>>>> pinctrl-0 =<
>>>> -&twl6030_pins
>>>> &twl6040_pins
>>>> &mcpdm_pins
>>>> &mcbsp1_pins
>>>> @@ -134,12 +120,6 @@
>>>> &hsusbb1_pins
>>>> >;
>>>>
>>>> - twl6030_pins: pinmux_twl6030_pins {
>>>> - pinctrl-single,pins =<
>>>> - 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
>>>> /* sys_nirq1.sys_nirq1 */
>>>> ->;
>>>> - };
>>>> -
>>>> twl6040_pins: pinmux_twl6040_pins {
>>>> pinctrl-single,pins =<
>>>> 0xe0 (PIN_OUTPUT | MUX_MODE3) /*
>>>> hdq_sio.gpio_127 */
>>>> @@ -265,6 +245,7 @@
>>>> };
>>>>
>>>> #include "twl6030.dtsi"
>>>> +#include "twl6030_omap4.dtsi"
>>>>
>>>> &i2c2 {
>>>> pinctrl-names = "default";
>>>> diff --git a/arch/arm/boot/dts/omap4-sdp.dts
>>>> b/arch/arm/boot/dts/omap4-sdp.dts
>>>> index 7951b4e..91030c3 100644
>>>> --- a/arch/arm/boot/dts/omap4-sdp.dts
>>>> +++ b/arch/arm/boot/dts/omap4-sdp.dts
>>>> @@ -142,23 +142,9 @@
>>>> };
>>>> };
>>>>
>>>> -&omap4_pmx_wkup {
>>>> - pinctrl-names = "default";
>>>> - pinctrl-0 =<
>>>> -&twl6030_wkup_pins
>>>> ->;
>>>> -
>>>> - twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
>>>> - pinctrl-single,pins =<
>>>> - 0x14 (PIN_OUTPUT | MUX_MODE2) /*
>>>> fref_clk0_out.sys_drm_msecure */
>>>> ->;
>>>> - };
>>>> -};
>>>> -
>>>> &omap4_pmx_core {
>>>> pinctrl-names = "default";
>>>> pinctrl-0 =<
>>>> -&twl6030_pins
>>>> &twl6040_pins
>>>> &mcpdm_pins
>>>> &dmic_pins
>>>> @@ -193,12 +179,6 @@
>>>> >;
>>>> };
>>>>
>>>> - twl6030_pins: pinmux_twl6030_pins {
>>>> - pinctrl-single,pins =<
>>>> - 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
>>>> /* sys_nirq1.sys_nirq1 */
>>>> ->;
>>>> - };
>>>> -
>>>> twl6040_pins: pinmux_twl6040_pins {
>>>> pinctrl-single,pins =<
>>>> 0xe0 (PIN_OUTPUT | MUX_MODE3) /*
>>>> hdq_sio.gpio_127 */
>>>> @@ -337,6 +317,7 @@
>>>> };
>>>>
>>>> #include "twl6030.dtsi"
>>>> +#include "twl6030_omap4.dtsi"
>>>>
>>>> &i2c2 {
>>>> pinctrl-names = "default";
>>>> diff --git a/arch/arm/boot/dts/twl6030_omap4.dtsi
>>>> b/arch/arm/boot/dts/twl6030_omap4.dtsi
>>>> new file mode 100644
>>>> index 0000000..a4fa570
>>>> --- /dev/null
>>>> +++ b/arch/arm/boot/dts/twl6030_omap4.dtsi
>>>> @@ -0,0 +1,38 @@
>>>> +/*
>>>> + * Copyright (C) 2013 Texas Instruments Incorporated -
>>>> http://www.ti.com/
>>>> + *
>>>> + * This program is free software; you can redistribute it and/or modify
>>>> + * it under the terms of the GNU General Public License version 2 as
>>>> + * published by the Free Software Foundation.
>>>> + */
>>>> +
>>>> +&twl {
>>>> + /*
>>>> + * On most OMAP4 platforms, the twl6030 IRQ line is connected
>>>> + * to the SYS_NIRQ1 line on OMAP and the twl6030 MSECURE line is
>>>> + * connected to the fref_clk0_out.sys_drm_msecure line.
>>>> + * Therefore, configure the defaults for the SYS_NIRQ1 and
>>>> + * fref_clk0_out.sys_drm_msecure pins here.
>>>> + */
>>>> + pinctrl-names = "default";
>>>> + pinctrl-0 =<
>>>> +&twl6030_pins
>>>> +&twl6030_wkup_pins
>>>> +>;
>>>> +};
>>>> +
>>>> +&omap4_pmx_wkup {
>>>> + twl6030_wkup_pins: pinmux_twl6030_wkup_pins {
>>>> + pinctrl-single,pins =<
>>>> + 0x14 (PIN_OUTPUT | MUX_MODE2) /*
>>>> fref_clk0_out.sys_drm_msecure */
>>>> +>;
>>>> + };
>>>> +};
>>>> +
>>>> +&omap4_pmx_core {
>>>> + twl6030_pins: pinmux_twl6030_pins {
>>>> + pinctrl-single,pins =<
>>>> + 0x15e (WAKEUP_EN | PIN_INPUT_PULLUP | MUX_MODE0)
>>>> /* sys_nirq1.sys_nirq1 */
>>>> +>;
>>>> + };
>>>> +};
>>>>
>>>
>>> --
>>> Regards,
>>> Nishanth Menon
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>>> the body of a message to majordomo@...r.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists