All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4/4] arch: remove BR2_GCC_TARGET_ARCH definitions on ARM
Date: Tue, 16 Sep 2014 00:09:28 +0200	[thread overview]
Message-ID: <20140915220928.GA22581@free.fr> (raw)
In-Reply-To: <1410814919-11523-5-git-send-email-thomas.petazzoni@free-electrons.com>

Thomas, All,

On 2014-09-15 23:01 +0200, Thomas Petazzoni spake thusly:
> On ARM, we were defining both the CPU type and the architecture
> variant. However, depending on the version of gcc, a given combination
> of (CPU, architecture) may not be the same. Since the architecture
> variant is implied by the CPU type, given the former is not necessary,
> and we can simply specify the latter.
> 
> From the gcc documentation:
> 
>   This specifies the name of the target ARM processor. GCC uses this
>   name to derive the name of the target ARM architecture (as if
>   specified by -march) and the ARM processor type for which to tune
>   for performance (as if specified by -mtune). Where this option is
>   used in conjunction with -march or -mtune, those options take
>   precedence over the appropriate part of this option.
> 
> Note that we verified that for all BR2_GCC_TARGET_ARCH value that
> existed, a proper BR2_GCC_TARGET_CPU value is defined.

I guess we'll keep this one in limbo until we have a final answer on
switching from -mcpu to -mtune for ARM.

Regards,
Yann E. MORIN.

> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
>  arch/Config.in.arm | 19 -------------------
>  1 file changed, 19 deletions(-)
> 
> diff --git a/arch/Config.in.arm b/arch/Config.in.arm
> index 6e3c4e6..68f6370 100644
> --- a/arch/Config.in.arm
> +++ b/arch/Config.in.arm
> @@ -352,25 +352,6 @@ config BR2_GCC_TARGET_CPU
>  	default "xscale"	if BR2_xscale
>  	default "iwmmxt"	if BR2_iwmmxt
>  
> -config BR2_GCC_TARGET_ARCH
> -	default "armv4t"	if BR2_arm920t
> -	default "armv4t"	if BR2_arm922t
> -	default "armv5te"	if BR2_arm926t
> -	default "armv6j"	if BR2_arm1136jf_s
> -	default "armv6zk"	if BR2_arm1176jz_s
> -	default "armv6zk"	if BR2_arm1176jzf_s
> -	default "armv7-a"	if BR2_cortex_a5
> -	default "armv7-a"	if BR2_cortex_a7
> -	default "armv7-a"	if BR2_cortex_a8
> -	default "armv7-a"	if BR2_cortex_a9
> -	default "armv7-a"	if BR2_cortex_a12
> -	default "armv7-a"	if BR2_cortex_a15
> -	default "armv4"		if BR2_fa526
> -	default "armv7-a"	if BR2_pj4
> -	default "armv4"		if BR2_strongarm
> -	default "armv5te"	if BR2_xscale
> -	default "iwmmxt"	if BR2_iwmmxt
> -
>  config BR2_GCC_TARGET_ABI
>  	default "aapcs-linux"
>  
> -- 
> 2.0.0
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

      reply	other threads:[~2014-09-15 22:09 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-15 21:01 [Buildroot] [PATCH 0/4] Fix march/mcpu conflict on ARM Thomas Petazzoni
2014-09-15 21:01 ` [Buildroot] [PATCH 1/4] arch: remove BR2_arm920 reference Thomas Petazzoni
2014-09-15 21:08   ` Yann E. MORIN
2014-09-18 20:12   ` Peter Korsgaard
2014-09-15 21:01 ` [Buildroot] [PATCH 2/4] arch: remove BR2_arm10t Thomas Petazzoni
2014-09-15 21:13   ` Yann E. MORIN
2014-09-15 21:16     ` Thomas Petazzoni
2014-09-18 20:12   ` Peter Korsgaard
2014-09-15 21:01 ` [Buildroot] [PATCH 3/4] arch: do not distinguish revisions of ARM1136JF-S Thomas Petazzoni
2014-09-15 21:16   ` Benoît Thébaudeau
2014-09-15 21:33   ` Yann E. MORIN
2014-09-15 21:39     ` Thomas Petazzoni
2014-09-15 21:44       ` Benoît Thébaudeau
2014-09-15 21:50         ` Thomas Petazzoni
2014-09-15 22:36           ` Benoît Thébaudeau
2014-09-15 22:02         ` Yann E. MORIN
2014-09-15 22:37           ` Benoît Thébaudeau
2014-09-15 21:48       ` Yann E. MORIN
2014-09-15 21:01 ` [Buildroot] [PATCH 4/4] arch: remove BR2_GCC_TARGET_ARCH definitions on ARM Thomas Petazzoni
2014-09-15 22:09   ` Yann E. MORIN [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140915220928.GA22581@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.