All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit] arch: pass cpu option instead of tune option on ARM
@ 2013-12-26 22:19 Thomas Petazzoni
  2014-01-23 15:31 ` Jonas Jensen
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2013-12-26 22:19 UTC (permalink / raw)
  To: buildroot

commit: http://git.buildroot.net/buildroot/commit/?id=d3539dd53bf9e37538fd4d8b89783a11fded119f
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master

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 |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/Config.in.arm b/arch/Config.in.arm
index c0fabb7..dd58744 100644
--- a/arch/Config.in.arm
+++ b/arch/Config.in.arm
@@ -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

^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2014-08-06 16:55 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-26 22:19 [Buildroot] [git commit] arch: pass cpu option instead of tune option on ARM Thomas Petazzoni
2014-01-23 15:31 ` Jonas Jensen
2014-01-23 18:15   ` Yann E. MORIN
2014-08-04 14:45     ` Thomas De Schampheleire
2014-08-06 15:29       ` [Buildroot] support/download/wrapper expects file output in temp directory but support/wrapper/git produces output.gz Graham Newton
2014-08-06 16:29         ` Yann E. MORIN
2014-08-06 16:55           ` Thomas De Schampheleire

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.