[PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com --- configs/imx8mp_evk_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig index cd5724e811ab..c34a10c2d906 100644 --- a/configs/imx8mp_evk_defconfig +++ b/configs/imx8mp_evk_defconfig @@ -4,7 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x40200000 CONFIG_SPL_GPIO_SUPPORT=y CONFIG_SPL_LIBCOMMON_SUPPORT=y CONFIG_SPL_LIBGENERIC_SUPPORT=y -CONFIG_SYS_MALLOC_F_LEN=0x8000 +CONFIG_SYS_MALLOC_F_LEN=0x10000 CONFIG_ENV_SIZE=0x1000 CONFIG_ENV_OFFSET=0x400000 CONFIG_SYS_I2C_MXC_I2C1=y
Hello Fabio,
-----Original Message----- From: Fabio Estevam festevam@gmail.com Sent: Monday, December 21, 2020 7:41 PM To: sbabic@denx.de Cc: peng.fan@nxp.com; u-boot@lists.denx.de; ZHIZHIKIN Andrey andrey.zhizhikin@leica-geosystems.com; uboot-imx@nxp.com; Fabio Estevam festevam@gmail.com Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
It would be really great to get a review from Peng here, as his commit 28fff3fa1c ("imx: imx8mp_evk: fix boot issue") changed exactly this option from 0x10000 to 0x8000.
I've tested my local build with your patch and it boots OK, hence I cannot seem to find a reasoning why the early malloc area size has been halved with above commit.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com
configs/imx8mp_evk_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig index cd5724e811ab..c34a10c2d906 100644 --- a/configs/imx8mp_evk_defconfig +++ b/configs/imx8mp_evk_defconfig @@ -4,7 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x40200000 CONFIG_SPL_GPIO_SUPPORT=y CONFIG_SPL_LIBCOMMON_SUPPORT=y CONFIG_SPL_LIBGENERIC_SUPPORT=y -CONFIG_SYS_MALLOC_F_LEN=0x8000 +CONFIG_SYS_MALLOC_F_LEN=0x10000 CONFIG_ENV_SIZE=0x1000 CONFIG_ENV_OFFSET=0x400000 CONFIG_SYS_I2C_MXC_I2C1=y -- 2.17.1
Tested-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com
Hi Andrey,
On Mon, Dec 21, 2020 at 7:04 PM ZHIZHIKIN Andrey andrey.zhizhikin@leica-geosystems.com wrote:
It would be really great to get a review from Peng here, as his commit 28fff3fa1c ("imx: imx8mp_evk: fix boot issue") changed exactly this option from 0x10000 to 0x8000.
Yes, agreed.
Peng,
Please let us know if you are happy with this change.
I've tested my local build with your patch and it boots OK, hence I cannot seem to find a reasoning why the early malloc area size has been halved with above commit.
Tested-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com
Thanks for reporting and testing it.
Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com
Reviewed-by: Peng Fan peng.fan@nxp.com
I think it might be the DM clk exhausted the space, we need find a way to improve that.
Thanks, Peng.
configs/imx8mp_evk_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig index cd5724e811ab..c34a10c2d906 100644 --- a/configs/imx8mp_evk_defconfig +++ b/configs/imx8mp_evk_defconfig @@ -4,7 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x40200000 CONFIG_SPL_GPIO_SUPPORT=y CONFIG_SPL_LIBCOMMON_SUPPORT=y CONFIG_SPL_LIBGENERIC_SUPPORT=y -CONFIG_SYS_MALLOC_F_LEN=0x8000 +CONFIG_SYS_MALLOC_F_LEN=0x10000 CONFIG_ENV_SIZE=0x1000 CONFIG_ENV_OFFSET=0x400000 CONFIG_SYS_I2C_MXC_I2C1=y -- 2.17.1
Hi Tom and Stefano,
On Tue, Dec 22, 2020 at 3:01 AM Peng Fan peng.fan@nxp.com wrote:
Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com
Reviewed-by: Peng Fan peng.fan@nxp.com
Please consider this one for 2021.01.
Thanks
On 22.12.20 11:59, Fabio Estevam wrote:
Hi Tom and Stefano,
On Tue, Dec 22, 2020 at 3:01 AM Peng Fan peng.fan@nxp.com wrote:
Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com
Reviewed-by: Peng Fan peng.fan@nxp.com
Please consider this one for 2021.01.
Yes, I pick it up.
Regards, Stefano
Thanks
All,
Subject: RE: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
I am thinking to remove the CLK CCF support for i.MX8M, and use simple CLK DM. Do you agree?
Thanks, Peng.
Subject: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
When booting imx8mp-evk the following allocation error message is seen:
U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300)
alloc space exhausted
Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards.
Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com
Reviewed-by: Peng Fan peng.fan@nxp.com
I think it might be the DM clk exhausted the space, we need find a way to improve that.
Thanks, Peng.
configs/imx8mp_evk_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configs/imx8mp_evk_defconfig b/configs/imx8mp_evk_defconfig index cd5724e811ab..c34a10c2d906
100644
--- a/configs/imx8mp_evk_defconfig +++ b/configs/imx8mp_evk_defconfig @@ -4,7 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x40200000 CONFIG_SPL_GPIO_SUPPORT=y CONFIG_SPL_LIBCOMMON_SUPPORT=y CONFIG_SPL_LIBGENERIC_SUPPORT=y -CONFIG_SYS_MALLOC_F_LEN=0x8000 +CONFIG_SYS_MALLOC_F_LEN=0x10000 CONFIG_ENV_SIZE=0x1000 CONFIG_ENV_OFFSET=0x400000 CONFIG_SYS_I2C_MXC_I2C1=y -- 2.17.1
Hi Peng,
On Wed, Dec 23, 2020 at 4:29 AM Peng Fan peng.fan@nxp.com wrote:
I am thinking to remove the CLK CCF support for i.MX8M, and use simple CLK DM. Do you agree?
As a minimal fix for 2021.01, I suggest that we go with my patch.
After 2021.01 is released we can consider using simple CLK DM.
Thanks
Subject: Re: [PATCH] imx8mp_evk: Increase CONFIG_SYS_MALLOC_F_LEN
Hi Peng,
On Wed, Dec 23, 2020 at 4:29 AM Peng Fan peng.fan@nxp.com wrote:
I am thinking to remove the CLK CCF support for i.MX8M, and use simple CLK DM. Do you agree?
As a minimal fix for 2021.01, I suggest that we go with my patch.
I agree.
After 2021.01 is released we can consider using simple CLK DM.
Sure. I need people agree on this before action, then prepare the stuff.
Thanks, Peng
Thanks
When booting imx8mp-evk the following allocation error message is seen: U-Boot 2021.01-rc3-00200-ge668bec96a5f (Dec 21 2020 - 14:36:42 -0300) alloc space exhausted Fix it by increasing CONFIG_SYS_MALLOC_F_LEN to 0x10000 like it is done on other i.MX8MM/8MN boards. Reported-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Signed-off-by: Fabio Estevam festevam@gmail.com Tested-by: Andrey Zhizhikin andrey.zhizhikin@leica-geosystems.com Reviewed-by: Peng Fan peng.fan@nxp.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
participants (5)
-
Fabio Estevam -
Peng Fan -
sbabic＠denx.de -
Stefano Babic
-
ZHIZHIKIN Andrey