Re: [PATCH 2/4] riscv: Set SYS_BOOTM_LEN default to 0x4000000

[Mattijs Korpershoek <mkorpershoek@kernel.org>]

Hi Martin,

Thank you for the patch.

On Sat, Jul 19, 2025 at 23:46, Martin Herren <sputnik@on-the-web.ch> wrote:

> This changes the default value to the most commonly used one among
> existing defconfigs.
>
> Acked-by: Michal Simek <michal.simek@amd.com> # xilinx_mbv
> Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>
>
> Signed-off-by: Martin Herren <sputnik@on-the-web.ch>

Minor note: there should *not* be a newline between the tags that you
applied (Acked-by, Reviewed-by) and your own sign-off. So the footer
block should be:

"""
    Acked-by: Michal Simek <michal.simek@amd.com> # xilinx_mbv
    Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>
    Signed-off-by: Martin Herren <sputnik@on-the-web.ch>
"""

I don't think you need to resend for this, usually the maintainer who
picks up your patch will fix this.

Also:

Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org>


> ---
>
> (no changes since v1)
>
>  boot/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/boot/Kconfig b/boot/Kconfig
> index 2ff6f003738..54ef7052c5c 100644
> --- a/boot/Kconfig
> +++ b/boot/Kconfig
> @@ -1057,7 +1057,7 @@ config SYS_BOOTM_LEN
>  	hex "Maximum size of a decompresed OS image"
>  	depends on CMD_BOOTM || CMD_BOOTI || CMD_BOOTZ || \
>  		LEGACY_IMAGE_FORMAT || SPL_LEGACY_IMAGE_FORMAT
> -	default 0x4000000 if PPC || ARM64
> +	default 0x4000000 if PPC || ARM64 || RISCV
>  	default 0x1000000 if X86 || ARCH_MX6 || ARCH_MX7
>  	default 0x800000
>  	help