From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/5] Allow arm tunes which support only thumb
Date: Fri, 25 Apr 2014 11:51:18 -0500 [thread overview]
Message-ID: <535A9286.4010309@windriver.com> (raw)
In-Reply-To: <cover.1398443925.git.Martin.Jansa@gmail.com>
On 4/25/14, 11:40 AM, Martin Jansa wrote:
> Now with 1.7 open for new changes can we please apply first 4 patches?
>
> The following changes since commit 866d2854139e0b00063dde06bc3a34d635ae2fc8:
>
> build-appliance-image: Update to head revision (2014-04-24 17:54:58 +0100)
>
> are available in the git repository at:
>
> git://git.openembedded.org/openembedded-core-contrib jansa/tune
> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/tune
>
> Andrei Gherzan (1):
> tune-cortexm3.inc: Cortex-M3 doesn't support ARM instructions
>
> Martin Jansa (4):
> feature-arm-thumb.inc: Replace inner quotes with apostrophes
> feature-arm-thumb.inc, arch-armv4.inc: Add "arm" to TUNE_FEATURES
> tune-cortexr4.inc: Add thumb and arm to TUNE_FEATURES
> WIP: feature-arm-thumb.inc: respect ARM_INSTRUCTION_SET when adding
> thumb suffix
I looked these over and I do like what I see.
The last WIP item is what I originally wanted to implement.. but at the time was
told not to do it that way... so I'm more then happy to support the package arch
clearly indicating if thumb was enabled for compilation or not.
--Mark
> meta/conf/machine/include/arm/arch-armv4.inc | 3 +-
> 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 +-
> .../conf/machine/include/arm/feature-arm-thumb.inc | 36 ++++++++++++++++------
> meta/conf/machine/include/tune-cortexm3.inc | 2 +-
> meta/conf/machine/include/tune-cortexr4.inc | 2 +-
> 7 files changed, 34 insertions(+), 15 deletions(-)
>
prev parent reply other threads:[~2014-04-25 16:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-25 16:40 [PATCH 0/5] Allow arm tunes which support only thumb Martin Jansa
2014-04-25 16:40 ` [PATCH 1/5] feature-arm-thumb.inc: Replace inner quotes with apostrophes Martin Jansa
2014-04-25 16:40 ` [PATCH 2/5] feature-arm-thumb.inc, arch-armv4.inc: Add "arm" to TUNE_FEATURES Martin Jansa
2014-04-25 16:40 ` [PATCH 3/5] tune-cortexr4.inc: Add thumb and arm " Martin Jansa
2014-04-25 16:40 ` [PATCH 4/5] tune-cortexm3.inc: Cortex-M3 doesn't support ARM instructions Martin Jansa
2014-04-27 15:22 ` Phil Blundell
2014-04-25 16:40 ` [PATCH 5/5] WIP: feature-arm-thumb.inc: respect ARM_INSTRUCTION_SET when adding thumb suffix Martin Jansa
2014-04-25 17:04 ` Richard Purdie
2014-04-25 18:33 ` Denys Dmytriyenko
2014-04-25 16:51 ` Mark Hatle [this message]
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=535A9286.4010309@windriver.com \
--to=mark.hatle@windriver.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