diff mbox series

[v2] board: stemmy: Increase boot image to 64 MB

Message ID 20211120225542.2971144-1-linus.walleij@linaro.org
State Accepted
Commit cc4dcd488cd339f931fa97dcda30b94a3d10a302
Headers show
Series [v2] board: stemmy: Increase boot image to 64 MB | expand

Commit Message

Linus Walleij Nov. 20, 2021, 10:55 p.m. UTC
When using a recent kernel with a bunch of compiled-in
stuff the kernel image easily becomes bigger than 8 MB
yielding this error:

  Loading Kernel Image
  Image too large: increase CONFIG_SYS_BOOTM_LEN
  Must RESET board to recover

Fix this by bumping to SZ_64MB.

Cc: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ChangeLog v1->v2:
- Bump to 64MB instead of 16MB
---
 include/configs/stemmy.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Tom Rini Nov. 22, 2021, 1:22 p.m. UTC | #1
On Sat, Nov 20, 2021 at 11:55:42PM +0100, Linus Walleij wrote:

> When using a recent kernel with a bunch of compiled-in
> stuff the kernel image easily becomes bigger than 8 MB
> yielding this error:
> 
>   Loading Kernel Image
>   Image too large: increase CONFIG_SYS_BOOTM_LEN
>   Must RESET board to recover
> 
> Fix this by bumping to SZ_64MB.
> 
> Cc: Stephan Gerhold <stephan@gerhold.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Reviewed-by: Tom Rini <trini@konsulko.com>
Stephan Gerhold Nov. 23, 2021, 1:15 p.m. UTC | #2
On Sat, Nov 20, 2021 at 11:55:42PM +0100, Linus Walleij wrote:
> When using a recent kernel with a bunch of compiled-in
> stuff the kernel image easily becomes bigger than 8 MB
> yielding this error:
> 
>   Loading Kernel Image
>   Image too large: increase CONFIG_SYS_BOOTM_LEN
>   Must RESET board to recover
> 
> Fix this by bumping to SZ_64MB.
> 
> Cc: Stephan Gerhold <stephan@gerhold.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Great, thanks for changing it! :)

Reviewed-by: Stephan Gerhold <stephan@gerhold.net>

> ---
> ChangeLog v1->v2:
> - Bump to 64MB instead of 16MB
> ---
>  include/configs/stemmy.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/configs/stemmy.h b/include/configs/stemmy.h
> index ed79b1203901..4bba2a627259 100644
> --- a/include/configs/stemmy.h
> +++ b/include/configs/stemmy.h
> @@ -14,6 +14,7 @@
>   * bootloader. New images are loaded at the same address for compatibility.
>   */
>  #define CONFIG_SYS_INIT_SP_ADDR		CONFIG_SYS_TEXT_BASE
> +#define CONFIG_SYS_BOOTM_LEN		SZ_64M
>  
>  /* FIXME: This should be loaded from device tree... */
>  #define CONFIG_SYS_L2_PL310
> -- 
> 2.31.1
>
Tom Rini Jan. 7, 2022, 5:04 p.m. UTC | #3
On Sat, Nov 20, 2021 at 11:55:42PM +0100, Linus Walleij wrote:

> When using a recent kernel with a bunch of compiled-in
> stuff the kernel image easily becomes bigger than 8 MB
> yielding this error:
> 
>   Loading Kernel Image
>   Image too large: increase CONFIG_SYS_BOOTM_LEN
>   Must RESET board to recover
> 
> Fix this by bumping to SZ_64MB.
> 
> Cc: Stephan Gerhold <stephan@gerhold.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Stephan Gerhold <stephan@gerhold.net>

Applied to u-boot/next, thanks!
diff mbox series

Patch

diff --git a/include/configs/stemmy.h b/include/configs/stemmy.h
index ed79b1203901..4bba2a627259 100644
--- a/include/configs/stemmy.h
+++ b/include/configs/stemmy.h
@@ -14,6 +14,7 @@ 
  * bootloader. New images are loaded at the same address for compatibility.
  */
 #define CONFIG_SYS_INIT_SP_ADDR		CONFIG_SYS_TEXT_BASE
+#define CONFIG_SYS_BOOTM_LEN		SZ_64M
 
 /* FIXME: This should be loaded from device tree... */
 #define CONFIG_SYS_L2_PL310