diff options
author | Pawel MOLL <pawel.moll@st.com> | 2008-07-29 17:34:26 +0100 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2008-07-29 21:32:48 +0200 |
commit | 9423969005586e6e27ca380e01b4a8c50698e2af (patch) | |
tree | 818bef3892109c05a4ad5ec3ae5e90dd4436c0c6 /sound/core/sound.c | |
parent | 815ecf8dec95d07e260a16ebe8409f4b7c0fdc0f (diff) | |
download | blackbird-op-linux-9423969005586e6e27ca380e01b4a8c50698e2af.tar.gz blackbird-op-linux-9423969005586e6e27ca380e01b4a8c50698e2af.zip |
ALSA: Fix limit of 8 PCM devices in SNDRV_CTL_IOCTL_PCM_NEXT_DEVICE
When compiled with CONFIG_SND_DYNAMIC_MINORS the ALSA core is fine
to have more than 8 PCM devices per card, except one place - the
SNDRV_CTL_IOCTL_PCM_NEXT_DEVICE ioctl, which will not enumerate
devices > 7. This patch fixes the issue, changing the devices list
organisation.
Instead of adding new device to the tail, the list is now kept always
ordered (by card number, then device number). Thus, during enumeration,
it is easy to discover the fact that there is no more given card's
devices. The same limit was present in OSS emulation code. It has
been fixed as well.
Additionally the device field of struct snd_pcm is now int, instead of
unsigned int, as there is no obvious reason for keeping it unsigned.
This caused a lot of problems with comparing this value with other
(almost always signed) variables. There is just one more place where
device number is unsigned - in struct snd_pcm_info, which should be
also sorted out in future.
Signed-off-by: Pawel MOLL <pawel.moll@st.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/core/sound.c')
-rw-r--r-- | sound/core/sound.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/core/sound.c b/sound/core/sound.c index 1003ae375d47..838dd9ee957c 100644 --- a/sound/core/sound.c +++ b/sound/core/sound.c @@ -34,8 +34,6 @@ #include <linux/kmod.h> #include <linux/mutex.h> -#define SNDRV_OS_MINORS 256 - static int major = CONFIG_SND_MAJOR; int snd_major; EXPORT_SYMBOL(snd_major); |