[Buildroot] [PATCH 3/3] arch/arm: Add ARMV8 (aarch32) toolchain config.

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon Aug 22 14:36:16 UTC 2016


Hello,

On Mon, 22 Aug 2016 20:12:28 +1000, Matt Flax wrote:

> @@ -174,8 +177,7 @@ config BR2_cortex_a53
>  	bool "cortex-A53"

It seems like this patch makes the assumption the previous Cortex-A53
patch has been applied.

Could you instead send a patch series that includes both patches
together. Don't *merge* the patches, just send them together in the
same series. It will also be less confusing than this standalone patch
labeled PATCH 3/3.

Also, I believe we need to merge arch/Config.in.aarch64 back into
arch/Config.in.arm in order to handle both 32 bits and 64 bits ARM
cores in the same Config.in.arm file, like we're doing for
Config.in.x86. Otherwise, we will end up duplicating all the ARM core
definitions for the 64 bits capable ones.

Thanks!

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


More information about the buildroot mailing list