[PATCH 10] ARM: Add support for IXP4xx CPU and for Goramo Multilink router platform.

[Sascha Hauer]

On Wed, Dec 22, 2010 at 01:57:22AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > +
> > +#define BAREBOX_START        0x00000
> > +#define BAREBOX_LENGTH       0x34000
> > +#define NPE_A_START          (BAREBOX_START + BAREBOX_LENGTH)
> > +#define NPE_A_LENGTH         0x05000
> > +#define NPE_B_START          (NPE_A_START + NPE_A_LENGTH)
> > +#define NPE_B_LENGTH         0x03000
> > +#define NPE_C_START          (NPE_B_START + NPE_B_LENGTH)
> > +#define NPE_C_LENGTH         0x04000
> > +#define NPE_ENV0_START       (NPE_C_START + NPE_C_LENGTH)
> > +#define NPE_ENV0_LENGTH      0x20000
> why not put the NPE firmware in the env?

That's a good idea. This way we can tell the users to put the firmware
to arch/arm/boards/.../env/firmware before compilation and avoid
redistribution problems if there are any. Another plus is that the
barebox binary is not larger than it has to be.
The only downside is that the environment gets bigger as it duplicates
the firmware, but even this could be avoided if we add a second (non
environment) filesystem image.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |