[U-Boot] [PATCH 3/4] Exynos5: Config: Increase SPL footprint for Exynos5420

[Simon Glass]

Hi Akshay,

On 30 May 2014 07:45, Akshay Saraswat <akshay.s at samsung.com> wrote:
> Max footprint for SPL in both Exynos 5250 and 5420 is limited to 14 KB.
> For Exynos5250 we need to keep it 14 KB because BL1 supports only fixed
> size SPL downloading. But in case of Exynos5420 we need not restrict it
> to 14 KB. And also, the SPL size for Exynos5420 is expected to increase
> with the upcoming patches and the patches under review right now.
>
> Signed-off-by: Akshay Saraswat <akshay.s at samsung.com>
> ---
>  include/configs/exynos5-dt.h    | 2 --
>  include/configs/exynos5250-dt.h | 2 ++
>  include/configs/exynos5420.h    | 2 ++
>  3 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/include/configs/exynos5-dt.h b/include/configs/exynos5-dt.h
> index 60b99a3..9900baf 100644
> --- a/include/configs/exynos5-dt.h
> +++ b/include/configs/exynos5-dt.h
> @@ -144,8 +144,6 @@
>
>  /* specific .lds file */
>  #define CONFIG_SPL_LDSCRIPT    "board/samsung/common/exynos-uboot-spl.lds"
> -#define CONFIG_SPL_MAX_FOOTPRINT       (14 * 1024)
> -
>
>  /* Miscellaneous configurable options */
>  #define CONFIG_SYS_LONGHELP            /* undef to save memory */
> diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
> index b7ff472..9fff1af 100644
> --- a/include/configs/exynos5250-dt.h
> +++ b/include/configs/exynos5250-dt.h
> @@ -20,6 +20,8 @@
>  #define MACH_TYPE_SMDK5250             3774
>  #define CONFIG_MACH_TYPE               MACH_TYPE_SMDK5250
>
> +#define CONFIG_SPL_MAX_FOOTPRINT       (14 * 1024)
> +
>  /* USB */
>  #define CONFIG_CMD_USB
>  #define CONFIG_USB_XHCI
> diff --git a/include/configs/exynos5420.h b/include/configs/exynos5420.h
> index 3a28bbc..fd25820 100644
> --- a/include/configs/exynos5420.h
> +++ b/include/configs/exynos5420.h
> @@ -25,6 +25,8 @@
>  #endif
>  #define CONFIG_IRAM_TOP                        0x02074000
>
> +#define CONFIG_SPL_MAX_FOOTPRINT       (32 * 1024)

Should this be 30? I think normally there is a 2KB space after it, so
we use either 14KB or 30KB.

> +
>  #define CONFIG_DEVICE_TREE_LIST "exynos5420-peach-pit exynos5420-smdk5420"
>
>  #define CONFIG_MAX_I2C_NUM     11
> --
> 1.7.12.4
>

Regards,
Simon