[Bug 1000819] Add support for Atmel AT91SAM9263

[bugzilla-daemon@bugs.ecos.sourceware.org]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1000819

John Dallaway <john@dallaway.org.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
         AssignedTo|backlog@bugs.ecos.sourcewar |john@dallaway.org.uk
                   |e.org                       |
     Ever Confirmed|0                           |1

--- Comment #11 from John Dallaway <john@dallaway.org.uk> 2011-03-16 12:05:52 GMT ---
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.

As it stands, this patch should only affect AT91 targets. The risk of breaking
other AT91 target support is small since there does not appear to be any
changes to the macros themselves. There is a risk of breaking compatibility
with non-contributed ports to AT91SAM7 hardware but the effort in fixing such
breakage should be limited to adding the new ARM7 package to the target
description in ecos.db. I don't have a problem with this.

Is there any other code in the existing AT91 variant HAL which might not be
appropriate for AT91SAM9? Any other comments on patch 1?

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.