[PATCH v4 22/29] efi: arm: Drop startup code from the app

[Caleb Connolly]

On 2/15/25 04:22, Simon Glass wrote:
> The EFI app uses different startup and relocation code and the existing
> code uses symbols not present in the app. Drop it.

Shouldn't this be squashed into patch 18 ("efi: arm: Omit the ARM 
start-up code in the EFI app")?
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Tom Rini <trini at konsulko.com>
> ---
> 
> (no changes since v1)
> 
>   arch/arm/cpu/armv8/Makefile | 2 ++
>   arch/arm/lib/Makefile       | 4 +++-
>   2 files changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/cpu/armv8/Makefile b/arch/arm/cpu/armv8/Makefile
> index 7bd6cf21b19..a8f5de58cc4 100644
> --- a/arch/arm/cpu/armv8/Makefile
> +++ b/arch/arm/cpu/armv8/Makefile
> @@ -3,7 +3,9 @@
>   # (C) Copyright 2000-2003
>   # Wolfgang Denk, DENX Software Engineering, wd at denx.de.
>   
> +ifndef CONFIG_EFI_APP
>   extra-y	:= start.o
> +endif
>   
>   obj-y	+= cpu.o
>   ifndef CONFIG_$(PHASE_)TIMER
> diff --git a/arch/arm/lib/Makefile b/arch/arm/lib/Makefile
> index 9500ba6fddc..a41566c816d 100644
> --- a/arch/arm/lib/Makefile
> +++ b/arch/arm/lib/Makefile
> @@ -24,11 +24,13 @@ endif  # ARM64
>   endif  # CONFIG_EFI_APP
>   
>   ifndef CONFIG_XPL_BUILD
> +ifndef CONFIG_EFI_APP
>   ifdef CONFIG_ARM64
>   obj-y	+= relocate_64.o
>   else
>   obj-y	+= relocate.o
> -endif
> +endif  # ARM64
> +endif  # EFI_APP
>   
>   obj-$(CONFIG_CPU_V7M) += cmd_boot.o
>   obj-$(CONFIG_OF_LIBFDT) += bootm-fdt.o

-- 
Caleb (they/them)