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

Albert ARIBAUD
2010-10-26T06:31:28Z

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.