lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAB=otbR-n7f=O3+YTPnfO6wRWz2aEcUnbjHoD746GsNgFW3tzQ@mail.gmail.com>
Date:	Mon, 22 Jul 2013 19:24:13 +0300
From:	Ruslan Bilovol <ruslan.bilovol@...com>
To:	Nishanth Menon <nm@...com>
Cc:	benoit.cousson@...aro.org, tony@...mide.com,
	devicetree-discuss@...ts.ozlabs.org, linux-omap@...r.kernel.org,
	linux-kernel@...r.kernel.org, uri.y@...iscite.com
Subject: Re: [PATCH] arm: dts: twl6030: typical connection to omap4 as a
 separate dtsi file

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

Powered by Openwall GNU/*/Linux Powered by OpenVZ