Re: [RESEND v2 12/13] arm: sc5xx: remove LOADADDR and use kernel_addr_r

Greg Malysa
2026-04-06T02:22:28Z

Hi Caleb,

Sorry for the delay on this response.

On Thu, Apr 2, 2026 at 11:29 AM Caleb Ethridge <jcethrid@gmail.com> wrote:
>
> From: Caleb Ethridge <caleb.ethridge@analog.com>
>
> Remove the SC5XX_LOADADDR Kconfig option, replace its users with
> CONFIG_SYS_LOAD_ADDR, and update the ADI boot environment to use
> `kernel_addr_r`.
>
> SC5XX_LOADADDR and the corresponding `loadaddr` usage were ADI-specific
> duplicates of standard U-Boot load address handling. U-Boot already uses
> CONFIG_SYS_LOAD_ADDR for the default load address and `kernel_addr_r`
> for boot commands, so keeping separate SC5XX-specific names is
> redundant.

I agree with dropping SC5XX_LOADADDRR and replacing with
CONFIG_SYS_LOAD_ADDR . But should we really replace ${loadaddr} with
${kernel_addr_r}? We're loading/booting a fit image not a raw kernel.
bootm defaults to CONFIG_SYS_LOAD_ADDR which is exported as loadaddr
in include/env_default.h.

Thanks,
Greg