[Buildroot] [PATCH 1/1] Add ARM11 MPCore CPU target support

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Dec 23 22:10:24 UTC 2015


Yann,

Sergi sent a v2 of his patch... but your suggestion doesn't work. See
below.

On Sat, 28 Nov 2015 15:30:30 +0100, Yann E. MORIN wrote:

> I wonder if we should not make VFP optional, and change the CPU nme on
> whether the user has enabled VFP, like:
> 
>     config BR2_arm11mpcore
>         bool "mpcore"
>         [...]
>         select BR2_ARM_CPU_MAYBE_HAS_VFPV2
>         [...]
> 
>     config BR2_GCC_TARGET_CPU
>         [...]
>         default "mpcore"        if BR2_arm11mpcore && BR2_ARM_CPU_HAS_VFPV2
>         default "mpcorenovfp"   if BR2_arm11mpcore
>         [...]

This doesn't work, because BR2_ARM_CPU_HAS_VFPV2 is not a user visible
option, and there is nothing that selects it in the current Buildroot.
Therefore the "mpcore" case can never be reached.

Consequently, I am tempted to take v1 of Sergi's patch. Yes, it means
we don't support the no-VFP variant of ARM11 MPCore, but I'm not sure
there are lots of ARM11 MPCore without a VFP, so maybe supporting
mpcorenovfp is not worth the effort for the moment.

Thoughts?

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


More information about the buildroot mailing list