* [PATCH 1/3] imx8mm_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB @ 2021-08-29 19:39 Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 2/3] imx8mq_evk: " Oleksandr Suvorov 2021-10-07 14:13 ` [PATCH 1/3] imx8mm_evk: " sbabic 0 siblings, 2 replies; 17+ messages in thread From: Oleksandr Suvorov @ 2021-08-29 19:39 UTC (permalink / raw) To: u-boot Cc: Peng Fan, Oleksandr Suvorov, Alice Guo, Oleksandr Andrushchenko, Patrick Delaunay The BSP platform LmP supports the board NXP iMX8M Mini EVK. The kernel size in LmP exceeds 32Mb. Increase the maximum size of an uncompressed kernel to fix the following error: Uncompressing Kernel Image Error: inflate() returned -5 Image too large: increase CONFIG_SYS_BOOTM_LEN Must RESET board to recover Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> --- include/configs/imx8mm_evk.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/imx8mm_evk.h b/include/configs/imx8mm_evk.h index 8f3dd8fb61..33c7ec544b 100644 --- a/include/configs/imx8mm_evk.h +++ b/include/configs/imx8mm_evk.h @@ -10,7 +10,7 @@ #include <linux/stringify.h> #include <asm/arch/imx-regs.h> -#define CONFIG_SYS_BOOTM_LEN (32 * SZ_1M) +#define CONFIG_SYS_BOOTM_LEN (64 * SZ_1M) #define CONFIG_SPL_MAX_SIZE (148 * 1024) #define CONFIG_SYS_MONITOR_LEN SZ_512K #define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_USE_SECTOR -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 2/3] imx8mq_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 [PATCH 1/3] imx8mm_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB Oleksandr Suvorov @ 2021-08-29 19:39 ` Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 3/3] imx8qm_mek: " Oleksandr Suvorov 2021-10-07 14:13 ` [PATCH 2/3] imx8mq_evk: " sbabic 2021-10-07 14:13 ` [PATCH 1/3] imx8mm_evk: " sbabic 1 sibling, 2 replies; 17+ messages in thread From: Oleksandr Suvorov @ 2021-08-29 19:39 UTC (permalink / raw) To: u-boot Cc: Peng Fan, Oleksandr Suvorov, Alice Guo, Heiko Schocher, Jaehoon Chung, Patrick Delaunay, Simon Glass The BSP platform LmP supports the board NXP iMX8M Plus EVK. The kernel size in LmP exceeds 32Mb. Increase the maximum size of an uncompressed kernel to fix the following error: Uncompressing Kernel Image Error: inflate() returned -5 Image too large: increase CONFIG_SYS_BOOTM_LEN Must RESET board to recover Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> --- include/configs/imx8mq_evk.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/imx8mq_evk.h b/include/configs/imx8mq_evk.h index af81a43cbc..26ad56c999 100644 --- a/include/configs/imx8mq_evk.h +++ b/include/configs/imx8mq_evk.h @@ -10,7 +10,7 @@ #include <linux/stringify.h> #include <asm/arch/imx-regs.h> -#define CONFIG_SYS_BOOTM_LEN (32 * SZ_1M) +#define CONFIG_SYS_BOOTM_LEN (64 * SZ_1M) #define CONFIG_SPL_MAX_SIZE (124 * 1024) #define CONFIG_SYS_MONITOR_LEN (512 * 1024) -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 ` [PATCH 2/3] imx8mq_evk: " Oleksandr Suvorov @ 2021-08-29 19:39 ` Oleksandr Suvorov 2021-08-29 19:55 ` Marek Vasut 2021-10-07 14:13 ` sbabic 2021-10-07 14:13 ` [PATCH 2/3] imx8mq_evk: " sbabic 1 sibling, 2 replies; 17+ messages in thread From: Oleksandr Suvorov @ 2021-08-29 19:39 UTC (permalink / raw) To: u-boot Cc: Peng Fan, Oleksandr Suvorov, Fabio Estevam, Marek Vasut, Oliver Graute The BSP platform LmP supports the board NXP iMX8QM MEK. The kernel size in LmP exceeds 32Mb. Increase the maximum size of an uncompressed kernel to fix the following error: Uncompressing Kernel Image Error: inflate() returned -5 Image too large: increase CONFIG_SYS_BOOTM_LEN Must RESET board to recover Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> --- include/configs/imx8qm_mek.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/configs/imx8qm_mek.h b/include/configs/imx8qm_mek.h index 99e73a9446..03fa51168a 100644 --- a/include/configs/imx8qm_mek.h +++ b/include/configs/imx8qm_mek.h @@ -10,6 +10,8 @@ #include <linux/stringify.h> #include <asm/arch/imx-regs.h> +#define CONFIG_SYS_BOOTM_LEN (64 * SZ_1M) + #ifdef CONFIG_SPL_BUILD #define CONFIG_SPL_MAX_SIZE (124 * 1024) #define CONFIG_SYS_MONITOR_LEN (1024 * 1024) -- 2.31.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 ` [PATCH 3/3] imx8qm_mek: " Oleksandr Suvorov @ 2021-08-29 19:55 ` Marek Vasut 2021-08-29 20:53 ` Tom Rini 2021-08-30 11:11 ` Oleksandr Suvorov 2021-10-07 14:13 ` sbabic 1 sibling, 2 replies; 17+ messages in thread From: Marek Vasut @ 2021-08-29 19:55 UTC (permalink / raw) To: Oleksandr Suvorov, u-boot; +Cc: Peng Fan, Fabio Estevam, Oliver Graute On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > The BSP platform LmP supports the board NXP iMX8QM MEK. The > kernel size in LmP exceeds 32Mb. Increase the maximum size > of an uncompressed kernel to fix the following error: > Uncompressing Kernel Image > Error: inflate() returned -5 > Image too large: increase CONFIG_SYS_BOOTM_LEN > Must RESET board to recover > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:55 ` Marek Vasut @ 2021-08-29 20:53 ` Tom Rini 2021-08-30 11:11 ` Oleksandr Suvorov 1 sibling, 0 replies; 17+ messages in thread From: Tom Rini @ 2021-08-29 20:53 UTC (permalink / raw) To: Marek Vasut Cc: Oleksandr Suvorov, u-boot, Peng Fan, Fabio Estevam, Oliver Graute [-- Attachment #1: Type: text/plain, Size: 645 bytes --] On Sun, Aug 29, 2021 at 09:55:16PM +0200, Marek Vasut wrote: > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > of an uncompressed kernel to fix the following error: > > Uncompressing Kernel Image > > Error: inflate() returned -5 > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > Must RESET board to recover > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. Once moved to Kconfig, yes, that would be a good idea and fairly easy. -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:55 ` Marek Vasut 2021-08-29 20:53 ` Tom Rini @ 2021-08-30 11:11 ` Oleksandr Suvorov 2021-08-30 12:18 ` Marek Vasut 2021-09-15 3:06 ` Peng Fan (OSS) 1 sibling, 2 replies; 17+ messages in thread From: Oleksandr Suvorov @ 2021-08-30 11:11 UTC (permalink / raw) To: Marek Vasut; +Cc: u-boot, Peng Fan, Fabio Estevam, Oliver Graute On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > of an uncompressed kernel to fix the following error: > > Uncompressing Kernel Image > > Error: inflate() returned -5 > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > Must RESET board to recover > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. I completely agree if NXP doesn't have objections. @Peng Fan Do you mind? -- Best regards, Oleksandr Suvorov Software Engineer W: www.foundries.io ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 11:11 ` Oleksandr Suvorov @ 2021-08-30 12:18 ` Marek Vasut 2021-08-30 15:05 ` Marcel Ziswiler 2021-09-15 3:06 ` Peng Fan (OSS) 1 sibling, 1 reply; 17+ messages in thread From: Marek Vasut @ 2021-08-30 12:18 UTC (permalink / raw) To: Oleksandr Suvorov; +Cc: u-boot, Peng Fan, Fabio Estevam, Oliver Graute On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: >> >> On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: >>> The BSP platform LmP supports the board NXP iMX8QM MEK. The >>> kernel size in LmP exceeds 32Mb. Increase the maximum size >>> of an uncompressed kernel to fix the following error: >>> Uncompressing Kernel Image >>> Error: inflate() returned -5 >>> Image too large: increase CONFIG_SYS_BOOTM_LEN >>> Must RESET board to recover >>> >> >> Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > I completely agree if NXP doesn't have objections. > @Peng Fan Do you mind? Increase it for all of arm64 , or all of U-Boot even. This has nothing to do with NXP. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 12:18 ` Marek Vasut @ 2021-08-30 15:05 ` Marcel Ziswiler 2021-08-30 15:10 ` Marek Vasut 2021-08-30 17:46 ` Tom Rini 0 siblings, 2 replies; 17+ messages in thread From: Marcel Ziswiler @ 2021-08-30 15:05 UTC (permalink / raw) To: marex, oleksandr.suvorov; +Cc: peng.fan, u-boot, festevam, oliver.graute On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: > > > > > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > > > of an uncompressed kernel to fix the following error: > > > > Uncompressing Kernel Image > > > > Error: inflate() returned -5 > > > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > > > Must RESET board to recover > > > > > > > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > > > I completely agree if NXP doesn't have objections. > > @Peng Fan Do you mind? > > Increase it for all of arm64 , or all of U-Boot even. This has nothing > to do with NXP. In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: https://marc.info/?l=u-boot&m=162999598824381 ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 15:05 ` Marcel Ziswiler @ 2021-08-30 15:10 ` Marek Vasut 2021-08-30 16:34 ` Marcel Ziswiler 2021-08-30 17:46 ` Tom Rini 1 sibling, 1 reply; 17+ messages in thread From: Marek Vasut @ 2021-08-30 15:10 UTC (permalink / raw) To: Marcel Ziswiler, oleksandr.suvorov Cc: peng.fan, u-boot, festevam, oliver.graute On 8/30/21 5:05 PM, Marcel Ziswiler wrote: > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: >> On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: >>> On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: >>>> >>>> On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: >>>>> The BSP platform LmP supports the board NXP iMX8QM MEK. The >>>>> kernel size in LmP exceeds 32Mb. Increase the maximum size >>>>> of an uncompressed kernel to fix the following error: >>>>> Uncompressing Kernel Image >>>>> Error: inflate() returned -5 >>>>> Image too large: increase CONFIG_SYS_BOOTM_LEN >>>>> Must RESET board to recover >>>>> >>>> >>>> Maybe we should increase the default for arm64 instead ? 8 MiB is too small. >>> >>> I completely agree if NXP doesn't have objections. >>> @Peng Fan Do you mind? >> >> Increase it for all of arm64 , or all of U-Boot even. This has nothing >> to do with NXP. > > In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: > > https://marc.info/?l=u-boot&m=162999598824381 I suspect the correct solution for that is to use bootm_size and let U-Boot figure out where to place the kernel / DT / etc. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 15:10 ` Marek Vasut @ 2021-08-30 16:34 ` Marcel Ziswiler 2021-08-30 16:49 ` Marek Vasut 0 siblings, 1 reply; 17+ messages in thread From: Marcel Ziswiler @ 2021-08-30 16:34 UTC (permalink / raw) To: marex, oleksandr.suvorov; +Cc: peng.fan, u-boot, festevam, oliver.graute On Mon, 2021-08-30 at 17:10 +0200, Marek Vasut wrote: > On 8/30/21 5:05 PM, Marcel Ziswiler wrote: > > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > > > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: > > > > > > > > > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > > > > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > > > > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > > > > > of an uncompressed kernel to fix the following error: > > > > > > Uncompressing Kernel Image > > > > > > Error: inflate() returned -5 > > > > > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > > > > > Must RESET board to recover > > > > > > > > > > > > > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > > > > > > > I completely agree if NXP doesn't have objections. > > > > @Peng Fan Do you mind? > > > > > > Increase it for all of arm64 , or all of U-Boot even. This has nothing > > > to do with NXP. > > > > In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: > > > > https://marc.info/?l=u-boot&m=162999598824381 > > I suspect the correct solution for that is to use bootm_size and let > U-Boot figure out where to place the kernel / DT / etc. Sure, but how would that work with FIT images e.g. containing kernel binary, device tree(s) and an initrd? ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 16:34 ` Marcel Ziswiler @ 2021-08-30 16:49 ` Marek Vasut 0 siblings, 0 replies; 17+ messages in thread From: Marek Vasut @ 2021-08-30 16:49 UTC (permalink / raw) To: Marcel Ziswiler, oleksandr.suvorov Cc: peng.fan, u-boot, festevam, oliver.graute On 8/30/21 6:34 PM, Marcel Ziswiler wrote: > On Mon, 2021-08-30 at 17:10 +0200, Marek Vasut wrote: >> On 8/30/21 5:05 PM, Marcel Ziswiler wrote: >>> On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: >>>> On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: >>>>> On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: >>>>>> >>>>>> On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: >>>>>>> The BSP platform LmP supports the board NXP iMX8QM MEK. The >>>>>>> kernel size in LmP exceeds 32Mb. Increase the maximum size >>>>>>> of an uncompressed kernel to fix the following error: >>>>>>> Uncompressing Kernel Image >>>>>>> Error: inflate() returned -5 >>>>>>> Image too large: increase CONFIG_SYS_BOOTM_LEN >>>>>>> Must RESET board to recover >>>>>>> >>>>>> >>>>>> Maybe we should increase the default for arm64 instead ? 8 MiB is too small. >>>>> >>>>> I completely agree if NXP doesn't have objections. >>>>> @Peng Fan Do you mind? >>>> >>>> Increase it for all of arm64 , or all of U-Boot even. This has nothing >>>> to do with NXP. >>> >>> In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: >>> >>> https://marc.info/?l=u-boot&m=162999598824381 >> >> I suspect the correct solution for that is to use bootm_size and let >> U-Boot figure out where to place the kernel / DT / etc. > > Sure, but how would that work with FIT images e.g. containing kernel binary, device tree(s) and an initrd? I think I must really be missing something from the discussion you linked, sorry. Can you briefly summarize it, so we are on the same page? I would say, increase BOOTM_LEN for everyone, or at least arm64, because right now it is too small. You say it can cause issues. I clearly do not see the issues it can cause, so, please help clarify it. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 15:05 ` Marcel Ziswiler 2021-08-30 15:10 ` Marek Vasut @ 2021-08-30 17:46 ` Tom Rini 2021-08-30 19:42 ` Marcel Ziswiler 1 sibling, 1 reply; 17+ messages in thread From: Tom Rini @ 2021-08-30 17:46 UTC (permalink / raw) To: Marcel Ziswiler Cc: marex, oleksandr.suvorov, peng.fan, u-boot, festevam, oliver.graute [-- Attachment #1: Type: text/plain, Size: 1505 bytes --] On Mon, Aug 30, 2021 at 03:05:32PM +0000, Marcel Ziswiler wrote: > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: > > > > > > > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > > > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > > > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > > > > of an uncompressed kernel to fix the following error: > > > > > Uncompressing Kernel Image > > > > > Error: inflate() returned -5 > > > > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > > > > Must RESET board to recover > > > > > > > > > > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > > > > > I completely agree if NXP doesn't have objections. > > > @Peng Fan Do you mind? > > > > Increase it for all of arm64 , or all of U-Boot even. This has nothing > > to do with NXP. > > In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: > > https://marc.info/?l=u-boot&m=162999598824381 In that yes, if we allow for larger kernels to be loaded, we also need to ensure platforms use sane relocation values, it also needs to be considered. But even if we have CONFIG_SYS_BOOTM_LEN set large, unless we then also disable device tree / initrd relocation, we don't have a silent problem? -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 17:46 ` Tom Rini @ 2021-08-30 19:42 ` Marcel Ziswiler 0 siblings, 0 replies; 17+ messages in thread From: Marcel Ziswiler @ 2021-08-30 19:42 UTC (permalink / raw) To: trini; +Cc: peng.fan, marex, u-boot, festevam, oleksandr.suvorov, oliver.graute On Mon, 2021-08-30 at 13:46 -0400, Tom Rini wrote: > On Mon, Aug 30, 2021 at 03:05:32PM +0000, Marcel Ziswiler wrote: > > On Mon, 2021-08-30 at 14:18 +0200, Marek Vasut wrote: > > > On 8/30/21 1:11 PM, Oleksandr Suvorov wrote: > > > > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: > > > > > > > > > > On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: > > > > > > The BSP platform LmP supports the board NXP iMX8QM MEK. The > > > > > > kernel size in LmP exceeds 32Mb. Increase the maximum size > > > > > > of an uncompressed kernel to fix the following error: > > > > > > Uncompressing Kernel Image > > > > > > Error: inflate() returned -5 > > > > > > Image too large: increase CONFIG_SYS_BOOTM_LEN > > > > > > Must RESET board to recover > > > > > > > > > > > > > > > > Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > > > > > > > I completely agree if NXP doesn't have objections. > > > > @Peng Fan Do you mind? > > > > > > Increase it for all of arm64 , or all of U-Boot even. This has nothing > > > to do with NXP. > > > > In general, I agree. However, in practice this can have devastating effects on stuff as discussed here: > > > > https://marc.info/?l=u-boot&m=162999598824381 > > In that yes, if we allow for larger kernels to be loaded, we also need > to ensure platforms use sane relocation values, it also needs to be > considered. Exactly. > But even if we have CONFIG_SYS_BOOTM_LEN set large, unless > we then also disable device tree / initrd relocation, we don't have a > silent problem? Well, I am not saying we should NOT increase CONFIG_SYS_BOOTM_LEN. I am just cautioning that this may cause further issue resp. might require further adjustments down the road. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-30 11:11 ` Oleksandr Suvorov 2021-08-30 12:18 ` Marek Vasut @ 2021-09-15 3:06 ` Peng Fan (OSS) 1 sibling, 0 replies; 17+ messages in thread From: Peng Fan (OSS) @ 2021-09-15 3:06 UTC (permalink / raw) To: Oleksandr Suvorov, Marek Vasut Cc: u-boot, Peng Fan, Fabio Estevam, Oliver Graute On 2021/8/30 19:11, Oleksandr Suvorov wrote: > On Sun, Aug 29, 2021 at 10:55 PM Marek Vasut <marex@denx.de> wrote: >> >> On 8/29/21 9:39 PM, Oleksandr Suvorov wrote: >>> The BSP platform LmP supports the board NXP iMX8QM MEK. The >>> kernel size in LmP exceeds 32Mb. Increase the maximum size >>> of an uncompressed kernel to fix the following error: >>> Uncompressing Kernel Image >>> Error: inflate() returned -5 >>> Image too large: increase CONFIG_SYS_BOOTM_LEN >>> Must RESET board to recover >>> >> >> Maybe we should increase the default for arm64 instead ? 8 MiB is too small. > > I completely agree if NXP doesn't have objections. > @Peng Fan Do you mind? > I am ok with increase ARM64 default bootm len. Regards, Peng. ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/3] imx8qm_mek: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 ` [PATCH 3/3] imx8qm_mek: " Oleksandr Suvorov 2021-08-29 19:55 ` Marek Vasut @ 2021-10-07 14:13 ` sbabic 1 sibling, 0 replies; 17+ messages in thread From: sbabic @ 2021-10-07 14:13 UTC (permalink / raw) To: Oleksandr Suvorov, U-Boot > The BSP platform LmP supports the board NXP iMX8QM MEK. The > kernel size in LmP exceeds 32Mb. Increase the maximum size > of an uncompressed kernel to fix the following error: > Uncompressing Kernel Image > Error: inflate() returned -5 > Image too large: increase CONFIG_SYS_BOOTM_LEN > Must RESET board to recover > Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 2/3] imx8mq_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 ` [PATCH 2/3] imx8mq_evk: " Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 3/3] imx8qm_mek: " Oleksandr Suvorov @ 2021-10-07 14:13 ` sbabic 1 sibling, 0 replies; 17+ messages in thread From: sbabic @ 2021-10-07 14:13 UTC (permalink / raw) To: Oleksandr Suvorov, U-Boot > The BSP platform LmP supports the board NXP iMX8M Plus EVK. The > kernel size in LmP exceeds 32Mb. Increase the maximum size > of an uncompressed kernel to fix the following error: > Uncompressing Kernel Image > Error: inflate() returned -5 > Image too large: increase CONFIG_SYS_BOOTM_LEN > Must RESET board to recover > Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 1/3] imx8mm_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB 2021-08-29 19:39 [PATCH 1/3] imx8mm_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 2/3] imx8mq_evk: " Oleksandr Suvorov @ 2021-10-07 14:13 ` sbabic 1 sibling, 0 replies; 17+ messages in thread From: sbabic @ 2021-10-07 14:13 UTC (permalink / raw) To: Oleksandr Suvorov, U-Boot > The BSP platform LmP supports the board NXP iMX8M Mini EVK. The > kernel size in LmP exceeds 32Mb. Increase the maximum size > of an uncompressed kernel to fix the following error: > Uncompressing Kernel Image > Error: inflate() returned -5 > Image too large: increase CONFIG_SYS_BOOTM_LEN > Must RESET board to recover > Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@foundries.io> Applied to u-boot-imx, master, thanks ! Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2021-10-07 14:18 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-29 19:39 [PATCH 1/3] imx8mm_evk: Increase CONFIG_SYS_BOOTM_LEN to 64MB Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 2/3] imx8mq_evk: " Oleksandr Suvorov 2021-08-29 19:39 ` [PATCH 3/3] imx8qm_mek: " Oleksandr Suvorov 2021-08-29 19:55 ` Marek Vasut 2021-08-29 20:53 ` Tom Rini 2021-08-30 11:11 ` Oleksandr Suvorov 2021-08-30 12:18 ` Marek Vasut 2021-08-30 15:05 ` Marcel Ziswiler 2021-08-30 15:10 ` Marek Vasut 2021-08-30 16:34 ` Marcel Ziswiler 2021-08-30 16:49 ` Marek Vasut 2021-08-30 17:46 ` Tom Rini 2021-08-30 19:42 ` Marcel Ziswiler 2021-09-15 3:06 ` Peng Fan (OSS) 2021-10-07 14:13 ` sbabic 2021-10-07 14:13 ` [PATCH 2/3] imx8mq_evk: " sbabic 2021-10-07 14:13 ` [PATCH 1/3] imx8mm_evk: " sbabic
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.