[U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL

[Q.Y. Gong]

Hi York,

> -----Original Message-----
> From: york sun
> Sent: Wednesday, September 07, 2016 11:36 PM
> To: Q.Y. Gong <qianyu.gong at nxp.com>; u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Vincent Hu
> <mingkai.hu at nxp.com>
> Subject: Re: [PATCH] armv8: ls1043a: Extend the size for SPL
> 
> On 09/07/2016 03:33 AM, Gong Qianyu wrote:
> > The SPL images are growing much bigger especially when DEBUG is ON.
> > So need to fix the values for them.
> >
> > Signed-off-by: Gong Qianyu <Qianyu.Gong at nxp.com>
> > ---
> >  include/configs/ls1043a_common.h | 25 ++++++++++++++++---------
> >  1 file changed, 16 insertions(+), 9 deletions(-)
> >
> > diff --git a/include/configs/ls1043a_common.h
> > b/include/configs/ls1043a_common.h
> > index e55fcb2..fa20e6d 100644
> > --- a/include/configs/ls1043a_common.h
> > +++ b/include/configs/ls1043a_common.h
> > @@ -69,16 +69,22 @@
> >  #define CONFIG_SPL_SERIAL_SUPPORT
> >  #define CONFIG_SPL_DRIVERS_MISC_SUPPORT  #define
> > CONFIG_SPL_MMC_SUPPORT
> > -#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
> 	0xf0
> > +#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
> 	0x110
> >  #define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS		0x500
> >
> >  #define CONFIG_SPL_TEXT_BASE		0x10000000
> > -#define CONFIG_SPL_MAX_SIZE		0x1d000
> > -#define CONFIG_SPL_STACK		0x1001e000
> > -#define CONFIG_SPL_PAD_TO		0x1d000
> > +/*
> > + * CONFIG_SPL_MAX_SIZE is limited by OCRAM memory size(128 KiB) and
> > + * a reserved stack size(4 KiB).
> > + * So notice that even if DEBUG is ON, the SPL
> > +image(spl/u-boot-spl.bin)
> > + * should not be > 124 KiB.
> > + */
> 
> Qianyu,
> 
> It is good to see comment here. However, I am concerned about your calculation.
> Beside the image and stack, the early MMU tables are at the beginning of OCRAM.
> Did you count them?
> 
> York

No. We don't set up MMU tables in SPL stage on LS1043A/LS1046A.
Seems so far only LS2080A has done that in SPL.


Regards,
Qianyu