[U-Boot] ALERT! >90% of all board configurations BROKEN

[Albert ARIBAUD]

Le 26/10/2010 02:11, Reinhard Meyer a écrit :

> That is #ifdef-ed away in case of ARM-relocation. Perhaps we should remove
> all code that pertains to "WITHOUT_RELOC"... Would make the rest of the code
> less obscure...

> I changed my board.config like this:
> ...
> /*#define CONFIG_SYS_GBL_DATA_SIZE 128*/	/* 128 bytes for initial data */
> ...
> #define	CONFIG_SYS_INIT_SP_ADDR		(CONFIG_SYS_SDRAM_BASE + 0x1000 /*- CONFIG_SYS_GBL_DATA_SIZE*/)
> ...
> and it still compiles. The second line is the only use of GBL_DATA_SIZE,
> since that is often set to somewhere in SRAM, it is still ok - even if
> set to the end of SRAM it will most likely wrap into the repeated SRAM
> image on most SoCs.

Not all SoCs have SRAM. While the kirkwood family always has it, as does 
the Marvell 5182, the 5281 doesn't, for instance.

Amicalement,
-- 
Albert.