From mboxrd@z Thu Jan  1 00:00:00 1970
From: Amit Virdi <amit.virdi@st.com>
Date: Tue, 27 Mar 2012 12:08:18 +0530
Subject: [U-Boot] [PATCH 18/25] SPEAr: Correct the definition of
	CONFIG_SYS_MONITOR_BASE
In-Reply-To: <201203071531.06350.sr@denx.de>
References: <1331121854-20494-1-git-send-email-amit.virdi@st.com>
	<1331121854-20494-19-git-send-email-amit.virdi@st.com>
	<201203071531.06350.sr@denx.de>
Message-ID: <4F71605A.1070907@st.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Stefan,

>>
>>
>>   #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.

Thanks
Amit Virdi