diff mbox series

[v1] board: toradex: verdin-am62_r5: Increase SPL_STACK_R_MALLOC_SIMPLE_LEN

Message ID 20240320164845.141894-1-hiagofranco@gmail.com
State Accepted
Commit bd0aedde3ea3691616c17c720e2d25351308c0a1
Delegated to: Tom Rini
Headers show
Series [v1] board: toradex: verdin-am62_r5: Increase SPL_STACK_R_MALLOC_SIMPLE_LEN | expand

Commit Message

Hiago De Franco March 20, 2024, 4:48 p.m. UTC 
From: Hiago De Franco <hiago.franco@toradex.com>

Increase the SPL_STACK_R_MALLOC_SIMPLE_LEN to 0x200000 to accommodate the
size of tispl.bin fit image. With the recent upgrade of ti-linux-firmware
from version v9.1.0 to v9.2.5, the size of tispl.bin fit image has
increased to 1.4MB, causing allocation errors in the R5 SPL:

```
alloc space exhausted
Could not get FIT buffer of 1325056 bytes
        check CONFIG_SPL_SYS_MALLOC_SIZE
```

Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
---
 configs/verdin-am62_r5_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Francesco Dolcini March 21, 2024, 5:07 p.m. UTC | #1 
On Wed, Mar 20, 2024 at 01:48:45PM -0300, Hiago De Franco wrote:
> From: Hiago De Franco <hiago.franco@toradex.com>
> 
> Increase the SPL_STACK_R_MALLOC_SIMPLE_LEN to 0x200000 to accommodate the
> size of tispl.bin fit image. With the recent upgrade of ti-linux-firmware
> from version v9.1.0 to v9.2.5, the size of tispl.bin fit image has
> increased to 1.4MB, causing allocation errors in the R5 SPL:
> 
> ```
> alloc space exhausted
> Could not get FIT buffer of 1325056 bytes
>         check CONFIG_SPL_SYS_MALLOC_SIZE
> ```
> 
> Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
Reviewed-by: Francesco Dolcini <francesco.dolcini@toradex.com>


Tom, this is also a candidate for master, thanks :-)

Francesco
Tom Rini March 21, 2024, 7:15 p.m. UTC | #2 
On Thu, Mar 21, 2024 at 06:07:32PM +0100, Francesco Dolcini wrote:
> On Wed, Mar 20, 2024 at 01:48:45PM -0300, Hiago De Franco wrote:
> > From: Hiago De Franco <hiago.franco@toradex.com>
> > 
> > Increase the SPL_STACK_R_MALLOC_SIMPLE_LEN to 0x200000 to accommodate the
> > size of tispl.bin fit image. With the recent upgrade of ti-linux-firmware
> > from version v9.1.0 to v9.2.5, the size of tispl.bin fit image has
> > increased to 1.4MB, causing allocation errors in the R5 SPL:
> > 
> > ```
> > alloc space exhausted
> > Could not get FIT buffer of 1325056 bytes
> >         check CONFIG_SPL_SYS_MALLOC_SIZE
> > ```
> > 
> > Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
> Reviewed-by: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> 
> Tom, this is also a candidate for master, thanks :-)

OK, thanks.
Tom Rini March 22, 2024, 7:26 p.m. UTC | #3 
On Wed, 20 Mar 2024 13:48:45 -0300, Hiago De Franco wrote:

> Increase the SPL_STACK_R_MALLOC_SIMPLE_LEN to 0x200000 to accommodate the
> size of tispl.bin fit image. With the recent upgrade of ti-linux-firmware
> from version v9.1.0 to v9.2.5, the size of tispl.bin fit image has
> increased to 1.4MB, causing allocation errors in the R5 SPL:
> 
> ```
> alloc space exhausted
> Could not get FIT buffer of 1325056 bytes
>         check CONFIG_SPL_SYS_MALLOC_SIZE
> ```
> 
> [...]

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

Patch

diff --git a/configs/verdin-am62_r5_defconfig b/configs/verdin-am62_r5_defconfig
index a3febdcb6941..5b4b81c925a0 100644
--- a/configs/verdin-am62_r5_defconfig
+++ b/configs/verdin-am62_r5_defconfig
@@ -35,7 +35,7 @@  CONFIG_SPL_BSS_MAX_SIZE=0x3000
 CONFIG_SPL_SYS_REPORT_STACK_F_USAGE=y
 CONFIG_SPL_SYS_MALLOC_SIMPLE=y
 CONFIG_SPL_STACK_R=y
-CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x140000
+CONFIG_SPL_STACK_R_MALLOC_SIMPLE_LEN=0x200000
 CONFIG_SPL_SEPARATE_BSS=y
 CONFIG_SPL_SYS_MALLOC=y
 CONFIG_SPL_HAS_CUSTOM_MALLOC_START=y