From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Baruch Siach <baruch@tkos.co.il>
Cc: devicetree-discuss@lists.ozlabs.org,
linux-kernel@vger.kernel.org, grant.likely@secretlab.ca,
plagnioj@jcrosoft.com, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH V3 1/2] ARM: at91: dt: at91sam9g45 family and board device tree files
Date: Mon, 24 Oct 2011 16:32:37 +0200 [thread overview]
Message-ID: <4EA57705.6060103@atmel.com> (raw)
In-Reply-To: <20111024141253.GF26649@sapphire.tkos.co.il>
On 10/24/2011 04:12 PM, Baruch Siach :
> Hi Nicolas,
>
> On Mon, Oct 24, 2011 at 04:05:00PM +0200, Nicolas Ferre wrote:
>> Create a new device tree source file for Atmel at91sam9g45 SoC family.
>> The Evaluation Kit at91sam9m10g45ek includes it.
>> This first basic support will be populated as drivers and boards will be
>> converted to device tree.
>> Contains serial, dma and interrupt controllers.
>>
>> The generic board file still takes advantage of platform data for early serial
>> init. As we need a storage media and the NAND flash driver is not converted to
>> DT yet, we keep old initialization for it.
>>
>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>> ---
>
> [snip]
>
>> +DT_MACHINE_START(at91sam9m10g45ek_dt, "Atmel AT91SAM (Device Tree)")
>
> Since this is a generic AT91 machine descriptor, won't "at91sam_dt" be a more
> appropriate name?
For sure that makes sense. I queue this change and make it appear only
in the pull request if it is the only change requested.
>> + /* Maintainer: Atmel */
>> + .timer = &at91sam926x_timer,
>> + .map_io = at91_map_io,
>> + .init_early = ek_init_early,
>> + .init_irq = at91_dt_init_irq,
>> + .init_machine = at91_dt_device_init,
>> + .dt_compat = at91_dt_board_compat,
>> +MACHINE_END
>
> baruch
Thanks for your review,
Best regards,
--
Nicolas Ferre
next prev parent reply other threads:[~2011-10-24 14:32 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-03 10:00 [PATCH V2] AT91: dt: at91sam9g45 family and board device tree files Nicolas Ferre
2011-10-03 8:52 ` Russell King - ARM Linux
2011-10-05 13:00 ` Rob Herring
2011-10-07 12:56 ` Nicolas Ferre
2011-10-18 10:07 ` Nicolas Ferre
2011-10-18 20:44 ` Rob Herring
2011-10-19 8:03 ` Nicolas Ferre
[not found] ` <4E9E8444.3070603-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2011-10-20 2:06 ` Rob Herring
[not found] ` <4E9F8226.4030503-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-10-24 14:05 ` [PATCH V3 1/2] ARM: at91: " Nicolas Ferre
2011-10-24 14:05 ` [PATCH V3 2/2] ARM: at91: add at91sam9g20 and Calao USB A9G20 DT support Nicolas Ferre
[not found] ` <d139db89b4e9830e813abf1329ce1ac499653f2c.1319464310.git.nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2011-10-24 14:09 ` [PATCH V3 (bugfix to fold)] ARM: at91: usb_a9g20.dts: fix memory location Nicolas Ferre
2011-10-24 14:49 ` [PATCH V3 2/2] ARM: at91: add at91sam9g20 and Calao USB A9G20 DT support Rob Herring
2011-10-24 14:12 ` [PATCH V3 1/2] ARM: at91: dt: at91sam9g45 family and board device tree files Baruch Siach
2011-10-24 14:32 ` Nicolas Ferre [this message]
[not found] ` <15ab652499e7f6f8a26724bfd90643a8f3f96ad9.1319464310.git.nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
2011-10-24 14:44 ` Rob Herring
2011-10-25 11:06 ` Nicolas Ferre
2011-10-24 14:34 ` [PATCH V2] AT91: " Grant Likely
[not found] ` <20111024143451.GA8708-e0URQFbLeQY2iJbIjFUEsiwD8/FfD2ys@public.gmane.org>
2011-10-24 14:42 ` Nicolas Ferre
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=4EA57705.6060103@atmel.com \
--to=nicolas.ferre@atmel.com \
--cc=baruch@tkos.co.il \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=plagnioj@jcrosoft.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).