diff mbox series

[U-Boot] exynos5: allow bigger BL2 (u-boot.bin)

Message ID 20181119164503.10977-1-guillaume.gardet@free.fr
State Changes Requested
Delegated to: Tom Rini
Headers show
Series [U-Boot] exynos5: allow bigger BL2 (u-boot.bin) | expand

Commit Message

Guillaume GARDET Nov. 19, 2018, 4:45 p.m. UTC
This patch allows SPL to load a bigger BL2 (u-boot.bin).
This is needed for and has been tested on Arndale board, as u-boot.bin is 
now bigger than 512K with GCC8.

Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>

Cc: Minkyu Kang <mk7.kang@samsung.com>
Cc: Tom Rini <trini@konsulko.com>
---
 include/configs/exynos5-common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini Nov. 19, 2018, 4:46 p.m. UTC | #1
On Mon, Nov 19, 2018 at 05:45:03PM +0100, Guillaume GARDET wrote:

> This patch allows SPL to load a bigger BL2 (u-boot.bin).
> This is needed for and has been tested on Arndale board, as u-boot.bin is 
> now bigger than 512K with GCC8.
> 
> Signed-off-by: Guillaume GARDET <guillaume.gardet@free.fr>
> 
> Cc: Minkyu Kang <mk7.kang@samsung.com>
> Cc: Tom Rini <trini@konsulko.com>
> ---
>  include/configs/exynos5-common.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Are these platforms using thumb2 to build?  Were it not for some early
Cortex-A platforms with thumb2 errata I'd be strongly inclined to do a
global switch to making that option default.
diff mbox series

Patch

diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-common.h
index cd2a9046af..7da4be5ca8 100644
--- a/include/configs/exynos5-common.h
+++ b/include/configs/exynos5-common.h
@@ -89,7 +89,7 @@ 
 /* Configuration of BL1, BL2, ENV Blocks on mmc */
 #define CONFIG_RES_BLOCK_SIZE	(512)
 #define CONFIG_BL1_SIZE	(16 << 10) /*16 K reserved for BL1*/
-#define CONFIG_BL2_SIZE	(512UL << 10UL) /* 512 KB */
+#define CONFIG_BL2_SIZE	(512UL << 11UL) /* 1 MB */
 #define CONFIG_ENV_SIZE	(16 << 10) /* 16 KB */
 
 #define CONFIG_BL1_OFFSET	(CONFIG_RES_BLOCK_SIZE + CONFIG_SEC_FW_SIZE)