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.orgwrote:
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