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

Greg Malysa
2026-04-06T17:49:13Z

Hi Caleb,

On Mon, Apr 6, 2026 at 9:32 AM Caleb Ethridge <jcethrid@gmail.com> wrote:
>
> Hi Greg,
>
> > 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.
>
> We have recently opened up our support to include build systems which
> produce a kernel image + dtb combo instead of the full fit image. It
> probably makes sense then to leave the kernel_addr_r define in for use
> with booti if needed, but remove the kernel_addr_r from the bootm commands since
> they are redundant.

That sounds good to me. I'll run it through CI and add a tag in the morning.

Thanks,
Greg