Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 0/6] Correct and improve the ARM tunings
Date: Tue,  2 Apr 2019 21:30:58 +0200	[thread overview]
Message-ID: <cover.1554233389.git.pkj@axis.com> (raw)

These patches are intended to improve the ARM tunings and came about
after I had to study the tune files a lot more than I first had
anticipated...

The first patch (arch-armv8a.inc: Correct
PACKAGE_EXTRA_ARCHS_tune-armv8a-*) avoids odd architectures such as
"crc" and "crypto" to be included in PACKAGE_EXTRA_ARCHS.

The following three patches (where i have sent the latter two to the
list before) restores the "armv*" features to TUNE_FEATURES for all ARM
based SoCs as it does not make sense to remove them from the SoC
specific tune files just to determine whether -mcpu or -march shall be
used. This is because the SoCs still support the armv* features
specified in their TUNE_FEATURES even if they also have more specific
SoC tunings.

The next patch (arch-arm64.inc: Lower the priority of aarch64 in
MACHINEOVERRIDES) makes sure that ${SOC_FAMILY} (if present) and
${MACHINE} have higher override priorities than aarch64, as should be.

Finally, the last patch (arm-tunes: Add armv8a to TUNE_FEATURES as
appropriate) adds the "armv8a" feature to TUNE_FEATURES for the
Cortex-A32, Cortex-A35, Cortex-A53 and Cortex-A72 to match all the other
ARM tunings.

//Peter

The following changes since commit a397fd17e42d745e6a23dee86e82b887f3d25ccd:

  layer.conf: Update to warrior release name series (2019-04-02 15:24:50 +0100)

are available in the git repository at:

  git://push.yoctoproject.org/poky-contrib pkj/arm-tunings

Peter Kjellerstedt (6):
  arch-armv8a.inc: Correct PACKAGE_EXTRA_ARCHS_tune-armv8a-*
  Revert "arch-armv5-dsp.inc: Check for dsp only to enable 'e' in
    package arches"
  Revert "arm-tunes: Remove -march option if mcpu is already added"
  arm-tunes: Prefer the -mcpu option over -march
  arch-arm64.inc: Lower the priority of aarch64 in MACHINEOVERRIDES
  arm-tunes: Add armv8a to TUNE_FEATURES as appropriate

 meta/conf/machine/include/arm/arch-arm.inc       |  6 +++++
 meta/conf/machine/include/arm/arch-arm64.inc     |  2 +-
 meta/conf/machine/include/arm/arch-armv4.inc     |  2 +-
 meta/conf/machine/include/arm/arch-armv5-dsp.inc |  2 +-
 meta/conf/machine/include/arm/arch-armv5.inc     |  2 +-
 meta/conf/machine/include/arm/arch-armv6.inc     |  2 +-
 meta/conf/machine/include/arm/arch-armv7a.inc    |  2 +-
 meta/conf/machine/include/arm/arch-armv7ve.inc   |  2 +-
 meta/conf/machine/include/arm/arch-armv8a.inc    | 12 +++++-----
 meta/conf/machine/include/tune-arm1136jf-s.inc   |  6 ++---
 meta/conf/machine/include/tune-arm920t.inc       |  6 ++---
 meta/conf/machine/include/tune-arm926ejs.inc     |  6 ++---
 meta/conf/machine/include/tune-arm9tdmi.inc      |  6 ++---
 meta/conf/machine/include/tune-cortexa15.inc     | 29 ++++++++++-------------
 meta/conf/machine/include/tune-cortexa17.inc     | 29 ++++++++++-------------
 meta/conf/machine/include/tune-cortexa32.inc     | 11 ++++-----
 meta/conf/machine/include/tune-cortexa35.inc     |  6 ++---
 meta/conf/machine/include/tune-cortexa5.inc      | 29 ++++++++++-------------
 meta/conf/machine/include/tune-cortexa53.inc     | 10 ++++----
 meta/conf/machine/include/tune-cortexa7.inc      | 29 ++++++++++-------------
 meta/conf/machine/include/tune-cortexa72.inc     |  6 ++---
 meta/conf/machine/include/tune-cortexa8.inc      | 21 +++++++----------
 meta/conf/machine/include/tune-cortexa9.inc      | 30 ++++++++++--------------
 meta/conf/machine/include/tune-ep9312.inc        |  3 +--
 meta/conf/machine/include/tune-iwmmxt.inc        |  3 +--
 meta/conf/machine/include/tune-strongarm1100.inc |  5 ++--
 meta/conf/machine/include/tune-thunderx.inc      |  2 +-
 meta/conf/machine/include/tune-xscale.inc        |  9 +++----
 28 files changed, 125 insertions(+), 153 deletions(-)

-- 
2.12.0



             reply	other threads:[~2019-04-02 19:31 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-02 19:30 Peter Kjellerstedt [this message]
2019-04-02 19:30 ` [PATCH 1/6] arch-armv8a.inc: Correct PACKAGE_EXTRA_ARCHS_tune-armv8a-* Peter Kjellerstedt
2019-04-02 19:31 ` [PATCH 2/6] Revert "arch-armv5-dsp.inc: Check for dsp only to enable 'e' in package arches" Peter Kjellerstedt
2019-04-02 20:52   ` akuster808
2019-04-02 22:29     ` Peter Kjellerstedt
2019-04-02 19:31 ` [PATCH 3/6] Revert "arm-tunes: Remove -march option if mcpu is already added" Peter Kjellerstedt
2019-04-02 19:31 ` [PATCH 4/6] arm-tunes: Prefer the -mcpu option over -march Peter Kjellerstedt
2019-04-02 19:31 ` [PATCH 5/6] arch-arm64.inc: Lower the priority of aarch64 in MACHINEOVERRIDES Peter Kjellerstedt
2019-04-02 19:31 ` [PATCH 6/6] arm-tunes: Add armv8a to TUNE_FEATURES as appropriate Peter Kjellerstedt
2019-04-02 20:26 ` [PATCH 0/6] Correct and improve the ARM tunings Richard Purdie
2019-04-02 22:27   ` Peter Kjellerstedt
2019-04-03  6:22   ` Adrian Bunk
2019-04-03 19:29     ` Andre McCurdy
2019-04-03 20:24       ` Adrian Bunk
2019-04-03 20:48         ` Andre McCurdy
2019-04-04  8:00           ` Adrian Bunk
2019-04-17  8:33           ` Martin Jansa
2019-04-17 10:26             ` Peter Kjellerstedt

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=cover.1554233389.git.pkj@axis.com \
    --to=peter.kjellerstedt@axis.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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