From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v5 2/5] arm: support Thumb-1 with CONFIG_SYS_THUMB_BUILD
Date: Tue, 1 Sep 2015 14:48:20 +0200 [thread overview]
Message-ID: <55E59E94.7040802@denx.de> (raw)
In-Reply-To: <55CD976E.1090501@denx.de>
Hi Albert,
On 14.08.2015 09:23, Stefan Roese wrote:
> On 17.07.2015 22:50, Albert ARIBAUD wrote:
>> When building a Thumb-1-only target with CONFIG_SYS_THUMB_BUILD,
>> some files fail to build, most of the time because they include
>> mcr instructions, which only exist for Thumb-2.
>>
>> This patch introduces a Kconfig option CONFIG_THUMB2 and uses
>> it to select between Thumb-2 and ARM mode for the aforementioned
>> files.
>>
>> Signed-off-by: Albert ARIBAUD <albert.u.boot@aribaud.net>
>> ---
>> This patch has been build-tested and run-tested on ED Mini V2,
>> above the "edmini: switch to SPL" patch, and found to reduce
>> U-Boot size by 25% and SPL size by 14%... and to run fine. :)
>>
>> This patch has also been tested against side effects on the
>> non-Thumb wireless_space target. The binaries produced with
>> and without this patch were found to differ only by their
>> version string.
>>
>> Changes in v5: None
>> Changes in v4: None
>> Changes in v3:
>> - added arch/arm/lib/mem{set,cpy}.S to the list of modules
>> which should always be built in ARM state.
>> - Selected HAS_THUMB2 for CPU_V7M.
>> - Fixed invalidate_l2_cache() when building for Thumb-1.
>>
>> Changes in v2:
>> - fixed a typo in the commit message
>> - added file arch/arm/thumb1/include/asm/proc-armv/system.h,
>> which overrides arch/arm/include/asm/proc-armv/system.h
>> when building for Thumb-1 and provides non-functional but
>> Thumb-compilable IRQ and FIQ related macros and functions.
>
> Whats the current status of this patch(set)? Do you plan to pull it in
> soon (when)? Or is something missing?
>
> I'm asking since I have a ST SPEAr600 related patchset here which
> requires this THUMB1 support. And would like to commit this series soon
> as well.
Sorry for bugging you again on this. Any updates? What are the plans?
Thanks,
Stefan
next prev parent reply other threads:[~2015-09-01 12:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-17 20:50 [U-Boot] [PATCH v5 0/5] Add support for Thumb-1 builds Albert ARIBAUD
2015-07-17 20:50 ` [U-Boot] [PATCH v5 1/5] stm32f429-discovery: add CONFIG_SYS_THUMB_BUILD Albert ARIBAUD
2015-07-17 20:50 ` [U-Boot] [PATCH v5 2/5] arm: support Thumb-1 with CONFIG_SYS_THUMB_BUILD Albert ARIBAUD
2015-08-14 7:23 ` Stefan Roese
2015-09-01 12:48 ` Stefan Roese [this message]
2015-07-17 20:50 ` [U-Boot] [PATCH v5 3/5] kirkwood: support CONFIG_SYS_THUMB_BUILD Albert ARIBAUD
2015-07-17 20:50 ` [U-Boot] [PATCH v5 4/5] openrd: switch to CONFIG_SYS_THUMB_BUILD Albert ARIBAUD
2015-07-17 20:50 ` [U-Boot] [PATCH v5 5/5] tricorder: " Albert ARIBAUD
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=55E59E94.7040802@denx.de \
--to=sr@denx.de \
--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