From mboxrd@z Thu Jan 1 00:00:00 1970 From: Prabhakar Kushwaha Date: Thu, 8 Sep 2016 06:35:13 +0000 Subject: [U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL In-Reply-To: References: <1473243693-37360-1-git-send-email-Qianyu.Gong@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Gong, You have increased SPL size to 124KB. It is very high. As per my understanding 40K has been reserved for headers to be used during secure boot. Have you consider the secure boot case? Regards, Prabhakar > -----Original Message----- > From: Q.Y. Gong > Sent: Thursday, September 08, 2016 8:04 AM > To: york sun ; u-boot at lists.denx.de > Cc: Prabhakar Kushwaha ; Vincent Hu > > Subject: RE: [PATCH] armv8: ls1043a: Extend the size for SPL > > Hi York, > > > -----Original Message----- > > From: york sun > > Sent: Wednesday, September 07, 2016 11:36 PM > > To: Q.Y. Gong ; u-boot at lists.denx.de > > Cc: Prabhakar Kushwaha ; Vincent Hu > > > > 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 > > > --- > > > 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