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 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.