Re: [Bug 1000819] Add support for Atmel AT91SAM9263

[Grant Edwards <grant.b.edwards@gmail.com>]

On 2011-05-18, bugzilla-daemon@bugs.ecos.sourceware.org <bugzilla-daemon@bugs.ecos.sourceware.org> wrote:
> Please do not reply to this email. Use the web interface provided at:
> http://bugs.ecos.sourceware.org/show_bug.cgi?id=1000819
>
> --- Comment #28 from Daniel Helgason <dhelgason@shaw.ca> 2011-05-18 18:19:50 BST ---
> (In reply to comment #11)
>> Let's try to push through with review and get it checked in. I've invited all
>> interested parties to add themselves to the CC list and add their own comment
>> where necessary/appropriate.
>> 
>> Evgeniy, the use of multiple patches is a great help - thank you!
>> 
>> Let's start with the ARM7/ARM9 abstraction work (patch 1). This looks to be a
>> case of moving the existing HAL cache macros (which are not appropriate for
>> AT91SAM9) from the AT91 variant package to a new ARM7 package. I assume that
>> there is nothing AT91-specific in the new package so it could be used by any
>> other ARM7 ports in the future. Please confirm.
>> ...
>
> Is is correct to have AT91 as a variant? I see it more as a package that
> defines a set of common I/O and provides macros for common AT91 stuff.

That makes a lot of sense to me.  In reality, "AT91" is a common set
of peripherals that are used on a variety of Atmel parts having
several different architectures, varieties, and platforms.

> Would it
> make sense if things were arranged more like:
>
> ARM (arch)
> +- ARM7 (variant)
>    + SAM7S (platform)
>    + SAM7X (platform)
>    + Other Non-AT91 chip (platform)
>    + SAM7S-EK (board)
>    + SAM7X-EK (board)
>    ...etc.
> +  ARM9 (variant)
>    + SAM9263 (platform)
>    + SAM9G20 (platform)
>    + SAM9RL64 (platform)
>    + Other Non-AT91 chip (platform)
>    + SAM9G20-EK (board)
>    + SAM99RL-EK (board
>    ...etc.
> + AT91 (I/O support package)
>
> Or maybe I'm just confused about the relationship between arch, var,
> and plf?

You wouldn't be the first. :)

-- 
Grant Edwards               grant.b.edwards        Yow! Look into my eyes and
                                  at               try to forget that you have
                              gmail.com            a Macy's charge card!