[U-Boot] [PATCH 1/2] mx51evk: support new relocation scheme

[Stefano Babic]

On 10/26/2010 12:03 PM, Heiko Schocher wrote:
> Hello Wolfgang,
> 
> Wolfgang Denk schrieb:
>> Dear Heiko Schocher,
>>
>> In message <4CC693FE.4090500 at denx.de> you wrote:
>>>>> Don;t know if IRAM_BASE_SIZE exists ...
>>>> we must define it. I set it in the board configuration file, but it is
>>>> wrong. The value should be moved into register definition file for the
>>>> SoC (imx-regs.h)
>>> Full-Ack. This is a SoC define, not board dependend.
>>
>> It may be board dependent - for exmaple, when there is some large and
>> conveniently usable SRAM on a board, you might want to use that
>> instead of any limited area on the SoC.
> 
> IRAM_BASE_SIZE is SoC dependend! You maybe mean CONFIG_SYS_INIT_SP_ADDR
> which can be of course board dependend.

Agree. IRAM_BASE_SIZE and IRAM_BASE are SoC dependent and they must be
set in the register definition file for the MX.51. At the moment,
IRAM_BASE is set with a wrong value (that is a smaller IRAM is actually
defined as the one processor has) and IRAM_BASE_SIZE is not set, but is
derived in board configuration file (both things must be corrected,
anyway).

Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================