[Buildroot] [PATCH 2/2] ne10: Add aarch64 configuration option

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sat Apr 29 12:09:53 UTC 2017


Hello,

On Fri, 28 Apr 2017 23:28:06 +0100, J Evans wrote:

> +ifeq ($(BR2_aarch64),y)
> +NE10_CONF_OPTS += \
> +    -DNE10_LINUX_TARGET_ARCH=aarch64
> +endif

PATCH 1/2 is already enabling ne10 on AArch64, so there's something
wrong going on here. This change should go with the patch enabling ne10
on AArch64.

But I believe you should in fact rework your patches:

 - One patch that just bumps ne10, keeping only the ARM support

 - One patch that refactors the architecture dependency with a
   Config.in hidden boolean BR2_PACKAGE_NE10_ARCH_SUPPORTS

 - One patch that allows to use ne10 on AArch64

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com



More information about the buildroot mailing list