Thanks Sachin, actually, no, it is from pulsecore/svolume_arm.c, unfortunately arm assembler isn't my specialty (yet).

Ubuntu gets around this by building with -mimplicit-it=thumb. Since I have temporarily given up on the panda (re: 708883),  I now have the code working now on beagle using this method, although I agree with Loic that it should be fixed.

Once I get some of the arm buffer cpu wakes experimentation done and published, I'll return to this.

Kurt Taylor (irc krtaylor)


On 9 February 2011 04:00, Sachin Gupta <sachin.gupta@linaro.org> wrote:
Hi Kurt,
 
   the problem seems to be in function pa_volume_s16ne_arm.which seems to be arm optimised version of pa_volume_s16ne_c.For the timebeing for your experimentation you can also avoid using arm optimised pa_volume_s16ne_arm.and let pulse audio use pa_volume_s16ne_c.
 
hope this helps.
 
Thanks
Sachin

On Tue, Feb 8, 2011 at 2:09 AM, Loļc Minier <loic.minier@linaro.org> wrote:
On Mon, Feb 07, 2011, Kurt Taylor wrote:
> I am seeing the following when trying to build pulseaudio on a Beagleboard
> running a current ALIP daily:
> ...
>   CC     libpulsecore_1.0_la-svolume_
> arm.lo
> {standard input}: Assembler messages:
> {standard input}:82: Error: thumb conditional instruction should be in IT
> block -- `addcs r0,r8'
> {standard input}:83: Error: thumb conditional instruction should be in IT
> block -- `movcs r6,r0'
> {standard input}:98: Error: thumb conditional instruction should be in IT
> block -- `addcs r0,r8'
> {standard input}:99: Error: thumb conditional instruction should be in IT
> block -- `movcs r6,r0'
> {standard input}:119: Error: thumb conditional instruction should be in IT
> block -- `addcs r0,r8'
> {standard input}:120: Error: thumb conditional instruction should be in IT
> block -- `movcs r6,r0'
> ...
>
> I am using the normal pulseaudio build (bootstrap.sh, configure, make). The
> build worked fine on a Pandaboard with Ubuntu 10.10. It looks like something
> is not being detected correctly via bootstrap/configure on ALIP.
>
> I have searched and seen commits in archive for adding "-Wa,
> -mimplicit-it=thumb". I have added this to CFLAGS without success.

 The package does a regular configure and then:
   make -C src libpulsecore_0.9.22_la-svolume_arm.lo CFLAGS+=-Wa,-mimplicit-it=thumb
 on ARM.


 This is really ugly though; I guess the asm snippets in
 src/pulsecore/svolume_arm.c should be ported, this is explained in:
https://wiki.ubuntu.com/ARM/Thumb2PortingHowto#Types of Assembly Language

  Bye
--
Loļc Minier

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev