
On 4/17/2014 5:37 AM, York Sun wrote:
On 04/02/2014 09:02 PM, Prabhakar Kushwaha wrote:
Hi All,
I just want to share my concern with respect to this patch. This patch removed hard-coded value for 0x80000 and use CONFIG_SYS_MONITOR_LEN.
There were many Freescale boards config which don’t have correct CONFIG_SYS_MONITOR_LEN(due to size increase from 512KB to 768KB). I fixed them in subsequent patch.
This linker file is being used by many other powerpc boards/platforms. Because of hard-coded value of 0x80000 in linker file, there boards are booting till now. but after this patch, there is a possibility there boards may not boot because of wrong value defined for CONFIG_SYS_MONITOR_LEN by board maintainers.
I will request powerpc board maintainers to check config file for correct value of CONFIG_SYS_MONITOR_LEN.
Prabhakar,
Where are we for this patch? I don't see any comment.
I did not get any comments :(
Looks like every one is OK with it.
regards, Prabhakar