- Alsa-devel - mailman.alsa-project.org

[alsa-devel] pcm low level
by ODINOT Jean-Marc ETRALI 22 Mar '07

22 Mar '07

********************************* This message and any attachments (the "message") are confidential and intended solely for the addressees. Any unauthorised use or dissemination is prohibited. Messages are susceptible to alteration. France Telecom Group shall not be liable for the message if altered, changed or falsified. If you are not the intended addressee of this message, please cancel it immediately and inform the sender. ********************************

1 0

Re: [alsa-devel] [Alsa-devel] Static build of alsa-lib
by Takashi Iwai 22 Mar '07

22 Mar '07

At Tue, 20 Mar 2007 08:13:41 -0700 (PDT), Ciaccia wrote: > > Hi Takashi, > It's clear that the concept of dynamic loading doesn't > conflict with static library, I think my original > question was not completely clear. > > I would like to port a sound application (just PCM, no > midi or other stuff) to an embedded system. The core > of this embedded system is a Cirrus EP9302 ARM > processor, which supports hardware floating points. > > ARM processors have several ABIs (EABI, OABI, ???) and > programs compiled using one ABI are not linkable > (neither at compile time, nor at run time) with other > binaries, because of the different format. This is a > problem for binary distributions (such as Debian), > since the same binary does not work on all ARM > architectures. > > For a reason I really don't understand, EP9302 > hardware floating point binaries only work when > compiled with -static (don't ask me why...), and > therefore I wanted to have a "static" (=without shared > libraries) ALSA application. If this would not be > possible I can always use OSS (which does not requires > shared objects to be loaded at run time), but I still > think it should be possible to develop ALSA > applications for architectures where dynamic loading > is not available... > > Is there a way to achieve this? Sure, what I meant is that the patch had no function to disable the libdl and libpthread explicitly but only checked. The new patch below, for example, can give you options --with-libdl and --with-pthread. For disabling libdl, pass --with-libdl=no. If this works for you, I'll apply it to the upstream. Takashi diff -r 6d0a999aef24 Makefile.am --- a/Makefile.am Fri Mar 16 15:22:27 2007 +0100 +++ b/Makefile.am Tue Mar 20 16:55:23 2007 +0100 @@ -1,4 +1,7 @@ SUBDIRS=doc include src modules -SUBDIRS=doc include src modules +SUBDIRS=doc include src +if BUILD_MODULES +SUBDIRS += modules +endif if BUILD_PCM_PLUGIN_SHM SUBDIRS += aserver endif diff -r 6d0a999aef24 configure.in --- a/configure.in Fri Mar 16 15:22:27 2007 +0100 +++ b/configure.in Tue Mar 20 16:55:00 2007 +0100 @@ -148,6 +148,44 @@ else else AC_MSG_RESULT(no) fi + +ALSA_DEPLIBS="" +if test "$softfloat" != "yes"; then + ALSA_DEPLIBS="-lm" +fi + +dnl Check for libdl +AC_MSG_CHECKING(for libdl) +AC_ARG_WITH(libdl, + [ --with-libdl Use libdl for plugins (default = yes)], + [ have_libdl="$withval" ], [ have_libdl="yes" ]) +if test "$have_libdl" = "yes"; then + AC_CHECK_LIB([dl], [dlsym], [HAVE_LIBDL="yes"]) + if test "$HAVE_LIBDL" = "yes" ; then + ALSA_DEPLIBS="$ALSA_DEPLIBS -ldl" + AC_DEFINE([HAVE_LIBDL], 1, [Have libdl]) + fi +else + AC_MSG_RESULT(no) +fi +AM_CONDITIONAL(BUILD_MODULES, test "$HAVE_LIBDL"="yes") + +dnl Check for pthread +AC_MSG_CHECKING(for pthread) +AC_ARG_WITH(pthread, + [ --with-pthread Use pthread (default = yes)], + [ have_pthread="$withval" ], [ have_pthread="yes" ]) +if test "$have_pthread" = "yes"; then + AC_CHECK_LIB([pthread], [pthread_join], [HAVE_LIBPTHREAD="yes"]) + if test "$HAVE_LIBPTHREAD" = "yes"; then + ALSA_DEPLIBS="$ALSA_DEPLIBS -lpthread" + AC_DEFINE([HAVE_LIBPTHREAD], 1, [Have libpthread]) + fi +else + AC_MSG_RESULT(no) +fi + +AC_SUBST(ALSA_DEPLIBS) dnl Check for architecture AC_MSG_CHECKING(for architecture) @@ -318,6 +356,21 @@ fi if test "$build_pcm_ioplug" = "yes"; then build_pcm_extplug="yes" +fi + +if test "$HAVE_LIBDL" != "yes"; then + build_pcm_meter="no" + build_pcm_ladspa="no" + build_pcm_pcm_ioplug="no" + build_pcm_pcm_extplug="no" +fi + +if test "$HAVE_LIBPTHREAD" != "yes"; then + build_pcm_share="no" +fi + +if test "$softfloat" != "yes"; then + build_pcm_lfloat="no" fi AM_CONDITIONAL(BUILD_PCM_PLUGIN, test x$build_pcm_plugin = xyes) diff -r 6d0a999aef24 include/local.h --- a/include/local.h Fri Mar 16 15:22:27 2007 +0100 +++ b/include/local.h Tue Mar 20 16:16:36 2007 +0100 @@ -36,6 +36,11 @@ #include "config.h" #ifdef SUPPORT_RESMGR #include <resmgr.h> +#endif +#ifdef HAVE_LIBDL +#include <dlfcn.h> +#else +#define RTLD_NOW 0 #endif #define _snd_config_iterator list_head diff -r 6d0a999aef24 src/Makefile.am --- a/src/Makefile.am Fri Mar 16 15:22:27 2007 +0100 +++ b/src/Makefile.am Tue Mar 20 16:16:36 2007 +0100 @@ -41,7 +41,7 @@ libasound_la_LIBADD += alisp/libalisp.la libasound_la_LIBADD += alisp/libalisp.la endif SUBDIRS += compat conf -libasound_la_LIBADD += compat/libcompat.la -lm -ldl -lpthread +libasound_la_LIBADD += compat/libcompat.la @ALSA_DEPLIBS@ libasound_la_LDFLAGS = -version-info $(COMPATNUM) $(VSYMS) diff -r 6d0a999aef24 src/async.c --- a/src/async.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/async.c Tue Mar 20 16:16:36 2007 +0100 @@ -151,9 +151,11 @@ int snd_async_del_handler(snd_async_hand if (!list_empty(&handler->hlist)) goto _end; switch (handler->type) { +#ifdef BUILD_PCM case SND_ASYNC_HANDLER_PCM: err = snd_pcm_async(handler->u.pcm, -1, 1); break; +#endif case SND_ASYNC_HANDLER_CTL: err = snd_ctl_async(handler->u.ctl, -1, 1); break; diff -r 6d0a999aef24 src/conf.c --- a/src/conf.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/conf.c Tue Mar 20 16:16:36 2007 +0100 @@ -415,12 +415,13 @@ beginning:</P> #include <stdarg.h> -#include <dlfcn.h> #include <limits.h> #include <sys/stat.h> -#include <pthread.h> #include <locale.h> #include "local.h" +#ifdef HAVE_LIBPTHREAD +#include <pthread.h> +#endif #ifndef DOC_HIDDEN @@ -3080,7 +3081,9 @@ int snd_config_update_r(snd_config_t **_ return 1; } +#ifdef HAVE_LIBPTHREAD static pthread_mutex_t snd_config_update_mutex = PTHREAD_MUTEX_INITIALIZER; +#endif /** * \brief Updates #snd_config by rereading the global configuration files (if needed). @@ -3099,9 +3102,13 @@ int snd_config_update(void) { int err; +#ifdef HAVE_LIBPTHREAD pthread_mutex_lock(&snd_config_update_mutex); +#endif err = snd_config_update_r(&snd_config, &snd_config_global_update, NULL); +#ifdef HAVE_LIBPTHREAD pthread_mutex_unlock(&snd_config_update_mutex); +#endif return err; } @@ -3128,15 +3135,18 @@ int snd_config_update_free(snd_config_up */ int snd_config_update_free_global(void) { +#ifdef HAVE_LIBPTHREAD pthread_mutex_lock(&snd_config_update_mutex); +#endif if (snd_config) snd_config_delete(snd_config); snd_config = NULL; if (snd_config_global_update) snd_config_update_free(snd_config_global_update); snd_config_global_update = NULL; +#ifdef HAVE_LIBPTHREAD pthread_mutex_unlock(&snd_config_update_mutex); - +#endif /* FIXME: better to place this in another place... */ snd_dlobj_cache_cleanup(); diff -r 6d0a999aef24 src/confmisc.c --- a/src/confmisc.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/confmisc.c Tue Mar 20 16:16:36 2007 +0100 @@ -946,6 +946,8 @@ SND_DLSYM_BUILD_VERSION(snd_func_card_na SND_DLSYM_BUILD_VERSION(snd_func_card_name, SND_CONFIG_DLSYM_VERSION_EVALUATE); #endif +#ifdef BUILD_PCM + /** * \brief Returns the pcm identification of a device. * \param dst The function puts the handle to the result configuration node @@ -1198,6 +1200,8 @@ int snd_func_private_pcm_subdevice(snd_c #ifndef DOC_HIDDEN SND_DLSYM_BUILD_VERSION(snd_func_private_pcm_subdevice, SND_CONFIG_DLSYM_VERSION_EVALUATE); #endif + +#endif /* BUILD_PCM */ /** * \brief Copies the specified configuration node. diff -r 6d0a999aef24 src/control/control.c --- a/src/control/control.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/control/control.c Tue Mar 20 16:16:36 2007 +0100 @@ -47,7 +47,6 @@ and IEC958 structure. #include <string.h> #include <fcntl.h> #include <signal.h> -#include <dlfcn.h> #include <sys/poll.h> #include "control_local.h" diff -r 6d0a999aef24 src/control/hcontrol.c --- a/src/control/hcontrol.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/control/hcontrol.c Tue Mar 20 16:16:36 2007 +0100 @@ -48,11 +48,13 @@ to reduce overhead accessing the real co #include <string.h> #include <fcntl.h> #include <sys/ioctl.h> -#include <pthread.h> #ifndef DOC_HIDDEN #define __USE_GNU #endif #include "control_local.h" +#ifdef HAVE_LIBPTHREAD +#include <pthread.h> +#endif #ifndef DOC_HIDDEN #define NOT_FOUND 1000000000 @@ -420,17 +422,22 @@ static void snd_hctl_sort(snd_hctl_t *hc static void snd_hctl_sort(snd_hctl_t *hctl) { unsigned int k; +#ifdef HAVE_LIBPTHREAD static pthread_mutex_t sync_lock = PTHREAD_MUTEX_INITIALIZER; +#endif assert(hctl); assert(hctl->compare); INIT_LIST_HEAD(&hctl->elems); +#ifdef HAVE_LIBPTHREAD pthread_mutex_lock(&sync_lock); +#endif compare_hctl = hctl; qsort(hctl->pelems, hctl->count, sizeof(*hctl->pelems), hctl_compare); +#ifdef HAVE_LIBPTHREAD pthread_mutex_unlock(&sync_lock); - +#endif for (k = 0; k < hctl->count; k++) list_add_tail(&hctl->pelems[k]->list, &hctl->elems); } diff -r 6d0a999aef24 src/dlmisc.c --- a/src/dlmisc.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/dlmisc.c Tue Mar 20 16:51:13 2007 +0100 @@ -28,7 +28,6 @@ */ #define _GNU_SOURCE -#include <dlfcn.h> #include "list.h" #include "local.h" @@ -53,13 +52,19 @@ void *snd_dlopen(const char *name, int m if (name == NULL) return &snd_dlsym_start; #else +#ifdef HAVE_LIBDL if (name == NULL) { Dl_info dlinfo; if (dladdr(snd_dlopen, &dlinfo) > 0) name = dlinfo.dli_fname; } #endif +#endif +#ifdef HAVE_LIBDL return dlopen(name, mode); +#else + return NULL; +#endif } /** @@ -76,7 +81,11 @@ int snd_dlclose(void *handle) if (handle == &snd_dlsym_start) return 0; #endif +#ifdef HAVE_LIBDL return dlclose(handle); +#else + return 0; +#endif } /** @@ -91,6 +100,7 @@ int snd_dlclose(void *handle) */ static int snd_dlsym_verify(void *handle, const char *name, const char *version) { +#ifdef HAVE_LIBDL int res; char *vname; @@ -107,6 +117,9 @@ static int snd_dlsym_verify(void *handle if (res < 0) SNDERR("unable to verify version for symbol %s", name); return res; +#else + return 0; +#endif } /** @@ -139,10 +152,16 @@ void *snd_dlsym(void *handle, const char return NULL; } #endif - err = snd_dlsym_verify(handle, name, version); - if (err < 0) - return NULL; +#ifdef HAVE_LIBDL + if (version) { + err = snd_dlsym_verify(handle, name, version); + if (err < 0) + return NULL; + } return dlsym(handle, name); +#else + return NULL; +#endif } /* diff -r 6d0a999aef24 src/hwdep/hwdep.c --- a/src/hwdep/hwdep.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/hwdep/hwdep.c Tue Mar 20 16:16:36 2007 +0100 @@ -33,7 +33,6 @@ #include <unistd.h> #include <string.h> #include <fcntl.h> -#include <dlfcn.h> #include <sys/ioctl.h> #include "hwdep_local.h" diff -r 6d0a999aef24 src/mixer/simple_abst.c --- a/src/mixer/simple_abst.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/mixer/simple_abst.c Tue Mar 20 16:52:35 2007 +0100 @@ -82,14 +82,14 @@ static int try_open(snd_mixer_class_t *c free(xlib); return -ENXIO; } - event_func = dlsym(h, "alsa_mixer_simple_event"); + event_func = snd_dlsym(h, "alsa_mixer_simple_event", NULL); if (event_func == NULL) { SNDERR("Symbol 'alsa_mixer_simple_event' was not found in '%s'", xlib); snd_dlclose(h); free(xlib); return -ENXIO; } - init_func = dlsym(h, "alsa_mixer_simple_init"); + init_func = snd_dlsym(h, "alsa_mixer_simple_init", NULL); if (init_func == NULL) { SNDERR("Symbol 'alsa_mixer_simple_init' was not found in '%s'", xlib); snd_dlclose(h); diff -r 6d0a999aef24 src/pcm/pcm.c --- a/src/pcm/pcm.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/pcm/pcm.c Tue Mar 20 16:16:36 2007 +0100 @@ -634,7 +634,6 @@ playback devices. #include <malloc.h> #include <stdarg.h> #include <signal.h> -#include <dlfcn.h> #include <sys/poll.h> #include <sys/shm.h> #include <sys/mman.h> diff -r 6d0a999aef24 src/pcm/pcm_hooks.c --- a/src/pcm/pcm_hooks.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/pcm/pcm_hooks.c Tue Mar 20 16:16:36 2007 +0100 @@ -27,7 +27,6 @@ * */ -#include <dlfcn.h> #include "pcm_local.h" #include "pcm_generic.h" diff -r 6d0a999aef24 src/pcm/pcm_ladspa.c --- a/src/pcm/pcm_ladspa.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/pcm/pcm_ladspa.c Tue Mar 20 16:16:36 2007 +0100 @@ -33,7 +33,6 @@ */ #include <dirent.h> -#include <dlfcn.h> #include <locale.h> #include <math.h> #include "pcm_local.h" diff -r 6d0a999aef24 src/pcm/pcm_rate.c --- a/src/pcm/pcm_rate.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/pcm/pcm_rate.c Tue Mar 20 16:51:41 2007 +0100 @@ -29,7 +29,6 @@ */ #include <inttypes.h> #include <byteswap.h> -#include <dlfcn.h> #include "pcm_local.h" #include "pcm_plugin.h" #include "pcm_rate.h" @@ -1326,7 +1325,7 @@ int snd_pcm_rate_open(snd_pcm_t **pcmp, free(rate); return -ENOENT; } - open_func = dlsym(h, open_name); + open_func = snd_dlsym(h, open_name, NULL); if (! open_func) { SNDERR("Cannot find function %s", open_name); snd_dlclose(h); diff -r 6d0a999aef24 src/rawmidi/rawmidi.c --- a/src/rawmidi/rawmidi.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/rawmidi/rawmidi.c Tue Mar 20 16:16:36 2007 +0100 @@ -139,7 +139,6 @@ This example shows open and read/write r #include <stdarg.h> #include <unistd.h> #include <string.h> -#include <dlfcn.h> #include "rawmidi_local.h" /** diff -r 6d0a999aef24 src/seq/seq.c --- a/src/seq/seq.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/seq/seq.c Tue Mar 20 16:16:36 2007 +0100 @@ -777,7 +777,6 @@ void event_filter(snd_seq_t *seq, snd_se */ -#include <dlfcn.h> #include <sys/poll.h> #include "seq_local.h" diff -r 6d0a999aef24 src/timer/timer.c --- a/src/timer/timer.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/timer/timer.c Tue Mar 20 16:16:36 2007 +0100 @@ -72,7 +72,6 @@ This example shows opening a timer devic #include <unistd.h> #include <string.h> #include <fcntl.h> -#include <dlfcn.h> #include <signal.h> #include <sys/ioctl.h> #include "timer_local.h" diff -r 6d0a999aef24 src/timer/timer_query.c --- a/src/timer/timer_query.c Fri Mar 16 15:22:27 2007 +0100 +++ b/src/timer/timer_query.c Tue Mar 20 16:16:36 2007 +0100 @@ -31,7 +31,6 @@ #include <unistd.h> #include <string.h> #include <fcntl.h> -#include <dlfcn.h> #include <sys/ioctl.h> #include "timer_local.h" diff -r 6d0a999aef24 utils/alsa.pc.in --- a/utils/alsa.pc.in Fri Mar 16 15:22:27 2007 +0100 +++ b/utils/alsa.pc.in Tue Mar 20 16:16:36 2007 +0100 @@ -8,7 +8,7 @@ Version: @VERSION@ Version: @VERSION@ Requires: Libs: -L${libdir} -lasound -Libs.private: -lm -ldl -lpthread +Libs.private: @ALSA_DEPLIBS@ # -I${includedir}/alsa below is just for backward compatibility # (it was set so mistakely in the older version) Cflags: -I${includedir} -I${includedir}/alsa

2 3

[alsa-devel] During ADPCM playback, when is the reference byte set?
by Rask Ingemann Lambertsen 21 Mar '07

21 Mar '07

Hi. When an app is playing ADPCM samples, how does the driver know how to find the reference byte, other than assuming it is the first byte of sample data? Could there be more than one? Etc. -- Rask Ingemann Lambertsen

1 0

[alsa-devel] s/sf.net/alsa-devel.org/
by Rene Herman 21 Mar '07

21 Mar '07

Hi Takashi. This replaces all occurences of alsa-devel(a)lists.s[ource]f[orge].net that a simple recursive grep found in the current HG ALSA repos by alsa-devel(a)alsa-project.org. Signed-off-by: Rene Herman <rene.herman(a)gmail.com> Rene. diff -urN alsa.orig/alsa-driver/pci/bt87x.patch alsa/alsa-driver/pci/bt87x.patch --- alsa.orig/alsa-driver/pci/bt87x.patch 2006-06-29 23:22:39.000000000 +0200 +++ alsa/alsa-driver/pci/bt87x.patch 2007-03-20 17:52:59.000000000 +0100 @@ -39,7 +39,7 @@ + pci->device, subsystem_vendor, subsystem_device); snd_printk(KERN_DEBUG "please mail id, board name, and, " "if it works, the correct digital_rate option to " - "<alsa-devel(a)lists.sf.net>\n"); + "<alsa-devel(a)alsa-project.org>\n"); @@ -935,3 +948,5 @@ module_init(alsa_card_bt87x_init) diff -urN alsa.orig/alsa-kernel/Documentation/ALSA-Configuration.txt alsa/alsa-kernel/Documentation/ALSA-Configuration.txt --- alsa.orig/alsa-kernel/Documentation/ALSA-Configuration.txt 2007-02-26 17:00:05.000000000 +0100 +++ alsa/alsa-kernel/Documentation/ALSA-Configuration.txt 2007-03-20 17:54:24.000000000 +0100 @@ -2049,4 +2049,4 @@ https://bugtrack.alsa-project.org/bugs/ ALSA Developers ML - mailto:alsa-devel@lists.sourceforge.net + mailto:alsa-devel@alsa-project.org diff -urN alsa.orig/alsa-kernel/Documentation/Bt87x.txt alsa/alsa-kernel/Documentation/Bt87x.txt --- alsa.orig/alsa-kernel/Documentation/Bt87x.txt 2006-06-29 23:25:22.000000000 +0200 +++ alsa/alsa-kernel/Documentation/Bt87x.txt 2007-03-20 17:55:06.000000000 +0100 @@ -36,8 +36,8 @@ other values than the default 32000 (often it's 44100 or 64000). If you have an unknown card, please mail the ID and board name to -<alsa-devel(a)lists.sf.net>, regardless of whether audio capture works or -not, so that future versions of this driver know about your card. +<alsa-devel(a)alsa-project.org>, regardless of whether audio capture works +or not, so that future versions of this driver know about your card. Audio modes diff -urN alsa.orig/alsa-kernel/pci/bt87x.c alsa/alsa-kernel/pci/bt87x.c --- alsa.orig/alsa-kernel/pci/bt87x.c 2007-03-06 11:30:32.000000000 +0100 +++ alsa/alsa-kernel/pci/bt87x.c 2007-03-20 17:56:40.000000000 +0100 @@ -835,7 +835,7 @@ pci->device, pci->subsystem_vendor, pci->subsystem_device); snd_printk(KERN_DEBUG "please mail id, board name, and, " "if it works, the correct digital_rate option to " - "<alsa-devel(a)lists.sf.net>\n"); + "<alsa-devel(a)alsa-project.org>\n"); return 32000; /* default rate */ } diff -urN alsa.orig/alsa-kernel/pci/cs46xx/cs46xx_lib.c alsa/alsa-kernel/pci/cs46xx/cs46xx_lib.c --- alsa.orig/alsa-kernel/pci/cs46xx/cs46xx_lib.c 2006-11-21 18:00:06.000000000 +0100 +++ alsa/alsa-kernel/pci/cs46xx/cs46xx_lib.c 2007-03-20 17:55:54.000000000 +0100 @@ -3107,7 +3107,7 @@ snd_printk(KERN_ERR "ERROR: snd-cs46xx: never read ISV3 & ISV4 from AC'97\n"); snd_printk(KERN_ERR " Try reloading the ALSA driver, if you find something\n"); snd_printk(KERN_ERR " broken or not working on your soundcard upon\n"); - snd_printk(KERN_ERR " this message please report to alsa-devel(a)lists.sourceforge.net\n"); + snd_printk(KERN_ERR " this message please report to alsa-devel(a)alsa-project.org\n"); return -EIO; #endif diff -urN alsa.orig/alsa-lib/src/conf/cards/USB-Audio.conf alsa/alsa-lib/src/conf/cards/USB-Audio.conf --- alsa.orig/alsa-lib/src/conf/cards/USB-Audio.conf 2007-02-13 10:00:05.000000000 +0100 +++ alsa/alsa-lib/src/conf/cards/USB-Audio.conf 2007-03-20 17:57:20.000000000 +0100 @@ -9,7 +9,7 @@ # cards.USB-Audio.pcm.use_dmix."NoiseBlaster 3000" no # # If your device requires such a definition to work correctly, please report it -# to <alsa-devel(a)lists.sourceforge.net>. +# to <alsa-devel(a)alsa-project.org>. # If a device has sample formats not supported by dmix, dmix can be disabled

2 1

[alsa-devel] Problem in dummy driver
by Himanshu Chauhan 21 Mar '07

21 Mar '07

Dear all, I am working on the dummy driver provided with the ALSA. I took it from the linux kernel 2.6.20.1 I build the module and load it. The XMMS seems to play ok (doesn't hang and all) but none of the recording application seem to record from the driver. Can the driver in current state work as the loop back cable between applications? Thanks in advance... Warm Regards --Himanshu

2 1

[alsa-devel] RESUBSCRIBE - New alsa-devel mailing list
by Jaroslav Kysela 21 Mar '07

21 Mar '07

Hello everyone, because never ending story with SF services, I quickly setup new mailman alsa-devel list on new ALSA server (we'll get working list archive as bonus). Because I don't want to move automatically all subscribers, please, subscribe yourself to new list: http://mailman.alsa-project.org/mailman/listinfo/alsa-devel New list is available as alsa-devel(a)alsa-project.org while old on SF is available as alsa-devel(a)lists.sourceforge.net . The archive for new alsa-devel mailing list is here: http://mailman.alsa-project.org/pipermail/alsa-devel/ The posts from non-members are moderated now! Jaroslav ----- Jaroslav Kysela <perex(a)suse.cz> Linux Kernel Sound Maintainer ALSA Project, SUSE Labs

6 9

Re: [alsa-devel] Development Question
by M. Gagnon 21 Mar '07

21 Mar '07

> > >What is the output of the program? > > > > pending events size: 28 > > pending events size: 56 > > pending events size: 84 > > ... > > BEFORE DRAIN: pending events size: 476 > > AFTER DRAIN: pending events size: 0 > >Looks OK. In theory, all events should be sent somewhere. > >Please run aseqdump and send the events to its port. What does it >output? > % aseqdump Waiting for data at port 128:0. Press Ctrl+C to end. Source_ Event_________________ Ch _Data__ 0:1 Port subscribed 129:0 -> 128:0 129:0 Note on 2 60 127 0:1 Port unsubscribed 129:0 -> 128:0 Seems like only one event gets sent - just like what i hear. >Run "cat /proc/asound/seq/queues" and >"cat /proc/asound/seq/timer". > % cat /proc/asound/seq/queues queue 0: [Queue-0] owned by client : 128 lock status : Locked queued time events : 0 queued tick events : 0 timer state : Running timer PPQ : 96 current tempo : 500000 current BPM : 120 current time : 1.536000000 s current tick : 294 queue 1: [Queue-1] owned by client : 129 lock status : Locked queued time events : 0 queued tick events : 0 timer state : Running timer PPQ : 48 current tempo : 1000000 current BPM : 60 current time : 1.488000000 s current tick : 71 % cat /proc/asound/seq/timer Timer for queue 0 : system timer Period time : 0.004000000 Skew : 65536 / 65536 Timer for queue 1 : system timer Period time : 0.004000000 Skew : 65536 / 65536 thanks for your help just as reminder (so the output can be interpreted) this is my latest code /* all: g++ midi_receiver.cpp -o AlsaNoteTest -lasound */ #include <stdio.h> #include <alsa/asoundlib.h> #include "midi_receiver.h" #include <vector> #include <iostream> #include <unistd.h> #include <pthread.h> pthread_t thread1; char *message1 = "Thread 1"; int iret1; void *play_func( void *ptr ) { AlsaNotePlayer::play(); } int main() { AlsaNotePlayer::findDevices(); AlsaNotePlayer::openDevice(128, 0); AlsaNotePlayer::setTempo(60, 48); int time=0; int duration = 50; AlsaNotePlayer::noteOn_seq(1, 60, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 60, 0, time); AlsaNotePlayer::noteOn_seq(1, 62, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 62, 0, time); AlsaNotePlayer::noteOn_seq(1, 60, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 60, 0, time); AlsaNotePlayer::noteOn_seq(1, 59, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 59, 0, time); // ---------- AlsaNotePlayer::noteOn_seq(1, 63, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 63, 0, time); AlsaNotePlayer::noteOn_seq(1, 65, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 65, 0, time); AlsaNotePlayer::noteOn_seq(1, 63, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 63, 0, time); AlsaNotePlayer::noteOn_seq(1, 61, 127, time); time += duration; AlsaNotePlayer::noteOff_seq(1, 61, 0, time); iret1 = pthread_create( &thread1, NULL, play_func, (void*) message1); pthread_join( thread1, NULL); while(1) { usleep(10000); } } namespace AlsaNotePlayer { class MidiDevice { public: int client, port; char* name; MidiDevice(int client_arg, int port_arg, const char* name_arg) { client = client_arg; port = port_arg; name = (char*)name_arg; } }; std::vector<MidiDevice> devices; struct MidiContext { snd_seq_t* sequencer; int queue; snd_seq_addr_t address; int port; snd_seq_port_subscribe_t *subs; }; struct MidiContext MidiContext; snd_seq_addr_t address; void findDevices() { snd_seq_client_info_t* clientInfo; if (snd_seq_open(&MidiContext.sequencer, "default", SND_SEQ_OPEN_OUTPUT, 0) < 0) { return; } snd_seq_client_info_alloca(&clientInfo); snd_seq_client_info_set_client(clientInfo, -1); // iterate through clients while (snd_seq_query_next_client(MidiContext.sequencer, clientInfo) >= 0) { snd_seq_port_info_t* pinfo; snd_seq_port_info_alloca(&pinfo); snd_seq_port_info_set_client(pinfo, snd_seq_client_info_get_client(clientInfo)); snd_seq_port_info_set_port(pinfo, -1); // and now through ports while (snd_seq_query_next_port(MidiContext.sequencer, pinfo) >= 0) { unsigned int capability = snd_seq_port_info_get_capability(pinfo); if ((capability & SND_SEQ_PORT_CAP_SUBS_WRITE) == 0) { continue; } int client = (snd_seq_port_info_get_addr(pinfo))->client; int port = (snd_seq_port_info_get_addr(pinfo))->port; std::cout << client << ":" << port << " --> " << snd_seq_port_info_get_name(pinfo) << std::endl; devices.push_back( MidiDevice(client, port, snd_seq_port_info_get_name(pinfo)) ); } } if (snd_seq_set_client_name(MidiContext.sequencer, "Aria") < 0) { return; } //MidiContext.address.port = snd_seq_create_simple_port(MidiContext.sequencer, "Aria Port 0", SND_SEQ_PORT_CAP_SUBS_WRITE, SND_SEQ_PORT_TYPE_APPLICATION); MidiContext.address.port = snd_seq_create_simple_port(MidiContext.sequencer, "Aria Port 0", SND_SEQ_PORT_CAP_WRITE | SND_SEQ_PORT_CAP_SUBS_WRITE | SND_SEQ_PORT_CAP_READ, SND_SEQ_PORT_TYPE_APPLICATION); //SND_SEQ_PORT_TYPE_MIDI_GENERIC); MidiContext.address.client = snd_seq_client_id (MidiContext.sequencer); MidiContext.queue = snd_seq_alloc_queue (MidiContext.sequencer); snd_seq_set_client_pool_output (MidiContext.sequencer, 1024); } void setTempo(int tempo_arg, int resolution) { // argument is not yet considered snd_seq_queue_tempo_t *tempo; snd_seq_queue_tempo_alloca(&tempo); // snd_seq_queue_tempo_set_tempo(tempo, 1000000); // 60 BPM // snd_seq_queue_tempo_set_ppq(tempo, 48); // 48 PPQ snd_seq_queue_tempo_set_ppq(tempo, resolution); snd_seq_queue_tempo_set_tempo(tempo, 60*1000000/tempo_arg); snd_seq_set_queue_tempo(MidiContext.sequencer, MidiContext.queue, tempo); } void openDevice(int client, int port) { address.client = client; address.port = port; snd_seq_port_subscribe_alloca(&MidiContext.subs); snd_seq_port_subscribe_set_sender(MidiContext.subs, &MidiContext.address); snd_seq_port_subscribe_set_dest(MidiContext.subs, &address); snd_seq_subscribe_port(MidiContext.sequencer, MidiContext.subs); } void closeDevice() { snd_seq_port_subscribe_alloca(&MidiContext.subs); snd_seq_port_subscribe_set_sender(MidiContext.subs, &MidiContext.address); snd_seq_port_subscribe_set_dest(MidiContext.subs, &address); if (snd_seq_unsubscribe_port(MidiContext.sequencer, MidiContext.subs) < 0) { return; } //snd_seq_port_subscribe_free(MidiContext.subs); } //************************************************************************************// //************************************MIDI EVENTS*************************************// //************************************************************************************// void noteOn_seq(int channel, int note, int velocity, int tick) { snd_seq_event_t event; snd_seq_ev_clear(&event); event.queue = MidiContext.queue; event.source = MidiContext.address; snd_seq_ev_set_subs(&event); snd_seq_ev_set_source(&event, MidiContext.port); snd_seq_ev_set_noteon(&event, channel, note, velocity); snd_seq_ev_schedule_tick(&event, MidiContext.queue, 0, tick ); snd_seq_event_output(MidiContext.sequencer, &event); std::cout << "pending events size: " << snd_seq_event_output_pending(MidiContext.sequencer) << std::endl; } void noteOff_seq(int channel, int note, int velocity, int tick) { snd_seq_event_t event; snd_seq_ev_clear(&event); event.queue = MidiContext.queue; event.source = MidiContext.address; snd_seq_ev_set_subs(&event); snd_seq_ev_set_noteoff(&event, channel, note, velocity); snd_seq_ev_schedule_tick(&event, MidiContext.queue, 0 /*absolute*/, tick ); snd_seq_event_output(MidiContext.sequencer, &event); std::cout << "pending events size: " << snd_seq_event_output_pending(MidiContext.sequencer) << std::endl; } void play() { snd_seq_start_queue(MidiContext.sequencer, MidiContext.queue, 0); std::cout << "BEFORE DRAIN: pending events size: " << snd_seq_event_output_pending(MidiContext.sequencer) << std::endl; int output = snd_seq_drain_output(MidiContext.sequencer); std::cout << "AFTER DRAIN: pending events size: " << snd_seq_event_output_pending(MidiContext.sequencer) << std::endl; if(output == 0) std::cout << "all events drained" << std::endl; if(output > 0) std::cout << output << "bytes remaining" << std::endl; if(output < 0) { fprintf (stderr, "Error: (%s)\n", snd_strerror (output)); //std::cout << "error" << std::endl; } } } _________________________________________________________________ Ne perdez pas de temps dans les files dattente magasinez en ligne. http://magasiner.sympatico.msn.ca

1 0

Re: [alsa-devel] [Alsa-devel] [RFC][RFT] Adding support for Jazz16 based sound cards
by Rene Herman 20 Mar '07

20 Mar '07

On 03/19/2007 11:46 PM, Rask Ingemann Lambertsen wrote: >> This setup also shares that same problem with the BIOS generally >> not having reserved/routed the resources -- on a general PCI/ISA >> system the user needs to go into the BIOS setup and reserve the >> resources the card is (or will be, in this case) set to. So now the >> user really needs to know what resources are going to be assigned, >> destroying the plug and play idea... > > Not really. A standards compliant PnPBIOS set up with "PnP aware OS = > YES" will use only enough resources to boot the system, which should > leave enough irqs and dmas for the Linux PnP layer to do its work. There's also the "routed" bit though. I actually ran into this recently when I was playing with snd-sgalaxy (which is why I'm harping on it). An sgalaxy is also a (partly) software configurable card and when not given explicit parameters the driver just searches for an unrequested IRQ and DMA and then programs the card for it. Over here it grabbed IRQ 10 but then didn't work anyway, and only started working when I set IRQ 10 to "Legacy ISA" in the BIOS setup. It seems the IRQ simply was not available to ISA without having done that (no, no PCI cards were using IRQ 10 or anything). I must say though that my comment above was partly based on a somewhat foggy understanding of how things get assigned. Yes, _if_ this "routing" problem I seemed to be experiencing is not a serious problem in practice (Adam? Are you able to comment?) then the fact that the PnP layer will be handing out the resources taking into account the other (known) cards is indeed a practical advantage of using that layer. > I think there are three good reasons to use the PnP layer: > > 1) The PnP layer informs the PCI layer of irqs to try to avoid - please > see pnp_register_irq_resource() in drivers/pnp/resource.c. How does that help? On my systems (one AMI, all others Award BIOses) all PCI cards get assigned their IRQ before the OS ever boots. That "PnP OS: Yes/No" switch only effects whether or not the BIOS will fully configure all ISA-PnP cards. > 2) The PnP layer knows which resources are used by active devices, even > if they have not been allocated using e.g. request_region(). This > happens in practice. For example: Yes, this is an advantage. It does miss all the resources used by old ISA cards for which no (normal or pnp protocol as in this one) drivers have been loaded but I guess that's not much of a difference with a regular ISA driver... > 3) Knowing which resources a card can utilize, the PnP layer can try to > stay away from those resources when configuring other cards. The PnP layer > doesn't do that yet in a stock kernel, but I've patched it to do so. I'll > try to get those patches into the kernel. I guess that could be an advantage. When I was looking at pnp/isa integration a while ago myself I was looking at it from the angle of generic old ISA (ie, hardwired non settable resources) and concluded that for these there wasn't any point. Having looked a little closer at the resource assigment being done in the PnP layer now, I now feel there's some point, but still little for those. But with software settable devices such as this Jazz16, there ofcourse is rather more point. I believe the routing issue I refferred to above is the most important thing to have answered. I also still don't really like "splitting the one driver into the two pieces" like that but well, we don't actually have all that many software settable ISA devices and I guess it's not _too_ bad to have a few of these "front-ends" run as a protocol driver at boot as long as it stays "a few". Hey, I already said I like the concept... By the way; in the "straightforward version" I posted yesterday I neglected to reserve the SB_JAZZ16_CONFIG_PORT and the SB range before poking at it during setup. Attached version corrects that. Updated patch including your SB changes still at http://members.home.nl/rene.herman/jazz16.diff Looks more involved now, but posted mainly for completeness; correcting yesterday's post. Has been tested and found working again. Rene. diff --git a/Documentation/sound/alsa/ALSA-Configuration.txt b/Documentation/sound/alsa/ALSA-Configuration.txt index 9fef210..b60690a 100644 --- a/Documentation/sound/alsa/ALSA-Configuration.txt +++ b/Documentation/sound/alsa/ALSA-Configuration.txt @@ -1120,6 +1120,13 @@ Prior to version 0.9.0rc4 options had a This module supports multiple cards, autoprobe and ISA PnP. + Module snd-jazz16 + ----------------- + + Module for sound cards based Media Vision Jazz16 chip + + This module supports multiple cards. + Module snd-korg1212 ------------------- diff --git a/include/sound/sb.h b/include/sound/sb.h index 2dd5c8e..9594f89 100644 --- a/include/sound/sb.h +++ b/include/sound/sb.h @@ -38,6 +38,7 @@ enum sb_hw_type { SB_HW_ALS100, /* Avance Logic ALS100 chip */ SB_HW_ALS4000, /* Avance Logic ALS4000 chip */ SB_HW_DT019X, /* Diamond Tech. DT-019X / Avance Logic ALS-007 */ + SB_HW_JAZZ16, /* Media Vision Jazz16 chip */ }; #define SB_OPEN_PCM 0x01 @@ -140,8 +141,11 @@ #define SB_DSP_LO_OUTPUT_AUTO 0x1c #define SB_DSP_LO_INPUT_AUTO 0x2c #define SB_DSP_HI_OUTPUT_AUTO 0x90 #define SB_DSP_HI_INPUT_AUTO 0x98 +#define SB_DSP_ULAW_OUTPUT_AUTO 0x7f +#define SB_DSP_ULAW_INPUT_AUTO 0x8f #define SB_DSP_IMMED_INT 0xf2 #define SB_DSP_GET_VERSION 0xe1 +#define SB_DSP_GET_JAZZ_VERSION 0xfa #define SB_DSP_SPEAKER_ON 0xd1 #define SB_DSP_SPEAKER_OFF 0xd3 #define SB_DSP_DMA8_OFF 0xd0 @@ -265,6 +269,14 @@ #define SB_DMASETUP_DMA5 0x20 #define SB_DMASETUP_DMA6 0x40 #define SB_DMASETUP_DMA7 0x80 +/* Jazz16 configuration. */ +#define SB_JAZZ16_CONFIG_PORT 0x201 +#define SB_JAZZ16_WAKEUP 0xaf +#define SB_JAZZ16_SET_PORTS 0x50 + +/* Check the output of SB_DSP_GET_JAZZ_VERSION. */ +#define SB_VERSION_IS_JAZZ16(x) ((x) == 0x12) + /* * */ diff --git a/sound/isa/Kconfig b/sound/isa/Kconfig index a629b52..450255f 100644 --- a/sound/isa/Kconfig +++ b/sound/isa/Kconfig @@ -258,6 +258,14 @@ config SND_SW60XG To compile this driver as a module, choose M here: the module will be called snd-sw60xg. +config SND_JAZZ16 + tristate "Media Vision Jazz16" + depends on SND + select SND_OPL3_LIB + select SND_PCM + help + Say 'Y' or 'M' to include support for Media Vision Jazz16. + config SND_OPL3SA2 tristate "Yamaha OPL3-SA2/SA3" depends on SND diff --git a/sound/isa/sb/Makefile b/sound/isa/sb/Makefile index fd9d9c5..745e1c1 100644 --- a/sound/isa/sb/Makefile +++ b/sound/isa/sb/Makefile @@ -12,6 +12,7 @@ snd-sb16-objs := sb16.o snd-sbawe-objs := sbawe.o emu8000.o snd-emu8000-synth-objs := emu8000_synth.o emu8000_callback.o emu8000_patch.o emu8000_pcm.o snd-es968-objs := es968.o +snd-jazz16-objs := jazz16.o # # this function returns: @@ -30,6 +31,7 @@ obj-$(CONFIG_SND_SB16) += snd-sb16.o snd obj-$(CONFIG_SND_SBAWE) += snd-sbawe.o snd-sb16-dsp.o snd-sb-common.o obj-$(CONFIG_SND_ES968) += snd-es968.o snd-sb8-dsp.o snd-sb-common.o obj-$(CONFIG_SND_ALS4000) += snd-sb-common.o +obj-$(CONFIG_SND_JAZZ16) += snd-jazz16.o snd-sb8-dsp.o snd-sb-common.o ifeq ($(CONFIG_SND_SB16_CSP),y) obj-$(CONFIG_SND_SB16) += snd-sb16-csp.o obj-$(CONFIG_SND_SBAWE) += snd-sb16-csp.o diff --git a/sound/isa/sb/jazz16.c b/sound/isa/sb/jazz16.c new file mode 100644 index 0000000..988a5e9 --- /dev/null +++ b/sound/isa/sb/jazz16.c @@ -0,0 +1,334 @@ +/* Driver for Media Vision Jazz16 based boards. + * + * Copyright (c) 2007 by Rask Ingemann Lambertsen + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + * + * The Jazz16 is an SB Pro compatible chip with a 16-bit mode and higher + * playback and capture rates added to it. It is not SB 16 compatible. + * The chip has an MPU-401 interface which is handled by the MPU-401 driver. + * + * The IBM PPS Model 6015 has a Jazz16 chip on board. Please tell me if it + * works with this driver or not. + */ + +#include <sound/driver.h> +#include <linux/module.h> +#include <linux/moduleparam.h> +#include <linux/kernel.h> +#include <linux/init.h> +#include <linux/isa.h> +#include <linux/ioport.h> +#include <asm/dma.h> +#include <asm/io.h> +#include <sound/core.h> +#include <sound/sb.h> +#include <sound/mpu401.h> +#include <sound/opl3.h> +#include <sound/initval.h> + +#define CRD_NAME "Media Vision Jazz16" +#define DEV_NAME "jazz16" + +MODULE_DESCRIPTION(CRD_NAME); +MODULE_AUTHOR("Rask Ingemann Lambertsen <rask(a)sygehus.dk>"); +MODULE_LICENSE("GPL"); + +static int index = SNDRV_DEFAULT_IDX1; /* Index 0-MAX */ +static char *id = SNDRV_DEFAULT_STR1; /* ID for this card */ +static int enable = SNDRV_DEFAULT_ENABLE1; /* Enable this card */ +static long port = SNDRV_DEFAULT_PORT1; /* 0x220,0x240,0x260 */ +static long mpu_port = SNDRV_DEFAULT_PORT1; /* 0x310,0x320,0x330 */ +static int irq = SNDRV_DEFAULT_IRQ1; /* 5,9,3,7,10,15 */ +static int mpu_irq = SNDRV_DEFAULT_IRQ1; /* 5,9,3,7,10,15 */ +static int dma8 = SNDRV_DEFAULT_DMA1; /* 1,3 */ +static int dma16 = SNDRV_DEFAULT_DMA1; /* 5,7 */ + +module_param(index, int, 0444); +MODULE_PARM_DESC(index, "Index value for " CRD_NAME " soundcard."); +module_param(id, charp, 0444); +MODULE_PARM_DESC(id, "ID string for " CRD_NAME " soundcard."); +module_param(enable, bool, 0444); +MODULE_PARM_DESC(enable, "Enable " CRD_NAME " soundcard."); +module_param(port, long, 0444); +MODULE_PARM_DESC(port, "Port # for " CRD_NAME " driver."); +module_param(mpu_port, long, 0444); +MODULE_PARM_DESC(mpu_port, "MPU-401 port # for " CRD_NAME " driver."); +module_param(irq, int, 0444); +MODULE_PARM_DESC(irq, "IRQ # for " CRD_NAME " driver."); +module_param(mpu_irq, int, 0444); +MODULE_PARM_DESC(mpu_irq, "MPU-401 IRQ # for " CRD_NAME " driver."); +module_param(dma8, int, 0444); +MODULE_PARM_DESC(dma8, "8-bit DMA # for " CRD_NAME " driver."); +module_param(dma16, int, 0444); +MODULE_PARM_DESC(dma16, "16-bit DMA # for " CRD_NAME " driver."); + +static int __devinit snd_jazz16_match(struct device *dev, unsigned int n) +{ + if (!enable) + return 0; + + if (port == SNDRV_AUTO_PORT) { + snd_printk(KERN_ERR "%s: please specify port\n", dev->bus_id); + return 0; + } + if (irq == SNDRV_AUTO_IRQ) { + snd_printk(KERN_ERR "%s: please specify irq\n", dev->bus_id); + return 0; + } + if (dma8 == SNDRV_AUTO_DMA) { + snd_printk(KERN_ERR "%s: please specify dma8\n", dev->bus_id); + return 0; + } + if (dma16 == SNDRV_AUTO_DMA) { + snd_printk(KERN_ERR "%s: please specify dma16\n", dev->bus_id); + return 0; + } + return 1; +} + +#define BUSY_LOOPS 100000 + +static int __devinit jazz16_sbdsp_command(u8 value) +{ + int i; + + for (i = BUSY_LOOPS; i; i--) + if ((inb(SBP1(port, STATUS)) & 0x80) == 0) { + outb(value, SBP1(port, COMMAND)); + return 0; + } + return -EIO; +} + +static int __devinit jazz16_sbdsp_get_byte(void) +{ + int i; + + for (i = BUSY_LOOPS; i; i--) + if (inb(SBP1(port, DATA_AVAIL)) & 0x80) + return inb(SBP1(port, READ)); + + return -EIO; +} + +static int __devinit jazz16_sbdsp_reset(void) +{ + int error; + + outb(1, SBP1(port, RESET)); + udelay(10); + outb(0, SBP1(port, RESET)); + udelay(30); + + error = jazz16_sbdsp_get_byte(); + if (error < 0) + return error; + + return error == 0xaa ? 0 : -ENODEV; +} + +static void jazz16_set_ports(u8 value) +{ + outb(SB_JAZZ16_WAKEUP, SB_JAZZ16_CONFIG_PORT); + outb(SB_JAZZ16_SET_PORTS, SB_JAZZ16_CONFIG_PORT); + outb(value, SB_JAZZ16_CONFIG_PORT); +} + +#define JAZZ16_SET_DMAINTR 0xfb + +static int __devinit jazz16_enable(struct device *dev) +{ + static u8 irq_config[] = {0,0,2,3,0,1,0,4,0,2,5,0,0,0,0,6}; + static u8 dma_config[] = {0,1,0,2,0,3,0,4}; + u8 config = 0; + + int error; + + if (port != 0x220 && port != 0x240 && port != 0x260) + port = 0; + + if (mpu_port != 0x310 && mpu_port != 0x320 && mpu_port != 0x330) + mpu_port = 0; + + config = (port & 0x70) | ((mpu_port >> 4) & 0x07); + + snd_printd("%s: setting SB port = %#lx, MPU port = %#lx.\n", + dev->bus_id, port, mpu_port); + + jazz16_set_ports(config); + + snd_printd("%s: setting SB irq %d, dma %d&%d, MPU irq %d.\n", + dev->bus_id, irq, dma8, dma16, mpu_irq); + + if (!request_region(port, 16, "Jazz16")) + return -EBUSY; + + error = jazz16_sbdsp_command(JAZZ16_SET_DMAINTR); + if (error < 0) + goto out; + + config = dma_config[dma8] | dma_config[dma16] << 4; + + error = jazz16_sbdsp_command(config); + if (error < 0) + goto out; + + config = irq_config[irq] | irq_config[mpu_irq] << 4; + + error = jazz16_sbdsp_command(config); + if (error < 0) + goto out; + + error = jazz16_sbdsp_reset(); + if (error < 0) + goto out; + + error = jazz16_sbdsp_command(SB_DSP_GET_JAZZ_VERSION); + if (error < 0) + goto out; + + error = jazz16_sbdsp_get_byte(); + if (error < 0) + goto out; + + error = SB_VERSION_IS_JAZZ16(error) ? 0 : -ENODEV; + +out: release_region(port, 16); + return error; +} + +static irqreturn_t snd_jazz16_interrupt(int irq, void *dev_id) +{ + return snd_sb8dsp_interrupt(dev_id); +} + +static void snd_jazz16_free(struct snd_card *card) +{ + jazz16_set_ports(0); + release_and_free_resource(card->private_data); +} + +static int __devinit snd_jazz16_probe(struct device *dev, unsigned int n) +{ + struct snd_sb *chip; + struct snd_card *card; + struct snd_opl3 *opl3; + int error; + + card = snd_card_new(index, id, THIS_MODULE, 0); + if (!card) + return -EINVAL; + + card->private_data = request_region(SB_JAZZ16_CONFIG_PORT, 1, + "Jazz16 Config"); + if (!card->private_data) { + error = -EBUSY; + goto out; + } + card->private_free = snd_jazz16_free; + + if (mpu_port == SNDRV_AUTO_PORT) + mpu_port = 0; + + if (mpu_irq == SNDRV_AUTO_IRQ) + mpu_irq = 0; + + error = jazz16_enable(dev); + if (error < 0) + goto out; + + error = snd_sbdsp_create(card, port, irq, snd_jazz16_interrupt, + dma8, dma16, SB_HW_AUTO, &chip); + if (error < 0) + goto out; + + if (chip->hardware != SB_HW_JAZZ16) { + snd_printk(KERN_ERR "%s: not a Jazz16 chip at %#lx.\n", + dev->bus_id, chip->port); + error = -ENODEV; + goto out; + } + + strcpy(card->driver, "Jazz16"); + strcpy(card->shortname, "Media Vision Jazz16"); + sprintf(card->longname, "%s at %#lx, irq %d, dma8 %d, dma16 %d", + chip->name, chip->port, chip->irq, chip->dma8, chip->dma16); + + error = snd_sb8dsp_pcm(chip, 0, NULL); + if (error < 0) + goto out; + + error = snd_sbmixer_new(chip); + if (error < 0) + goto out; + + error = snd_opl3_create(card, chip->port, chip->port + 2, OPL3_HW_AUTO, + 1, &opl3); + if (error < 0) + snd_printk(KERN_WARNING "%s: no OPL device found, skipping.\n", + dev->bus_id); + else { + error = snd_opl3_timer_new(opl3, 1, 2); + if (error < 0) + goto out; + + error = snd_opl3_hwdep_new(opl3, 0, 1, NULL); + if (error < 0) + goto out; + } + + snd_card_set_dev(card, dev); + + error = snd_card_register(card); + if (error < 0) + goto out; + + dev_set_drvdata(dev, card); + return 0; + +out: snd_card_free(card); + return error; +} + +static int __devexit snd_jazz16_remove(struct device *dev, unsigned int n) +{ + snd_card_free(dev_get_drvdata(dev)); + dev_set_drvdata(dev, NULL); + return 0; +} + +static struct isa_driver snd_jazz16_driver = { + .match = snd_jazz16_match, + .probe = snd_jazz16_probe, + .remove = __devexit_p(snd_jazz16_remove), + + .driver = { + .name = DEV_NAME + } +}; + +static int __devinit alsa_card_jazz16_init(void) +{ + return isa_register_driver(&snd_jazz16_driver, 1); +} + +static void __devexit alsa_card_jazz16_exit(void) +{ + isa_unregister_driver(&snd_jazz16_driver); +} + +module_init (alsa_card_jazz16_init); +module_exit (alsa_card_jazz16_exit); diff --git a/sound/isa/sb/sb8_main.c b/sound/isa/sb/sb8_main.c index aea9e5e..70cd031 100644 --- a/sound/isa/sb/sb8_main.c +++ b/sound/isa/sb/sb8_main.c @@ -28,6 +28,9 @@ * * Wed Jul 12 22:02:55 CEST 2000 Uros Bizjak <uros(a)kss-loka.si> * Cleaned up and rewrote lowlevel routines. + * + * Sun Mar 11 18:45:38 CEST 2007 Rask Ingemann Lambertsen <rask(a)sygehus.dk> + * Added Jazz16 enhancements. */ #include <sound/driver.h> @@ -73,6 +76,15 @@ static struct snd_ratnum stereo_clocks[] } }; +/* For stereo playback and capture, the denominator is divided by two, so it + * must be even to get the intended sample rate. */ +static struct snd_ratnum jazz16_stereo_clock = { + .num = SB8_CLOCK, + .den_min = 2, + .den_max = 512, + .den_step = 2, +}; + static int snd_sb8_hw_constraint_rate_channels(struct snd_pcm_hw_params *params, struct snd_pcm_hw_rule *rule) { @@ -101,14 +113,54 @@ static int snd_sb8_hw_constraint_channel return 0; } +static int snd_jazz16_hw_constraint_rate_channels(struct snd_pcm_hw_params *params, + struct snd_pcm_hw_rule *rule) +{ + struct snd_interval *c = hw_param_interval(params, SNDRV_PCM_HW_PARAM_CHANNELS); + struct snd_interval *r = hw_param_interval(params, SNDRV_PCM_HW_PARAM_RATE); + unsigned int num = 0, den = 0; + int err = 0; + + if (c->max > 1) + err = snd_interval_ratnum(r, 1, &jazz16_stereo_clock, &num, &den); + else + err = snd_interval_ratnum(r, 1, &clock, &num, &den); + if (err >= 0 && den) { + params->rate_num = num; + params->rate_den = den; + } + return err; +} + +static int snd_jazz16_hw_constraint_channels_rate(struct snd_pcm_hw_params *params, + struct snd_pcm_hw_rule *rule) +{ + struct snd_interval *r = hw_param_interval(params, SNDRV_PCM_HW_PARAM_RATE); + int err = 0; + + /* Force mono mode if the sample rate interval doesn't allow stereo. */ + if (SB8_DEN(r->min) == SB8_DEN(r->max) + && SB8_DEN(r->min) & 1) + { + struct snd_interval t = { .min = 1, .max = 1 }; + struct snd_interval *c; + + c = hw_param_interval(params, SNDRV_PCM_HW_PARAM_CHANNELS); + err = snd_interval_refine(c, &t); + } + return err; +} + static int snd_sb8_playback_prepare(struct snd_pcm_substream *substream) { unsigned long flags; struct snd_sb *chip = snd_pcm_substream_chip(substream); struct snd_pcm_runtime *runtime = substream->runtime; - unsigned int mixreg, rate, size, count; + unsigned int mixreg, rate, size, count, dma; + unsigned char stereo, format; rate = runtime->rate; + stereo = runtime->channels > 1; switch (chip->hardware) { case SB_HW_PRO: if (runtime->channels > 1) { @@ -117,6 +169,12 @@ static int snd_sb8_playback_prepare(stru break; } /* fallthru */ + case SB_HW_JAZZ16: + if (runtime->format == SNDRV_PCM_FORMAT_MU_LAW) { + chip->playback_format = SB_DSP_ULAW_OUTPUT_AUTO; + break; + } + /* fall through */ case SB_HW_201: if (rate > 23000) { chip->playback_format = SB_DSP_HI_OUTPUT_AUTO; @@ -134,8 +192,26 @@ static int snd_sb8_playback_prepare(stru } size = chip->p_dma_size = snd_pcm_lib_buffer_bytes(substream); count = chip->p_period_size = snd_pcm_lib_period_bytes(substream); + switch (runtime->format) + { + case SNDRV_PCM_FORMAT_U8: + case SNDRV_PCM_FORMAT_MU_LAW: + dma = chip->dma8; + format = stereo ? SB_DSP_STEREO_8BIT : SB_DSP_MONO_8BIT; + break; + + case SNDRV_PCM_FORMAT_S16_LE: + dma = chip->dma16; + format = stereo ? SB_DSP_STEREO_16BIT : SB_DSP_MONO_16BIT; + break; + + default: + return -EINVAL; + } spin_lock_irqsave(&chip->reg_lock, flags); snd_sbdsp_command(chip, SB_DSP_SPEAKER_ON); + if (chip->hardware >= SB_HW_PRO) + snd_sbdsp_command(chip, format); if (runtime->channels > 1) { /* set playback stereo mode */ spin_lock(&chip->mixer_lock); @@ -173,7 +249,7 @@ static int snd_sb8_playback_prepare(stru snd_sbdsp_command(chip, count >> 8); } spin_unlock_irqrestore(&chip->reg_lock, flags); - snd_dma_program(chip->dma8, runtime->dma_addr, + snd_dma_program(dma, runtime->dma_addr, size, DMA_MODE_WRITE | DMA_AUTOINIT); return 0; } @@ -232,9 +308,11 @@ static int snd_sb8_capture_prepare(struc unsigned long flags; struct snd_sb *chip = snd_pcm_substream_chip(substream); struct snd_pcm_runtime *runtime = substream->runtime; - unsigned int mixreg, rate, size, count; + unsigned int mixreg, rate, size, count, dma; + unsigned char stereo, format; rate = runtime->rate; + stereo = runtime->channels > 1; switch (chip->hardware) { case SB_HW_PRO: if (runtime->channels > 1) { @@ -242,6 +320,12 @@ static int snd_sb8_capture_prepare(struc chip->capture_format = SB_DSP_HI_INPUT_AUTO; break; } + /* fall through */ + case SB_HW_JAZZ16: + if (runtime->format == SNDRV_PCM_FORMAT_MU_LAW) { + chip->capture_format = SB_DSP_ULAW_OUTPUT_AUTO; + break; + } chip->capture_format = (rate > 23000) ? SB_DSP_HI_INPUT_AUTO : SB_DSP_LO_INPUT_AUTO; break; case SB_HW_201: @@ -261,10 +345,26 @@ static int snd_sb8_capture_prepare(struc } size = chip->c_dma_size = snd_pcm_lib_buffer_bytes(substream); count = chip->c_period_size = snd_pcm_lib_period_bytes(substream); + switch (runtime->format) + { + case SNDRV_PCM_FORMAT_U8: + case SNDRV_PCM_FORMAT_MU_LAW: + dma = chip->dma8; + format = stereo ? SB_DSP_STEREO_8BIT : SB_DSP_MONO_8BIT; + break; + + case SNDRV_PCM_FORMAT_S16_LE: + dma = chip->dma16; + format = stereo ? SB_DSP_STEREO_16BIT : SB_DSP_MONO_16BIT; + break; + + default: + return -EINVAL; + } spin_lock_irqsave(&chip->reg_lock, flags); snd_sbdsp_command(chip, SB_DSP_SPEAKER_OFF); - if (runtime->channels > 1) - snd_sbdsp_command(chip, SB_DSP_STEREO_8BIT); + if (chip->hardware >= SB_HW_PRO) + snd_sbdsp_command(chip, format); snd_sbdsp_command(chip, SB_DSP_SAMPLE_RATE); if (runtime->channels > 1) { snd_sbdsp_command(chip, 256 - runtime->rate_den / 2); @@ -278,14 +378,14 @@ static int snd_sb8_capture_prepare(struc } else { snd_sbdsp_command(chip, 256 - runtime->rate_den); } - if (chip->capture_format != SB_DSP_OUTPUT) { + if (chip->capture_format != SB_DSP_INPUT) { count--; snd_sbdsp_command(chip, SB_DSP_BLOCK_SIZE); snd_sbdsp_command(chip, count & 0xff); snd_sbdsp_command(chip, count >> 8); } spin_unlock_irqrestore(&chip->reg_lock, flags); - snd_dma_program(chip->dma8, runtime->dma_addr, + snd_dma_program(dma, runtime->dma_addr, size, DMA_MODE_READ | DMA_AUTOINIT); return 0; } @@ -358,10 +458,12 @@ static snd_pcm_uframes_t snd_sb8_playbac { struct snd_sb *chip = snd_pcm_substream_chip(substream); size_t ptr; + int dma; if (chip->mode != SB_MODE_PLAYBACK_8) return 0; - ptr = snd_dma_pointer(chip->dma8, chip->p_dma_size); + dma = (substream->runtime->format == SNDRV_PCM_FORMAT_S16_LE) ? chip->dma16 : chip->dma8; + ptr = snd_dma_pointer(dma, chip->p_dma_size); return bytes_to_frames(substream->runtime, ptr); } @@ -369,10 +471,12 @@ static snd_pcm_uframes_t snd_sb8_capture { struct snd_sb *chip = snd_pcm_substream_chip(substream); size_t ptr; + int dma; if (chip->mode != SB_MODE_CAPTURE_8) return 0; - ptr = snd_dma_pointer(chip->dma8, chip->c_dma_size); + dma = (substream->runtime->format == SNDRV_PCM_FORMAT_S16_LE) ? chip->dma16 : chip->dma8; + ptr = snd_dma_pointer(dma, chip->c_dma_size); return bytes_to_frames(substream->runtime, ptr); } @@ -443,6 +547,20 @@ static int snd_sb8_open(struct snd_pcm_s runtime->hw = snd_sb8_capture; } switch (chip->hardware) { + case SB_HW_JAZZ16: + runtime->hw.formats = SNDRV_PCM_FMTBIT_U8 | SNDRV_PCM_FMTBIT_S16_LE + | SNDRV_PCM_FMTBIT_MU_LAW; + runtime->hw.rates = SNDRV_PCM_RATE_CONTINUOUS | SNDRV_PCM_RATE_8000_44100; + runtime->hw.rate_max = 45455; + runtime->hw.channels_max = 2; + snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_RATE, + snd_jazz16_hw_constraint_rate_channels, NULL, + SNDRV_PCM_HW_PARAM_CHANNELS, + SNDRV_PCM_HW_PARAM_RATE, -1); + snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_CHANNELS, + snd_jazz16_hw_constraint_channels_rate, NULL, + SNDRV_PCM_HW_PARAM_RATE, -1); + break; case SB_HW_PRO: runtime->hw.rate_max = 44100; runtime->hw.channels_max = 2; @@ -463,9 +581,11 @@ static int snd_sb8_open(struct snd_pcm_s default: break; } + if (chip->hardware == SB_HW_JAZZ16) + return 0; snd_pcm_hw_constraint_ratnums(runtime, 0, SNDRV_PCM_HW_PARAM_RATE, &hw_constraints_clock); - return 0; + return 0; } static int snd_sb8_close(struct snd_pcm_substream *substream) diff --git a/sound/isa/sb/sb_common.c b/sound/isa/sb/sb_common.c index 3094f38..69e72b5 100644 --- a/sound/isa/sb/sb_common.c +++ b/sound/isa/sb/sb_common.c @@ -146,8 +146,16 @@ static int snd_sbdsp_probe(struct snd_sb } break; case 3: - chip->hardware = SB_HW_PRO; - str = "Pro"; + spin_lock_irqsave(&chip->reg_lock, flags); + if (snd_sbdsp_command(chip, SB_DSP_GET_JAZZ_VERSION) && + SB_VERSION_IS_JAZZ16(snd_sbdsp_get_byte (chip))) { + chip->hardware = SB_HW_JAZZ16; + str = "Pro (Jazz16)"; + } else { + chip->hardware = SB_HW_PRO; + str = "Pro"; + } + spin_unlock_irqrestore (&chip->reg_lock, flags); break; case 4: chip->hardware = SB_HW_16; diff --git a/sound/isa/sb/sb_mixer.c b/sound/isa/sb/sb_mixer.c index 490b1ca..b16b592 100644 --- a/sound/isa/sb/sb_mixer.c +++ b/sound/isa/sb/sb_mixer.c @@ -811,6 +811,7 @@ int snd_sbmixer_new(struct snd_sb *chip) return err; break; case SB_HW_PRO: + case SB_HW_JAZZ16: if ((err = snd_sbmixer_init(chip, snd_sbpro_controls, ARRAY_SIZE(snd_sbpro_controls), @@ -946,6 +947,7 @@ void snd_sbmixer_suspend(struct snd_sb * save_mixer(chip, sb20_saved_regs, ARRAY_SIZE(sb20_saved_regs)); break; case SB_HW_PRO: + case SB_HW_JAZZ16: save_mixer(chip, sbpro_saved_regs, ARRAY_SIZE(sbpro_saved_regs)); break; case SB_HW_16: @@ -971,6 +973,7 @@ void snd_sbmixer_resume(struct snd_sb *c restore_mixer(chip, sb20_saved_regs, ARRAY_SIZE(sb20_saved_regs)); break; case SB_HW_PRO: + case SB_HW_JAZZ16: restore_mixer(chip, sbpro_saved_regs, ARRAY_SIZE(sbpro_saved_regs)); break; case SB_HW_16:

1 0

[alsa-devel] [Fwd: Re: [Alsa-devel] [RFC][RFT] Adding support for Jazz16 based sound cards]
by Rene Herman 20 Mar '07

20 Mar '07

I didn't see this one make the old list at all, so forward to the new one... -------- Original Message -------- Subject: Re: [Alsa-devel] [RFC][RFT] Adding support for Jazz16 based sound cards Date: Mon, 19 Mar 2007 23:46:07 +0100 From: Rask Ingemann Lambertsen <rask(a)sygehus.dk> To: Rene Herman <rene.herman(a)gmail.com> CC: alsa-devel(a)lists.sourceforge.net, Adam Belay <ambx1(a)neo.rr.com> References: <20070311232436.GB9778(a)sygehus.dk> <45FDDA0F.7050503(a)gmail.com> On Mon, Mar 19, 2007 at 01:32:15AM +0100, Rene Herman wrote: > > I just now tested this on a standalone Jazz16 and it seems to be working > fine for me. I once had ogg123 segfault on me at the end of playback but > was unsuccesful in reproducing -- may have been a fluke. Thanks. I'll try out ogg123. > As to the comments... > > The changes to the SB code to support the Jazz16 I ofcourse agree with. > Supporting a new card is great. I am however not a huge fan of the new > PnP protocol. It's cute, but I feel it's a little over-engineered and > could just as well live directly in snd-jazz16. See below. > This setup also shares that same problem with the BIOS generally not > having reserved/routed the resources -- on a general PCI/ISA system the > user needs to go into the BIOS setup and reserve the resources the card > is (or will be, in this case) set to. So now the user really needs to > know what resources are going to be assigned, destroying the plug and > play idea... Not really. A standards compliant PnPBIOS set up with "PnP aware OS = YES" will use only enough resources to boot the system, which should leave enough irqs and dmas for the Linux PnP layer to do its work. I think there are three good reasons to use the PnP layer: 1) The PnP layer informs the PCI layer of irqs to try to avoid - please see pnp_register_irq_resource() in drivers/pnp/resource.c. 2) The PnP layer knows which resources are used by active devices, even if they have not been allocated using e.g. request_region(). This happens in practice. For example: $ cat /sys/bus/pnp/devices/01\:02.00/resources state = active io 0x2a0-0x2af io 0x390-0x393 io 0x500-0x50f irq 5 dma 0 dma 1 $ cat /proc/ioports [...] 0200-0200 : ns558-pnp 0201-0201 : ns558-pnp 0208-020f : ns558-pnp 0213-0213 : ISAPnP 0280-028f : ES18xx 0356-0356 : ide2 0360-0361 : MPU401 UART 0376-0376 : 0000:00:07.1 0376-0376 : ide1 0390-0391 : OPL2/3 (left) 0392-0393 : OPL2/3 (right) 03c0-03df : vga+ 03e8-03ef : serial 03f6-03f6 : 0000:00:07.1 03f6-03f6 : ide0 03f8-03ff : serial 0500-050f : AD1816A 0800-0807 : ES18xx - CTRL [...] Notice that 0x2a0-0x2af is missing from /proc/ioports. The PnP layer will avoid using that port range because it knows a device is using it. 3) Knowing which resources a card can utilize, the PnP layer can try to stay away from those resources when configuring other cards. The PnP layer doesn't do that yet in a stock kernel, but I've patched it to do so. I'll try to get those patches into the kernel. An advantage of autoprobing in a PnP protocol driver (as opposed to doing so in a sound card driver) is that it happens before any file systems have been mounted, so if the autoprobe happens to find one of those #%!$@ NE2000 cards and lock up the system, it won't corrupt your disks. -- Rask Ingemann Lambertsen

1 0

[alsa-devel] test1
by Jaroslav Kysela 20 Mar '07

20 Mar '07

test1 ----- Jaroslav Kysela <perex(a)perex.cz>

1 0