From: "Cousson, Benoit" <b-cousson@ti.com>
To: Javier Martinez Canillas <javier@dowhile0.org>
Cc: Anil Kumar <anilkumar880@gmail.com>,
Florian Vaussard <florian.vaussard@epfl.ch>,
Tony Lindgren <tony@atomide.com>,
devicetree-discuss@lists.ozlabs.org,
Daniel Mack <zonque@gmail.com>,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Subject: Re: [PATCH 1/2] ARM: dts: OMAP3: Add GPMC controller
Date: Mon, 18 Feb 2013 13:26:24 +0100 [thread overview]
Message-ID: <51221DF0.2050705@ti.com> (raw)
In-Reply-To: <CABxcv=kDw6pSsdE29-dVqkk2cc-RKBZB6GN8cs+M+C4JEeSbdg@mail.gmail.com>
Hi Javier,
On 2/16/2013 5:44 PM, Javier Martinez Canillas wrote:
> On Sat, Feb 16, 2013 at 2:09 PM, Anil Kumar <anilkumar880@gmail.com> wrote:
>> Hi Florian,
>>
>> On Mon, Jan 28, 2013 at 11:24 PM, Florian Vaussard
>> <florian.vaussard@epfl.ch> wrote:
>>> Add device-tree support for the GPMC controller on the OMAP3.
>>>
>>> Signed-off-by: Florian Vaussard <florian.vaussard@epfl.ch>
>>> ---
>>> arch/arm/boot/dts/omap3.dtsi | 11 +++++++++++
>>> 1 files changed, 11 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
>>> index 6c63118..2ddae38 100644
>>> --- a/arch/arm/boot/dts/omap3.dtsi
>>> +++ b/arch/arm/boot/dts/omap3.dtsi
>>> @@ -403,5 +403,16 @@
>>> ti,timer-alwon;
>>> ti,timer-secure;
>>> };
>>> +
>>> + gpmc: gpmc@6e000000 {
>>> + compatible = "ti,omap3430-gpmc";
>>> + ti,hwmods = "gpmc";
>>> + reg = <0x6e000000 0x1000000>;
>>> + interrupts = <20>;
>>> + gpmc,num-cs = <8>;
>>> + gpmc,num-waitpins = <4>;
>>> + #address-cells = <2>;
>>> + #size-cells = <1>;
>>> + };
>>> };
>>> };
>>
>> Tested on devkit8000 (omap3 based board)
>>
>> Please take my Tested-by: Anil Kumar <anilk4.v@gmail.com>
>>
>> Thanks,
>> Anil
>>
>
> Hello Tony and Benoit,
>
> Could this patch be merged to one of your branches?
I'll take it.
> It has already my Reviewed-by and now Anil's Tested-by.
>
> Now that Daniel's OMAP GPMC binding were merged, there seems to be
> many people working on adding support on their boards DT for
> peripherals connected through the GPMC (NAND, OneNAND, SMSC LAN, etc).
>
> I think it will be easier if people can base their work on top of this
> patch instead of duplicating the work and sending the same patch to
> add a GPMC device node to omap3.dtsi on each patch-set.
Regards,
Benoit
WARNING: multiple messages have this Message-ID (diff)
From: b-cousson@ti.com (Cousson, Benoit)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: dts: OMAP3: Add GPMC controller
Date: Mon, 18 Feb 2013 13:26:24 +0100 [thread overview]
Message-ID: <51221DF0.2050705@ti.com> (raw)
In-Reply-To: <CABxcv=kDw6pSsdE29-dVqkk2cc-RKBZB6GN8cs+M+C4JEeSbdg@mail.gmail.com>
Hi Javier,
On 2/16/2013 5:44 PM, Javier Martinez Canillas wrote:
> On Sat, Feb 16, 2013 at 2:09 PM, Anil Kumar <anilkumar880@gmail.com> wrote:
>> Hi Florian,
>>
>> On Mon, Jan 28, 2013 at 11:24 PM, Florian Vaussard
>> <florian.vaussard@epfl.ch> wrote:
>>> Add device-tree support for the GPMC controller on the OMAP3.
>>>
>>> Signed-off-by: Florian Vaussard <florian.vaussard@epfl.ch>
>>> ---
>>> arch/arm/boot/dts/omap3.dtsi | 11 +++++++++++
>>> 1 files changed, 11 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
>>> index 6c63118..2ddae38 100644
>>> --- a/arch/arm/boot/dts/omap3.dtsi
>>> +++ b/arch/arm/boot/dts/omap3.dtsi
>>> @@ -403,5 +403,16 @@
>>> ti,timer-alwon;
>>> ti,timer-secure;
>>> };
>>> +
>>> + gpmc: gpmc at 6e000000 {
>>> + compatible = "ti,omap3430-gpmc";
>>> + ti,hwmods = "gpmc";
>>> + reg = <0x6e000000 0x1000000>;
>>> + interrupts = <20>;
>>> + gpmc,num-cs = <8>;
>>> + gpmc,num-waitpins = <4>;
>>> + #address-cells = <2>;
>>> + #size-cells = <1>;
>>> + };
>>> };
>>> };
>>
>> Tested on devkit8000 (omap3 based board)
>>
>> Please take my Tested-by: Anil Kumar <anilk4.v@gmail.com>
>>
>> Thanks,
>> Anil
>>
>
> Hello Tony and Benoit,
>
> Could this patch be merged to one of your branches?
I'll take it.
> It has already my Reviewed-by and now Anil's Tested-by.
>
> Now that Daniel's OMAP GPMC binding were merged, there seems to be
> many people working on adding support on their boards DT for
> peripherals connected through the GPMC (NAND, OneNAND, SMSC LAN, etc).
>
> I think it will be easier if people can base their work on top of this
> patch instead of duplicating the work and sending the same patch to
> add a GPMC device node to omap3.dtsi on each patch-set.
Regards,
Benoit
next prev parent reply other threads:[~2013-02-18 12:26 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-28 17:54 [PATCH 0/2] ARM: dts: OMAP3: Add GPMC controller and NAND memory to Overo Florian Vaussard
2013-01-28 17:54 ` Florian Vaussard
2013-01-28 17:54 ` [PATCH 1/2] ARM: dts: OMAP3: Add GPMC controller Florian Vaussard
2013-01-28 17:54 ` Florian Vaussard
2013-02-04 9:27 ` Javier Martinez Canillas
2013-02-04 9:27 ` Javier Martinez Canillas
2013-02-04 10:36 ` Florian Vaussard
2013-02-04 10:36 ` Florian Vaussard
[not found] ` <510F8F49.9020800-p8DiymsW2f8@public.gmane.org>
2013-02-04 11:57 ` Javier Martinez Canillas
2013-02-04 11:57 ` Javier Martinez Canillas
2013-02-04 17:32 ` Tony Lindgren
2013-02-04 17:32 ` Tony Lindgren
2013-02-04 18:15 ` Javier Martinez Canillas
2013-02-04 18:15 ` Javier Martinez Canillas
2013-02-05 17:23 ` Florian Vaussard
2013-02-05 17:23 ` Florian Vaussard
2013-02-05 19:40 ` Javier Martinez Canillas
2013-02-05 19:40 ` Javier Martinez Canillas
2013-02-16 13:09 ` Anil Kumar
2013-02-16 13:09 ` Anil Kumar
2013-02-16 16:44 ` Javier Martinez Canillas
2013-02-16 16:44 ` Javier Martinez Canillas
2013-02-18 12:26 ` Cousson, Benoit [this message]
2013-02-18 12:26 ` Cousson, Benoit
2013-02-19 1:00 ` Jon Hunter
2013-02-19 1:00 ` Jon Hunter
2013-03-14 16:01 ` Benoit Cousson
2013-03-14 16:01 ` Benoit Cousson
2013-01-28 17:54 ` [PATCH 2/2] ARM: dts: OMAP3: Add NAND memory for Overo products Florian Vaussard
2013-01-28 17:54 ` Florian Vaussard
2013-02-01 22:16 ` [PATCH 0/2] ARM: dts: OMAP3: Add GPMC controller and NAND memory to Overo Tony Lindgren
2013-02-01 22:16 ` Tony Lindgren
2013-02-04 8:58 ` Florian Vaussard
2013-02-04 8:58 ` Florian Vaussard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51221DF0.2050705@ti.com \
--to=b-cousson@ti.com \
--cc=anilkumar880@gmail.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=ezequiel.garcia@free-electrons.com \
--cc=florian.vaussard@epfl.ch \
--cc=javier@dowhile0.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=tony@atomide.com \
--cc=zonque@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.