From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 4/6] armv7: Use -march=armv7-a and thereby enable Thumb-2
Date: Wed, 23 May 2012 09:20:20 -0600 [thread overview]
Message-ID: <4FBD0034.8060505@wwwdotorg.org> (raw)
In-Reply-To: <20120523150145.GA11718@avionic-0098.mockup.avionic-design.de>
On 05/23/2012 09:01 AM, Thierry Reding wrote:
> * Aneesh V wrote:
>> Enable -march=armv7-a for armv7 platforms if the tool-chain
>> supports it. This in turn results in Thumb-2 code generated for
>> these platforms if CONFIG_SYS_THUMB_BUILD is enabled.
>>
>> Signed-off-by: Aneesh V <aneesh@ti.com> --- I believe armv7-a is
>> fine for all the SoCs except Tegra2 and I see that Tegra2 is
>> already making the necessary exception in
>> .../armv7/tegra2/config.mk
>>
>> Let me know if any other SoC has a problem with armv7-a
>
> I haven't been able to boot any Tegra2 boards with this patch
> applied. I'm guessing that some exceptions for Tegra2 code are
> still missing, though they shouldn't work with armv5 either since
> the AVP is armv4 only. For now I've conditionalized the
> -march=armv7-a on !CONFIG_TEGRA2.
I have the following patch locally, courtesy of Allen Martin. Allen,
are you going to (or did you) post the final version here?
> diff --git a/arch/arm/cpu/armv7/tegra2/config.mk
> b/arch/arm/cpu/armv7/tegra2/config.mk index fe9ef5b..f3ba6f5
> 100644 --- a/arch/arm/cpu/armv7/tegra2/config.mk +++
> b/arch/arm/cpu/armv7/tegra2/config.mk @@ -24,10 +24,12 @@ # MA
> 02111-1307 USA #
>
> -# Tegra has an ARMv4T CPU which runs board_init_f(), so we must
> build this -# file with compatible flags +# Tegra has an ARMv4T CPU
> which runs board_init_f(), so we must build these +# files with
> compatible flags ifdef CONFIG_TEGRA2 CFLAGS_arch/arm/lib/board.o +=
> -march=armv4t +CFLAGS_common/cmd_nvedit.o += -march=armv4t
> +CFLAGS_lib/string.o += -march=armv4t endif
>
> USE_PRIVATE_LIBGCC = yes
next prev parent reply other threads:[~2012-05-23 15:20 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-08 17:20 [U-Boot] [PATCH v4 0/6] Enable Thumb build for ARM platforms Aneesh V
2012-03-08 17:20 ` [U-Boot] [PATCH v4 1/6] arm: adapt asm/linkage.h from Linux Aneesh V
2012-03-11 2:01 ` Mike Frysinger
2012-03-12 8:11 ` Aneesh V
2012-03-08 17:20 ` [U-Boot] [PATCH v4 2/6] armv7: add appropriate headers for assembly functions Aneesh V
2012-03-11 2:02 ` Mike Frysinger
2012-03-08 17:20 ` [U-Boot] [PATCH v4 3/6] ARM: enable Thumb build Aneesh V
2012-03-11 2:02 ` Mike Frysinger
2012-03-12 8:20 ` Aneesh V
2012-03-08 17:20 ` [U-Boot] [PATCH v4 4/6] armv7: Use -march=armv7-a and thereby enable Thumb-2 Aneesh V
2012-05-23 15:01 ` Thierry Reding
2012-05-23 15:20 ` Stephen Warren [this message]
2012-05-24 6:46 ` Thierry Reding
2012-07-06 23:02 ` Marek Vasut
2012-07-06 23:32 ` Tetsuyuki Kobayashi
2012-07-06 23:43 ` Aneesh V
2012-07-07 7:59 ` Marek Vasut
2012-07-07 10:08 ` Albert ARIBAUD
2012-03-08 17:20 ` [U-Boot] [PATCH v4 5/6] omap4+: Avoid using __attribute__ ((__packed__)) Aneesh V
2012-03-08 17:20 ` [U-Boot] [PATCH v4 6/6] OMAP4: enable Thumb build Aneesh V
2012-03-15 13:25 ` [U-Boot] [PATCH v4 0/6] Enable Thumb build for ARM platforms Aneesh V
2012-03-15 16:29 ` Tom Rini
2012-04-03 22:48 ` Allen Martin
2012-04-03 22:59 ` Måns Rullgård
2012-03-15 20:23 ` Tom Rini
2012-03-15 20:46 ` Wolfgang Denk
2012-03-15 20:54 ` Tom Rini
2012-03-15 21:01 ` Wolfgang Denk
2012-03-15 21:12 ` Tom Rini
2012-03-15 22:21 ` Tom Rini
2012-03-16 7:16 ` Wolfgang Denk
2012-03-16 7:48 ` Sricharan R
2012-03-16 14:36 ` Tom Rini
2012-03-17 7:10 ` R, Sricharan
2012-03-16 15:27 ` [U-Boot] [PATCH 1/2] tools, config.mk: Add gcc-version.sh, cc-version test from Linux Tom Rini
2012-03-16 15:27 ` [U-Boot] [PATCH 2/2] Makefile: Add a 'checkthumb' rule Tom Rini
2012-03-16 16:07 ` Mike Frysinger
2012-03-16 17:09 ` Tom Rini
2012-03-16 16:34 ` [U-Boot] [PATCH v2] " Tom Rini
2012-03-16 19:40 ` Mike Frysinger
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=4FBD0034.8060505@wwwdotorg.org \
--to=swarren@wwwdotorg.org \
--cc=u-boot@lists.denx.de \
/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