All of lore.kernel.org
 help / color / mirror / Atom feed
From: york sun <york.sun@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL
Date: Wed, 7 Sep 2016 15:35:42 +0000	[thread overview]
Message-ID: <AM4PR0401MB17328DE6F0CE4F611B5A6A229AF80@AM4PR0401MB1732.eurprd04.prod.outlook.com> (raw)
In-Reply-To: 1473243693-37360-1-git-send-email-Qianyu.Gong@nxp.com

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

  reply	other threads:[~2016-09-07 15:35 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 [this message]
2016-09-08  2:33   ` Q.Y. Gong
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=AM4PR0401MB17328DE6F0CE4F611B5A6A229AF80@AM4PR0401MB1732.eurprd04.prod.outlook.com \
    --to=york.sun@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.