From: dave.martin@linaro.org (Dave Martin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: build ssi-fiq.S in ARM mode to prevent CONFIG_THUMB2_KERNEL build breakage
Date: Thu, 9 Aug 2012 15:50:05 +0100 [thread overview]
Message-ID: <20120809145005.GB17588@linaro.org> (raw)
In-Reply-To: <CAKGA1bniQK9DbZudrHk1mA7FyDZkQzBJbWNUFTFW3pc_2A2x4g@mail.gmail.com>
On Thu, Aug 09, 2012 at 09:32:59AM -0500, Matt Sealey wrote:
> Matt Sealey <matt@genesi-usa.com>
> Product Development Analyst, Genesi USA, Inc.
>
>
> On Thu, Aug 9, 2012 at 5:24 AM, Dave Martin <dave.martin@linaro.org> wrote:
> > On Wed, Aug 08, 2012 at 12:32:39PM -0500, Matt Sealey wrote:
> >
> > [...]
> >
> >> I'm going to do a trapse through and find where Russell nacked Dave's
> >> thumb-aware
> >> rewrite.. would you mind if you have any of these boards seeing if it
> >> really DOES
> >
> > There was no NAK because I didn't get as far as posting the patch,
>
> http://lists.arm.linux.org.uk/lurker/message/20111202.133911.393b6e28.en.html
>
> You did, twice :)
Well, I meant that I had not posted it as a stand-alone patch for
inclusion; I only posted it when commenting on other threads.
> > mostly because of the doubt about whether this code is ever relevant
> > on Thumb2-capable hardware.
>
> It's not, but it does break the build, and without disabling phyCORE boards for
> Thumb2 it will always break the build..
If you want me to push it, let me know. I don't think this should get
merged without some Tested-bys for hardware where the FIQ stuff actually
gets used.
The older post would be the one to use, since that at least got a
reasonable level of build testing.
Cheers
---Dave
next prev parent reply other threads:[~2012-08-09 14:50 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-07 22:45 [PATCH 0/2] ARM: imx: change building of SSI FIQ support for i.MX processors Matt Sealey
2012-08-07 22:45 ` [PATCH 1/2] ARM: build ssi-fiq.S in ARM mode to prevent CONFIG_THUMB2_KERNEL build breakage Matt Sealey
2012-08-08 6:55 ` Sascha Hauer
2012-08-08 17:32 ` Matt Sealey
2012-08-08 19:27 ` Sascha Hauer
2012-08-09 10:24 ` Dave Martin
2012-08-09 14:32 ` Matt Sealey
2012-08-09 14:50 ` Dave Martin [this message]
2012-08-09 16:05 ` Matt Sealey
2012-08-09 16:51 ` Dave Martin
2012-08-07 22:45 ` [PATCH 2/2] ARM: only build ssi-fiq.S et al if CONFIG_SND_IMX_SOC_PCM_FIQ is selected Matt Sealey
2012-08-08 6:52 ` Sascha Hauer
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=20120809145005.GB17588@linaro.org \
--to=dave.martin@linaro.org \
--cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).