From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Tue, 26 Oct 2010 08:31:28 +0200 Subject: [U-Boot] ALERT! >90% of all board configurations BROKEN In-Reply-To: <4CC61C96.9010806@emk-elektronik.de> References: <20101025200019.B5BBC152451@gemini.denx.de> <4CC5E865.70003@emk-elektronik.de> <20101025220712.35FCC135C64@gemini.denx.de> <4CC61C96.9010806@emk-elektronik.de> Message-ID: <4CC675C0.8030203@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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.