arch: pass cpu option instead of tune option on ARM
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Sat, 2 Nov 2013 15:39:06 +0000 (16:39 +0100)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Thu, 26 Dec 2013 22:19:04 +0000 (23:19 +0100)
Currently, the ARM Config.in logic specifies values for
--with-arch/-march and --with-tune/-mtune, but not for
--with-cpu/-mcpu. However, this causes problems on ARMv4, because
specifying --with-arch=armv4t isn't enough to make gcc generate ARMv4
code: one should also pass --with-cpu=<some ARMv4 CPU>.

Moreover, since Buildroot is generally designed to generate code
specifically for the configured target, it makes sense to give our own
--with-cpu/-mcpu value instead of relying on the default value used by
gcc, and only do small optimizations with -mtune.

Reported-by: Adam Hussein <kryme76@yahoo.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
arch/Config.in.arm

index c0fabb7ca4de11c1c31cc241eebdf29029fabf88..dd587443950b1dd032517cadcf650dc3b27ef1f3 100644 (file)
@@ -341,7 +341,7 @@ config BR2_ENDIAN
        default "LITTLE" if BR2_arm
        default "BIG"    if BR2_armeb
 
-config BR2_GCC_TARGET_TUNE
+config BR2_GCC_TARGET_CPU
        default "arm7tdmi"      if BR2_arm7tdmi
        default "arm7tdmi"      if BR2_arm720t
        default "arm7tdmi"      if BR2_arm740t