mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Lucas Stach <l.stach@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 5/5] ARM: highbank: fix Thumb build
Date: Wed, 11 Mar 2015 10:54:50 +0100	[thread overview]
Message-ID: <1426067690.30734.3.camel@pengutronix.de> (raw)
In-Reply-To: <20150311083814.GP24885@pengutronix.de>

Am Mittwoch, den 11.03.2015, 09:38 +0100 schrieb Sascha Hauer:
> On Tue, Mar 10, 2015 at 08:59:08PM +0100, Lucas Stach wrote:
> > Note: this only fixes the build failure, I haven't actually tested
> > if it works.
> > 
> > Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> > ---
> >  arch/arm/mach-highbank/smc.S | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Applied all but this one.
> 
> Could you tell us more about the build failure? It builds fine here with
> thumb2 enabled.
> 
The failure is this one:
Error: selected processor does not support Thumb mode `smc #0' 

Now looking at this again it may not only hit the Thumb build, but also
normal ARM mode builds. The check that is triggered here was only later
introduced into binutils, I can't really point at a specific version
though.

So you may not see this error depending on your toolchain version.

Regards,
Lucas
-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2015-03-11  9:55 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-10 19:59 [PATCH 1/5] pbl: let pbl-console depend on !CONSOLE_NONE Lucas Stach
2015-03-10 19:59 ` [PATCH 2/5] ARM: versatile: provide clk_set_rate stub Lucas Stach
2015-03-10 19:59 ` [PATCH 3/5] ARM: zynq: zedboard: don't select serial Lucas Stach
2015-03-10 19:59 ` [PATCH 4/5] ARM: radxa-rock: select options for regulator Lucas Stach
2015-03-10 19:59 ` [PATCH 5/5] ARM: highbank: fix Thumb build Lucas Stach
2015-03-11  8:38   ` Sascha Hauer
2015-03-11  9:54     ` Lucas Stach [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=1426067690.30734.3.camel@pengutronix.de \
    --to=l.stach@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.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