| Message ID | 20210706234301.4039318-1-ricardo@foundries.io |
|---|---|
| State | Accepted |
| Commit | 677a24cbc3ef47bef355df45eff289d2dc371df0 |
| Delegated to: | Michal Simek |
| Headers | show |
| Series | xilinx: zynqmp: increase CONFIG_SYS_SPL_MALLOC_SIZE to 16MB | expand |
On 7/7/21 1:43 AM, Ricardo Salveti wrote: > commit 03f1f78a9b44 ("spl: fit: Prefer a malloc()'d buffer for loading > images")' changed the way buffer allocation worked for SPL to a more > flexible method. > > For xilinx zynqmp the 1MB buffer is not necessarily enough when dealing > with complex fit images (e.g. containing FPGA/TF-A/OP-TEE/U-Boot > proper), which can easily reach up to 10MB, so increase the default > CONFIG_SYS_SPL_MALLOC_SIZE size to 16MB to cover more advanced > scenarios. > > Signed-off-by: Ricardo Salveti <ricardo@foundries.io> > --- > include/configs/xilinx_zynqmp.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h > index cadaf1a9631..bfe8a204279 100644 > --- a/include/configs/xilinx_zynqmp.h > +++ b/include/configs/xilinx_zynqmp.h > @@ -261,7 +261,7 @@ > #endif > > #define CONFIG_SYS_SPL_MALLOC_START 0x20000000 > -#define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 > +#define CONFIG_SYS_SPL_MALLOC_SIZE 0x1000000 > > #ifdef CONFIG_SPL_SYS_MALLOC_SIMPLE > # error "Disable CONFIG_SPL_SYS_MALLOC_SIMPLE. Full malloc needs to be used" > Applied. M
diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h index cadaf1a9631..bfe8a204279 100644 --- a/include/configs/xilinx_zynqmp.h +++ b/include/configs/xilinx_zynqmp.h @@ -261,7 +261,7 @@ #endif #define CONFIG_SYS_SPL_MALLOC_START 0x20000000 -#define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 +#define CONFIG_SYS_SPL_MALLOC_SIZE 0x1000000 #ifdef CONFIG_SPL_SYS_MALLOC_SIMPLE # error "Disable CONFIG_SPL_SYS_MALLOC_SIMPLE. Full malloc needs to be used"
commit 03f1f78a9b44 ("spl: fit: Prefer a malloc()'d buffer for loading images")' changed the way buffer allocation worked for SPL to a more flexible method. For xilinx zynqmp the 1MB buffer is not necessarily enough when dealing with complex fit images (e.g. containing FPGA/TF-A/OP-TEE/U-Boot proper), which can easily reach up to 10MB, so increase the default CONFIG_SYS_SPL_MALLOC_SIZE size to 16MB to cover more advanced scenarios. Signed-off-by: Ricardo Salveti <ricardo@foundries.io> --- include/configs/xilinx_zynqmp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)