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



_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to