[PATCH v2 03/16] image: Take entry point as an output of setup_booti
Simon Glass
sjg at chromium.org
Thu Jul 2 11:46:54 CEST 2026
Hi Yao,
On 2026-07-01T11:17:53, Yao Zi <me at ziyao.cc> wrote:
> image: Take entry point as an output of setup_booti
The function is called booti_setup(), not setup_booti - please fix the subject.
>
> For LoongArch the start of the image is not the entry
> point to the image.
>
> We refactor the code base to allow entry point to be
> supplied by setup_booti.
Use imperative mode:
Refactor the code base to allow...
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang at flygoat.com>
> Signed-off-by: Yao Zi <me at ziyao.cc>
>
> arch/arm/lib/image.c | 3 ++-
> arch/riscv/lib/image.c | 4 +++-
> arch/sandbox/lib/bootm.c | 2 +-
> boot/bootm.c | 5 +++--
> cmd/booti.c | 5 +++--
> common/spl/spl.c | 9 +++++----
> include/image.h | 3 ++-
> 7 files changed, 19 insertions(+), 12 deletions(-)
> diff --git a/include/image.h b/include/image.h
> @@ -1131,11 +1131,12 @@ int bootz_setup(ulong image, ulong *start, ulong *end);
> * @image: Address of image
> * @start: Returns start address of image
> * @size : Returns size image
> + * @entry: Returns entry point of image
> * @force_reloc: Ignore image->ep field, always place image to RAM start
> * Return: 0 if OK, 1 if the image was not recognised
> */
> int booti_setup(ulong image, ulong *relocated_addr, ulong *size,
> - bool force_reloc);
> + ulong *entry, bool force_reloc);
While you are here, please fix the @start: tag to match the actual
parameter name (relocated_addr), and drop 'aarch64' from the
description above since this now covers arm64, RISC-V and LoongArch.
Regards,
Simon
More information about the U-Boot
mailing list