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

[Amit Virdi]

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_SYS_MONITOR_BASE			CONFIG_SYS_TEXT_BASE
>>>> +#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