From: Phil Blundell <pb@pbcl.net>
To: Martin Jansa <martin.jansa@gmail.com>,
openembedded-core@lists.openembedded.org
Subject: Re: [PATCHv2 09/16] cortexa{7,15,17}: add VFPv4 tunes
Date: Wed, 18 Nov 2015 16:36:04 +0000 [thread overview]
Message-ID: <1447864564.1472.52.camel@pbcl.net> (raw)
In-Reply-To: <018bb1afb653b342e39f2d9c7a14645913c554ae.1447757562.git.Martin.Jansa@gmail.com>
On Tue, 2015-11-17 at 11:58 +0100, Martin Jansa wrote:
> * it was added only to hf cortexa7 in:
> commit e97d152ca13556b41a236c1a4cfb11e77ff857d7
> Author: Kristof Robot <krirobo@gmail.com>
> Date: Sun Jan 26 10:03:56 2014 +0100
>
> Add Cortex A7 support for NEONv2 & FPv4
>
> * add it to softfp cortexa7 and both versions for cortexa15 and
> cortexa17 tunes
I can't say I'm wildly thrilled about the combinatorial explosion of
tunes that differ only in a few letters. Can't we just enable NEON and
VFPv4 under the existing cortex-a15 tune? Those ISAs are both part of
the Cortex-A15 cpu, right?
Also...
> * we should also update Cortex-A7 and Cortex-A15, to use
> -march=armv7ve,
> but the problem is that oe-core has gcc-4.[89] and gcc-5.2 and gcc
> -4.8
> doesn't support it yet, maybe we should do this change after gcc
> -4.8
> is removed, for details see:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57907
... this part of the commit message is superseded by the following
patch, right?
p.
next prev parent reply other threads:[~2015-11-18 16:36 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 [this message]
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
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=1447864564.1472.52.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