All of lore.kernel.org
 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 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.