[PATCH] mach-imx: bootaux: fix building with disabled bootelf

[Heinrich Schuchardt]

On 8/26/23 15:24, Oleksandr Suvorov wrote:
> If CMD_ELF disabled and IMX_BOOTAUX enabled, the u-boot building ends
> up with a linking error [1]. Select LIB_ELF for all cases when
> valid_elf_image() is used in the imx_bootaux module.
>
> [1]
> ld: /tmp/ccaF1rpv.ltrans0.ltrans.o: in function `do_bootaux':
> arch/arm/mach-imx/imx_bootaux.c:108: undefined reference to `valid_elf_image'
>
> Fixes: c0f037f6a2a ("mach-imx: bootaux: elf firmware support")
> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov at foundries.io>
> ---
>
>   arch/arm/mach-imx/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index d94b5828d0d..cbe62d6b8f9 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -32,6 +32,7 @@ config IMX_RDC
>   config IMX_BOOTAUX
>   	bool "Support boot auxiliary core"
>   	depends on ARCH_MX7 || ARCH_MX6 || ARCH_VF610 || ARCH_IMX8 || ARCH_IMX8M
> +	select LIB_ELF if !ARCH_IMX8 || IMX8QXP

For CONFIG_IMX8QXP=y arch/arm/mach-imx/imx8/cpu.c uses valid_elf_image().

Shouldn't the IMX8QXP case be handled in arch/arm/mach-imx/imx8/Kconfig?

config IMX8QXP
	select LIB_ELF

Best regards

Heinrich

>   	help
>   	  bootaux [addr] to boot auxiliary core.
>