[PATCH 14/17] makefile: Add READELF command variable

[Ilias Apalodimas]

On Mon, 24 Feb 2025 at 07:56, Sam Edwards <cfsworks at gmail.com> wrote:
>
> This allows setting READELF=llvm-readelf in order to use the LLVM
> version of the readelf utility. It also aligns with the practice of not
> using $(CROSS_COMPILE) in any build recipes directly, reducing the
> number of places where $(CROSS_COMPILE) is used.
>
> Signed-off-by: Sam Edwards <CFSworks at gmail.com>
> ---
>  Makefile | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index c869b5c55fa..c1749e78d6f 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -406,6 +406,7 @@ LDR         = $(CROSS_COMPILE)ldr
>  STRIP          = $(CROSS_COMPILE)strip
>  OBJCOPY                = $(CROSS_COMPILE)objcopy
>  OBJDUMP                = $(CROSS_COMPILE)objdump
> +READELF                = $(CROSS_COMPILE)readelf
>  LEX            = flex
>  YACC           = bison
>  AWK            = awk
> @@ -2177,7 +2178,7 @@ System.map:       u-boot
>  # ARM relocations should all be R_ARM_RELATIVE (32-bit) or
>  # R_AARCH64_RELATIVE (64-bit).
>  checkarmreloc: u-boot
> -       @RELOC="`$(CROSS_COMPILE)readelf -r -W $< | cut -d ' ' -f 4 | \
> +       @RELOC="`$(READELF) -r -W $< | cut -d ' ' -f 4 | \
>                 grep R_A | sort -u`"; \
>         if test "$$RELOC" != "R_ARM_RELATIVE" -a \
>                  "$$RELOC" != "R_AARCH64_RELATIVE"; then \
> --
> 2.45.2
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>