[OE-core] [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Phil Blundell]

On Fri, 2011-10-21 at 18:16 +0200, Martin Jansa wrote:
> On Fri, Oct 21, 2011 at 08:40:20AM -0700, Khem Raj wrote:
> > On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> > > Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
> > > ---
> > >  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
> > >  1 files changed, 1 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > index 4ac2418..2f872b8 100644
> > > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > @@ -23,3 +23,4 @@ do_compile_prepend() {
> > >     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
> > >  }
> > >
> > > +ARM_INSTRUCTION_SET = "arm"
> > 
> > not averse to patch but it would be nice to know what fails in thumb mode.
> 
> selected processor does not support Thumb mode `swp r1,r2,[r3]'

If it's only that one error (presumably some kind of home-grown atomic
operation) then let's just fix the code in question rather than forcing
the whole thing to build in ARM-state.  Or if the code is hard to fix,
patch the makefile so that only the particular source files with the
assembly in are built for ARM and let the rest stay as Thumb.  Making
the whole package be ARM is a bit of a heavyweight fix for that issue.

p.