Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 03/31] arch/arm: add blind options to know the ARM architecture
Date: Mon, 20 Oct 2014 22:03:39 +0200	[thread overview]
Message-ID: <54456A9B.4020704@mind.be> (raw)
In-Reply-To: <20141019215528.GD12247@free.fr>

On 19/10/14 23:55, Yann E. MORIN wrote:
> Thomas, All,
> 
> On 2014-10-19 22:07 +0200, Thomas Petazzoni spake thusly:
>> In preparation to the removal of BR2_GCC_TARGET_ARCH for ARM, this
>> commit introduces a number of blind options for each ARM architecture,
>> so that packages/toolchains that had dependencies using
>> BR2_GCC_TARGET_ARCH can continue to express their dependencies. It can
>> also be used to simplify package dependencies that were using the
>> individual ARM core options.
>>
>> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>> ---
>>  arch/Config.in.arm | 39 +++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 39 insertions(+)
>>
>> diff --git a/arch/Config.in.arm b/arch/Config.in.arm
>> index 201ff1d..d2ba924 100644
>> --- a/arch/Config.in.arm
>> +++ b/arch/Config.in.arm
>> @@ -37,6 +37,27 @@ config BR2_ARM_CPU_HAS_THUMB
>>  config BR2_ARM_CPU_HAS_THUMB2
>>  	bool
> 
> Indirectly related to your patch: HAS_THUMB2 should select HAS_THUMB,
> since Thumb-2 is a superset of Thumb.
> 
> Then, you could just have (for example) BR2_ARM_CPU_ARMV4 and ditch
> BR2_ARM_CPU_ARMV4T since it can be reconstructed from BR2_ARM_CPU_ARMV4
> && HAS_THUMB.
> 
> The reasonning is that few package actually require knowing the exact
> variant, but rather the instruction set generation. Thus knowing about
> armv4t is equivalent to knowing about armv4 for most. Those few that
> actuallt require the Thumb isns would need to use the HAS_THUMB option,
> which very purpose is just that, telling whether thumb is supported or
> not (or HAS_THUMB2 for Thumb-2).

 +1 to this. I made the same reasoning.


 Regards,
 Arnout


> As for the J version, I wonder if we need to introduce a HAS_JAZELLE
> option. For armv6zk (Wikpedia refer to it as armv6kz), I also doubt we
> need to introduce a HAS_TRUSTZONE option either. For armv7a, I think we
> can just use armv7, since we will probably never support the R and M
> profiles in Buildroot.

[snip]


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

  reply	other threads:[~2014-10-20 20:03 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-19 20:07 [Buildroot] [PATCH 00/31] Removal of BR2_GCC_TARGET_TUNE and other arch/ updates Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 01/31] ffmpeg: remove dangling reference to BR2_arm10 Thomas Petazzoni
2014-10-19 21:34   ` Yann E. MORIN
2014-10-20  8:35   ` Samuel Martin
2014-10-20  9:10   ` Peter Korsgaard
2014-10-19 20:07 ` [Buildroot] [PATCH 02/31] libplayer: fix typo on BR2_nios2 Thomas Petazzoni
2014-10-19 21:34   ` Yann E. MORIN
2014-10-20  8:35   ` Samuel Martin
2014-10-20  9:11   ` Peter Korsgaard
2014-10-19 20:07 ` [Buildroot] [PATCH 03/31] arch/arm: add blind options to know the ARM architecture Thomas Petazzoni
2014-10-19 21:55   ` Yann E. MORIN
2014-10-20 20:03     ` Arnout Vandecappelle [this message]
2014-10-21 17:03     ` Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 04/31] ffmpeg: use the new BR2_ARM_CPU_ARM* options Thomas Petazzoni
2014-10-20 20:12   ` Arnout Vandecappelle
2014-10-19 20:07 ` [Buildroot] [PATCH 05/31] gst-ffmpeg: " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 06/31] gst-ffmpeg: allow usage of ARMv6 optimizations on ARMv7 platforms Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 07/31] gst1-libav: use the new BR2_ARM_CPU_ARM* options Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 08/31] gst1-libav: allow usage of ARMv6 optimizations on ARMv7 platforms Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 09/31] mplayer: use BR2_ARM_CPU_ARM* options Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 10/31] libplayer: use BR2_ARM_CPU_ARM* option Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 11/31] nettle: use the BR2_ARM_CPU_ARM* options Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 12/31] nodejs: use " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 13/31] qt5: use the " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 14/31] speex: use " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 15/31] vo-aacenc: use the " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 16/31] webkit: use " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 17/31] valgrind: use the " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 18/31] x264: use the BR2_ARM_CPU_ARMV7A option Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 19/31] libnspr: " Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 20/31] toolchain-external: use BR2_ARM_CPU_ARM* options Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 21/31] dhcpcd: do not pass --target option Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 22/31] arch/arm: do not distinguish revisions of ARM1136JF-S Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 23/31] arch/arm: remove BR2_GCC_TARGET_ARCH definitions on ARM Thomas Petazzoni
2014-10-19 20:07 ` [Buildroot] [PATCH 24/31] ffmpeg: use arch and cpu instead of tune Thomas Petazzoni
2014-10-19 22:09   ` Yann E. MORIN
2014-10-19 20:07 ` [Buildroot] [PATCH 25/31] arch/x86: get rid of BR2_GCC_TARGET_TUNE Thomas Petazzoni
2014-10-19 22:10   ` Yann E. MORIN
2014-10-19 20:08 ` [Buildroot] [PATCH 26/31] arch/sparc: " Thomas Petazzoni
2014-10-19 22:13   ` Yann E. MORIN
2014-10-19 20:08 ` [Buildroot] [PATCH 27/31] arch/powerpc: " Thomas Petazzoni
2014-10-19 22:17   ` Yann E. MORIN
2014-10-19 20:08 ` [Buildroot] [PATCH 28/31] arch/m68k: " Thomas Petazzoni
2014-10-19 22:18   ` Yann E. MORIN
2014-10-19 20:08 ` [Buildroot] [PATCH 29/31] gcc: do not use BR2_GCC_TARGET_TUNE anymore Thomas Petazzoni
2014-10-19 22:18   ` Yann E. MORIN
2014-10-19 20:08 ` [Buildroot] [PATCH 30/31] toolchain-external: " Thomas Petazzoni
2014-10-19 20:08 ` [Buildroot] [PATCH 31/31] arch: remove the BR2_GCC_TARGET_TUNE option Thomas Petazzoni

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=54456A9B.4020704@mind.be \
    --to=arnout@mind.be \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox