Re: [PATCH 0/6] Alsa support for Maemo SDK (n770)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Wed, 1 Nov 2006 13:06:00 -0400,
Eduardo Valentin wrote:
> 
> Hello all,
> 
>    This patch series will add alsa support for Maemo SDK (nokia 770).
> This is an alsa plugin and the patch series is against
> alsa-plugins-1.0.13rc2 source code.
> 
>    The plugin adds alsa support by communicating with n770's audio system.
> 
>    To add ALSA support, an EXT PCM I/O plugin and EXT CTL plugin were
> written. The code is based on SDK documentation from the alsa-project
> site.

Thanks for patches.  I committed them to ALSA HG repo now.

I noticed many compile warnings with maemo plugin like below.  Most of
them are harmless.  But a warning about pthread_mutexattr_settype
should be fixed.


Takashi


if /bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I..    -Wall -fPIC -DPIC -O2 -I/usr/include/alsa    -O2 -Wall -W -pipe -g -MT dsp-protocol.lo -MD -MP -MF ".deps/dsp-protocol.Tpo" -c -o dsp-protocol.lo dsp-protocol.c; \
	then mv -f ".deps/dsp-protocol.Tpo" ".deps/dsp-protocol.Plo"; else rm -f ".deps/dsp-protocol.Tpo"; exit 1; fi
mkdir .libs
 gcc -DHAVE_CONFIG_H -I. -I. -I.. -Wall -fPIC -DPIC -O2 -I/usr/include/alsa -O2 -Wall -W -pipe -g -MT dsp-protocol.lo -MD -MP -MF .deps/dsp-protocol.Tpo -c dsp-protocol.c  -fPIC -DPIC -o .libs/dsp-protocol.o
dsp-protocol.c: In function 'dsp_protocol_create':
dsp-protocol.c:111: warning: implicit declaration of function 'pthread_mutexattr_settype'
dsp-protocol.c: In function 'dsp_protocol_set_volume':
dsp-protocol.c:654: warning: pointer targets in passing argument 2 of 'dsp_protocol_linear2Q15' differ in signedness
dsp-protocol.c:654: warning: pointer targets in passing argument 3 of 'dsp_protocol_linear2Q15' differ in signedness
dsp-protocol.c: In function 'dsp_protocol_set_mic_enabled':
dsp-protocol.c:871: warning: unused parameter 'dsp_protocol'
if /bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I..    -Wall -fPIC -DPIC -O2 -I/usr/include/alsa    -O2 -Wall -W -pipe -g -MT dsp-ctl.lo -MD -MP -MF ".deps/dsp-ctl.Tpo" -c -o dsp-ctl.lo dsp-ctl.c; \
	then mv -f ".deps/dsp-ctl.Tpo" ".deps/dsp-ctl.Plo"; else rm -f ".deps/dsp-ctl.Tpo"; exit 1; fi
 gcc -DHAVE_CONFIG_H -I. -I. -I.. -Wall -fPIC -DPIC -O2 -I/usr/include/alsa -O2 -Wall -W -pipe -g -MT dsp-ctl.lo -MD -MP -MF .deps/dsp-ctl.Tpo -c dsp-ctl.c  -fPIC -DPIC -o .libs/dsp-ctl.o
dsp-ctl.c: In function 'dsp_ctl_elem_list':
dsp-ctl.c:139: warning: comparison between signed and unsigned
dsp-ctl.c: In function 'dsp_ctl_get_attribute':
dsp-ctl.c:224: warning: comparison between signed and unsigned
dsp-ctl.c: In function 'dsp_ctl_get_integer_info':
dsp-ctl.c:253: warning: comparison between signed and unsigned
dsp-ctl.c: In function 'dsp_ctl_read_integer':
dsp-ctl.c:281: warning: comparison between signed and unsigned
dsp-ctl.c:288: warning: pointer targets in passing argument 2 of 'dsp_protocol_get_volume' differ in signedness
dsp-ctl.c:288: warning: pointer targets in passing argument 3 of 'dsp_protocol_get_volume' differ in signedness
dsp-ctl.c: In function 'dsp_ctl_write_integer':
dsp-ctl.c:325: warning: comparison between signed and unsigned
dsp-ctl.c:332: warning: pointer targets in passing argument 2 of 'dsp_protocol_get_volume' differ in signedness
dsp-ctl.c:332: warning: pointer targets in passing argument 3 of 'dsp_protocol_get_volume' differ in signedness
dsp-ctl.c:350: warning: comparison between signed and unsigned
dsp-ctl.c: In function 'dsp_ctl_read_event':
dsp-ctl.c:378: warning: unused parameter 'ext'
dsp-ctl.c:378: warning: unused parameter 'id'
dsp-ctl.c:379: warning: unused parameter 'event_mask'
dsp-ctl.c: In function '_snd_ctl_dsp_ctl_open':
dsp-ctl.c:528: warning: unused parameter 'root'
/bin/sh ../libtool --tag=CC --mode=link gcc -Wall -fPIC -DPIC -O2 -I/usr/include/alsa    -O2 -Wall -W -pipe -g -shared -lm -ldl -lpthread -module -avoid-version -export-dynamic   -o libasound_module_ctl_dsp_ctl.la -rpath /usr/lib/alsa-lib  dsp-protocol.lo dsp-ctl.lo -lasound -lm -ldl -lpthread   -lasound 
gcc -shared  .libs/dsp-protocol.o .libs/dsp-ctl.o  -lm -ldl -lpthread /usr/lib/libasound.so  -Wl,-soname -Wl,libasound_module_ctl_dsp_ctl.so -o .libs/libasound_module_ctl_dsp_ctl.so
creating libasound_module_ctl_dsp_ctl.la
(cd .libs && rm -f libasound_module_ctl_dsp_ctl.la && ln -s ../libasound_module_ctl_dsp_ctl.la libasound_module_ctl_dsp_ctl.la)
if /bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I. -I..    -Wall -fPIC -DPIC -O2 -I/usr/include/alsa    -O2 -Wall -W -pipe -g -MT alsa-dsp.lo -MD -MP -MF ".deps/alsa-dsp.Tpo" -c -o alsa-dsp.lo alsa-dsp.c; \
	then mv -f ".deps/alsa-dsp.Tpo" ".deps/alsa-dsp.Plo"; else rm -f ".deps/alsa-dsp.Tpo"; exit 1; fi
 gcc -DHAVE_CONFIG_H -I. -I. -I.. -Wall -fPIC -DPIC -O2 -I/usr/include/alsa -O2 -Wall -W -pipe -g -MT alsa-dsp.lo -MD -MP -MF .deps/alsa-dsp.Tpo -c alsa-dsp.c  -fPIC -DPIC -o .libs/alsa-dsp.o
alsa-dsp.c: In function 'alsa_dsp_transfer':
alsa-dsp.c:146: warning: comparison between signed and unsigned
alsa-dsp.c:167: warning: label 'out' defined but not used
alsa-dsp.c: In function 'alsa_dsp_hw_params':
alsa-dsp.c:241: warning: unused parameter 'params'
alsa-dsp.c: In function 'alsa_dsp_pause':
alsa-dsp.c:365: warning: unused parameter 'enable'
alsa-dsp.c: In function '_snd_pcm_alsa_dsp_open':
alsa-dsp.c:647: warning: unused parameter 'root'

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux