From: Q.Y. Gong <qianyu.gong@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL
Date: Thu, 8 Sep 2016 02:33:54 +0000 [thread overview]
Message-ID: <VI1PR04MB211126F25FBCD9763B7E55ABE7FB0@VI1PR04MB2111.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <AM4PR0401MB17328DE6F0CE4F611B5A6A229AF80@AM4PR0401MB1732.eurprd04.prod.outlook.com>
Hi York,
> -----Original Message-----
> From: york sun
> Sent: Wednesday, September 07, 2016 11:36 PM
> To: Q.Y. Gong <qianyu.gong@nxp.com>; u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; Vincent Hu
> <mingkai.hu@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@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
next prev parent reply other threads:[~2016-09-08 2:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-07 10:21 [U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL Gong Qianyu
2016-09-07 15:35 ` york sun
2016-09-08 2:33 ` Q.Y. Gong [this message]
2016-09-08 6:35 ` Prabhakar Kushwaha
2016-09-08 7:01 ` Q.Y. Gong
2016-09-08 12:47 ` Ruchika Gupta
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=VI1PR04MB211126F25FBCD9763B7E55ABE7FB0@VI1PR04MB2111.eurprd04.prod.outlook.com \
--to=qianyu.gong@nxp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.