[U-Boot] [PATCH 18/25] SPEAr: Correct the definition of CONFIG_SYS_MONITOR_BASE

Amit Virdi amit.virdi at st.com
Tue Mar 27 09:42:48 CEST 2012

On 3/27/2012 12:35 PM, Stefan Roese wrote:
> Amit,
> On Tuesday 27 March 2012 08:38:18 Amit Virdi wrote:
>>>>    #define CONFIG_ENV_SIZE				0x02000
>>>> +#define CONFIG_MONITOR_IS_IN_RAM		1
>>> Why is CONFIG_MONITOR_IS_IN_RAM defined? And if really needed, please
>>> without the 1.
>> CONFIG_MONITOR_IS_IN_RAM is defined with the understanding that since,
>> u-boot is loaded by the BootROM (embedded in eROM) to RAM, so the
>> monitor is present in RAM. Please confirm if this understanding is correct.
> Hmmm, seems to be used on some Coldfire and Blackfin platforms. Its not needed
> for your platform. Please remove it.

Ok. Thanks for the clarification!

Best Regards
Amit Virdi

More information about the U-Boot mailing list