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.gu...@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.min...@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
>>
>
>
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to