On Thu, Feb 22, 2024 at 09:47:58AM +0300, Dan Carpenter wrote:

> In the Makefile there is a line that says this:
> 
>     # the checker needs the correct machine size
>     CHECKFLAGS += $(if $(CONFIG_64BIT),-m64,-m32)
> 
> So set CONFIG_64BIT so that we don't pass -m32 to the static checker.
> 
> Signed-off-by: Dan Carpenter <dan.carpen...@linaro.org>
> ---
>  arch/arm/Kconfig | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index fde85dc0d537..4c7be7cf9c33 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1,11 +1,15 @@
>  menu "ARM architecture"
>       depends on ARM
>  
> +config 64BIT
> +     bool
> +
>  config SYS_ARCH
>       default "arm"
>  
>  config ARM64
>       bool
> +     select 64BIT
>       select PHYS_64BIT
>       select SYS_CACHE_SHIFT_6
>       imply SPL_SEPARATE_BSS

That we have 'config 64BIT' in arch/{mips,riscv}/Kconfig as well is a
leftover of thinking this is like the Linux Kernel where there's no
top-level arch/Kconfig file, can you please move the two existing config
64BIT entries to arch/Kconfig and then select it for ARM64? Thanks.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to