[PATCH] configs: am62x_evm_a53_defconfig: Remove CONFIG_SYS_BOOTM_LEN

Increase the maximum size of the buffer that is used to decompress the OS image in to.
If image size is greater than the buffer, boot will fail with following error, "Error: inflate() returned -5" "gzip compressed: uncompress error -1"
Default limit for ARM64 platforms is 0x4000000, remove CONFIG_SYS_BOOTM_LEN=0x8000000, so that it follows default limit.
Signed-off-by: Kamlesh Gurudasani kamlesh@ti.com --- configs/am62x_evm_a53_defconfig | 1 - 1 file changed, 1 deletion(-)
diff --git a/configs/am62x_evm_a53_defconfig b/configs/am62x_evm_a53_defconfig index ca993b427b..dfa5ecc847 100644 --- a/configs/am62x_evm_a53_defconfig +++ b/configs/am62x_evm_a53_defconfig @@ -32,7 +32,6 @@ CONFIG_SPL_SPI=y CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 CONFIG_BOOTSTD_FULL=y -CONFIG_SYS_BOOTM_LEN=0x800000 CONFIG_BOOTCOMMAND="run findfdt; run envboot; bootflow scan -lb" CONFIG_BOARD_LATE_INIT=y CONFIG_SPL_MAX_SIZE=0x58000
--- base-commit: fd46ea0e701920eb205c2bce9d527bf0dec10b59 change-id: 20240708-remove-bootm-len-ab4e46039656
Best regards,

On Mon, Jul 08, 2024 at 05:20:40PM +0530, Kamlesh Gurudasani wrote:
Increase the maximum size of the buffer that is used to decompress the OS image in to.
If image size is greater than the buffer, boot will fail with following error, "Error: inflate() returned -5" "gzip compressed: uncompress error -1"
Default limit for ARM64 platforms is 0x4000000, remove CONFIG_SYS_BOOTM_LEN=0x8000000, so that it follows default limit.
Signed-off-by: Kamlesh Gurudasani kamlesh@ti.com
configs/am62x_evm_a53_defconfig | 1 - 1 file changed, 1 deletion(-)
Please check and correct all of the TI K3 platforms. We have at least this from a quick grep on am6*: configs/am62x_beagleplay_a53_defconfig:CONFIG_SYS_BOOTM_LEN=0x800000 configs/am62x_evm_a53_defconfig:CONFIG_SYS_BOOTM_LEN=0x800000 configs/am65x_evm_r5_defconfig:CONFIG_SYS_BOOTM_LEN=0x4000000 configs/am65x_evm_r5_usbdfu_defconfig:CONFIG_SYS_BOOTM_LEN=0x4000000 configs/am65x_evm_r5_usbmsc_defconfig:CONFIG_SYS_BOOTM_LEN=0x4000000
(And so are those larger values on the R5 core actually needed?)

Hi Kamlesh,
On 17:20-20240708, Kamlesh Gurudasani wrote:
Increase the maximum size of the buffer that is used to decompress the OS image in to.
If image size is greater than the buffer, boot will fail with following error, "Error: inflate() returned -5" "gzip compressed: uncompress error -1"
Default limit for ARM64 platforms is 0x4000000, remove CONFIG_SYS_BOOTM_LEN=0x8000000, so that it follows default limit.
Why were we failing with a larger buffer size..?
Regards, Manorit
Signed-off-by: Kamlesh Gurudasani kamlesh@ti.com
configs/am62x_evm_a53_defconfig | 1 - 1 file changed, 1 deletion(-)
diff --git a/configs/am62x_evm_a53_defconfig b/configs/am62x_evm_a53_defconfig index ca993b427b..dfa5ecc847 100644 --- a/configs/am62x_evm_a53_defconfig +++ b/configs/am62x_evm_a53_defconfig @@ -32,7 +32,6 @@ CONFIG_SPL_SPI=y CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 CONFIG_BOOTSTD_FULL=y -CONFIG_SYS_BOOTM_LEN=0x800000 CONFIG_BOOTCOMMAND="run findfdt; run envboot; bootflow scan -lb" CONFIG_BOARD_LATE_INIT=y CONFIG_SPL_MAX_SIZE=0x58000
base-commit: fd46ea0e701920eb205c2bce9d527bf0dec10b59 change-id: 20240708-remove-bootm-len-ab4e46039656
Best regards,
Kamlesh Gurudasani kamlesh@ti.com

Manorit Chawdhry m-chawdhry@ti.com writes:
Hi Kamlesh,
On 17:20-20240708, Kamlesh Gurudasani wrote:
Increase the maximum size of the buffer that is used to decompress the OS image in to.
If image size is greater than the buffer, boot will fail with following error, "Error: inflate() returned -5" "gzip compressed: uncompress error -1"
Default limit for ARM64 platforms is 0x4000000, remove CONFIG_SYS_BOOTM_LEN=0x8000000, so that it follows default limit.
Why were we failing with a larger buffer size..?
Thanks for reviewing, Manorit. Typo in the commit msg, extra zero. It should be 0x800000 as in actual diff. Will fix in v2.
Kamlesh
Regards, Manorit
Signed-off-by: Kamlesh Gurudasani kamlesh@ti.com
configs/am62x_evm_a53_defconfig | 1 - 1 file changed, 1 deletion(-)
diff --git a/configs/am62x_evm_a53_defconfig b/configs/am62x_evm_a53_defconfig index ca993b427b..dfa5ecc847 100644 --- a/configs/am62x_evm_a53_defconfig +++ b/configs/am62x_evm_a53_defconfig @@ -32,7 +32,6 @@ CONFIG_SPL_SPI=y CONFIG_SPL_LOAD_FIT=y CONFIG_SPL_LOAD_FIT_ADDRESS=0x81000000 CONFIG_BOOTSTD_FULL=y -CONFIG_SYS_BOOTM_LEN=0x800000 CONFIG_BOOTCOMMAND="run findfdt; run envboot; bootflow scan -lb" CONFIG_BOARD_LATE_INIT=y CONFIG_SPL_MAX_SIZE=0x58000
base-commit: fd46ea0e701920eb205c2bce9d527bf0dec10b59 change-id: 20240708-remove-bootm-len-ab4e46039656
Best regards,
Kamlesh Gurudasani kamlesh@ti.com
participants (3)
-
Kamlesh Gurudasani
-
Manorit Chawdhry
-
Tom Rini