Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Phil Blundell <pb@pbcl.net>
To: Martin Jansa <martin.jansa@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCHv2 11/16] arch-armv7ve: add tune include for armv7ve and use it from cortexa7 and cortexa15
Date: Wed, 18 Nov 2015 17:31:29 +0000	[thread overview]
Message-ID: <1447867889.1472.63.camel@pbcl.net> (raw)
In-Reply-To: <20151118172053.GF2213@jama>

On Wed, 2015-11-18 at 18:20 +0100, Martin Jansa wrote:
> So to get rid of -march we'll have to either
> 
> 1) check in arch-armv7a.inc if TUNE_FEATURES also contains some
> TUNE_FEATURE which will append -mcpu
> 2) don't include armv7a TUNE_FEATURE in TUNE_FEATURES_tune
> -cortexa15t,
> but the MACHINEOVERRIDES is still useful in many cases
> 3) use TUNE_CCARGS_remove to remove -march when adding -mcpu
> 4) rework whole tune files even more (probably by introducing another
> variable which will hold just one of -march,-mcpu,-mtune and gets
> appended to TUNE_CCARGS).

Or 5) just overwrite TUNE_CCARGS in tune-cortexa15.inc after including
arch-armv7a.inc, rather than appending to it.  

But personally I would be inclined to do (2) and add the appropriate
MACHINEOVERRIDES (which I think is just "armv7a" in practice) by steam
in tune-cortexa15.inc.

p.



  reply	other threads:[~2015-11-18 17:31 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-17 10:57 [PATCHv2 00/16] Various fixes from LGE Martin Jansa
2015-11-17 10:57 ` [PATCHv2 01/16] icu: force arm mode Martin Jansa
2015-11-17 10:57 ` [PATCHv2 02/16] guile, mailx, gcc, opensp, gstreamer1.0-libav, libunwind: disable thumb where it fails for qemuarm Martin Jansa
2015-11-17 10:57 ` [PATCHv2 03/16] directfb: add fPIC to CFLAGS Martin Jansa
2015-11-17 10:57 ` [PATCHv2 04/16] libunwind: fix build for qemuarm Martin Jansa
2015-11-17 10:57 ` [PATCHv2 05/16] gstreamer1.0-plugins-base: add PACKAGECONFIG for libvisual Martin Jansa
2015-11-17 10:57 ` [PATCHv2 06/16] gstreamer1.0-plugins-bad: fix dependencies for uvch264 PACKAGECONFIG Martin Jansa
2015-11-17 16:47   ` Burton, Ross
2015-11-17 10:57 ` [PATCHv2 07/16] gstreamer1.0-plugins-good: fix PACKAGECONFIG for gudev and add one for v4l2 and libv4l2 Martin Jansa
2015-11-17 14:08   ` Burton, Ross
2015-12-09  8:44     ` Martin Jansa
2015-11-17 10:58 ` [PATCHv2 08/16] gst-plugins-bad: improve FILES variables Martin Jansa
2015-11-17 10:58 ` [PATCHv2 09/16] cortexa{7,15,17}: add VFPv4 tunes Martin Jansa
2015-11-18 16:36   ` Phil Blundell
2015-11-18 16:49     ` Martin Jansa
2015-11-17 10:58 ` [PATCHv2 10/16] tune-*: use mcpu instead of mtune for ARM tunes Martin Jansa
2015-11-17 10:58 ` [PATCHv2 11/16] arch-armv7ve: add tune include for armv7ve and use it from cortexa7 and cortexa15 Martin Jansa
2015-11-18 16:38   ` Phil Blundell
2015-11-18 16:51     ` Martin Jansa
2015-11-18 17:05       ` Phil Blundell
2015-11-18 17:20         ` Martin Jansa
2015-11-18 17:31           ` Phil Blundell [this message]
2015-11-17 10:58 ` [PATCHv2 12/16] feature-arm-vfp.inc: respect vfpv4 when setting -mfloat-abi and ARMPKGSFX_EABI Martin Jansa
2015-11-18 16:41   ` Phil Blundell
2015-11-18 18:37     ` [PATCHv3 1/6] meta/conf/machine: use ' inside quoted values Martin Jansa
2015-11-18 18:37       ` [PATCHv3 2/6] cortexa{7,15,17}: add VFPv4 tunes Martin Jansa
2015-11-18 22:31         ` Phil Blundell
2015-11-18 18:37       ` [PATCHv3 3/6] tune-*: use mcpu instead of mtune for ARM tunes Martin Jansa
2015-11-18 18:37       ` [PATCHv3 4/6] arch-armv7ve: add tune include for armv7ve and use it from cortexa7 and cortexa15 Martin Jansa
2015-11-18 18:37       ` [PATCHv3 5/6] feature-arm-vfp.inc: respect vfpv4 when setting -mfloat-abi and ARMPKGSFX_EABI Martin Jansa
2015-12-07 22:35         ` Martin Jansa
2015-12-08 10:34           ` Richard Purdie
2015-12-08 14:03             ` Martin Jansa
2015-11-18 18:37       ` [PATCHv3 6/6] feature-arm-vfp.inc: add -mfpu=vfpv4 when vfpv4 is in TUNE_FEATURES Martin Jansa
2015-11-17 10:58 ` [PATCHv2 13/16] " Martin Jansa
2015-11-17 10:58 ` [PATCHv2 14/16] rootfs_*.bbclass: don't add BUILDNAME to do_rootfs vardepsexclude Martin Jansa
2015-11-17 10:58 ` [PATCHv2 15/16] libsecret: add dependency on intltool-native Martin Jansa
2015-11-17 10:58 ` [PATCHv2 16/16] libsdl2: require GLES when building Wayland support Martin Jansa

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=1447867889.1472.63.camel@pbcl.net \
    --to=pb@pbcl.net \
    --cc=martin.jansa@gmail.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