Re: [PATCH 1/2] config: nxp: bk4: Increase size of CONFIG_SYS_MALLOC_F_LEN (to 0x1000)

[Lukasz Majewski <lukma@denx.de>]

[-- Attachment #1: Type: text/plain, Size: 2821 bytes --]

Hi Fabio,

> Hi Lukasz,
> 
> On Tue, Jun 24, 2025 at 5:47 PM Lukasz Majewski <lukma@denx.de> wrote:
> >
> > The commit c69103218ee4 ("i2c: mxc_i2c: add DM_FLAG_PRE_RELOC flag")
> > has enabled by default the i2c initialization in the pre-relocation
> > phase.
> > It turned out that vf610 based BK4 board had too small
> > SYS_MALLOC_F_LEN pool size.
> >
> > After increasing it to 0x1000 the board boots again.
> >
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>  
> 
> It seems other Vybrid boards also have the same issue.
> 
> What about removing CONFIG_SYS_MALLOC_F_LEN from all the Vybrid boards
> so that the default of 0x2000 is used?
> 

Good idea.

I will check if this can be safely done - for some reason smaller
values were used on Vybrid boards.

> diff --git a/configs/bk4r1_defconfig b/configs/bk4r1_defconfig
> index 2b72ec916327..82ebf704c605 100644
> --- a/configs/bk4r1_defconfig
> +++ b/configs/bk4r1_defconfig
> @@ -4,7 +4,6 @@ CONFIG_SYS_THUMB_BUILD=y
>  CONFIG_ARCH_VF610=y
>  CONFIG_TEXT_BASE=0x3f401000
>  CONFIG_SYS_MALLOC_LEN=0x402000
> -CONFIG_SYS_MALLOC_F_LEN=0x800
>  CONFIG_NR_DRAM_BANKS=1
>  CONFIG_ENV_SIZE=0x2000
>  CONFIG_ENV_OFFSET=0x200000
> diff --git a/configs/pcm052_defconfig b/configs/pcm052_defconfig
> index d0d558ab11ad..81d703ec679d 100644
> --- a/configs/pcm052_defconfig
> +++ b/configs/pcm052_defconfig
> @@ -4,7 +4,6 @@ CONFIG_SYS_THUMB_BUILD=y
>  CONFIG_ARCH_VF610=y
>  CONFIG_TEXT_BASE=0x3f401000
>  CONFIG_SYS_MALLOC_LEN=0x202000
> -CONFIG_SYS_MALLOC_F_LEN=0x400
>  CONFIG_NR_DRAM_BANKS=1
>  CONFIG_ENV_SIZE=0x2000
>  CONFIG_ENV_OFFSET=0xA0000
> diff --git a/configs/vf610twr_defconfig b/configs/vf610twr_defconfig
> index c823cb0a30ee..8129484513dd 100644
> --- a/configs/vf610twr_defconfig
> +++ b/configs/vf610twr_defconfig
> @@ -4,7 +4,6 @@ CONFIG_SYS_THUMB_BUILD=y
>  CONFIG_ARCH_VF610=y
>  CONFIG_TEXT_BASE=0x3f401000
>  CONFIG_SYS_MALLOC_LEN=0x202000
> -CONFIG_SYS_MALLOC_F_LEN=0x400
>  CONFIG_NR_DRAM_BANKS=1
>  CONFIG_ENV_SIZE=0x2000
>  CONFIG_ENV_OFFSET=0xC0000
> diff --git a/configs/vf610twr_nand_defconfig
> b/configs/vf610twr_nand_defconfig index 84e73c0c6657..51365990d0fb
> 100644 --- a/configs/vf610twr_nand_defconfig
> +++ b/configs/vf610twr_nand_defconfig
> @@ -4,7 +4,6 @@ CONFIG_SYS_THUMB_BUILD=y
>  CONFIG_ARCH_VF610=y
>  CONFIG_TEXT_BASE=0x3f401000
>  CONFIG_SYS_MALLOC_LEN=0x0220000
> -CONFIG_SYS_MALLOC_F_LEN=0x400
>  CONFIG_NR_DRAM_BANKS=1
>  CONFIG_ENV_SIZE=0x20000
>  CONFIG_ENV_OFFSET=0x180000




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]