[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