[ALSA] fix behaviour of ac97_enum_mixer elements
authorSergey Vlasov <vsu@altlinux.ru>
Mon, 25 Apr 2005 09:22:20 +0000 (11:22 +0200)
committerJaroslav Kysela <perex@suse.cz>
Sun, 29 May 2005 08:05:17 +0000 (10:05 +0200)
AC97 Codec
ac97_enum mixer elements (e.g., 'Capture Source') did not work because
of wrong bitmask calculation in snd_ac97_get_enum_double() and
snd_ac97_put_enum_double().

https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1072

Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>
sound/pci/ac97/ac97_codec.c

index 1ad7f83a65e1ea6844a33b6c12f430cc74e89dd0..36a33ae9ae030b9b846a8961fb1394754d299d5c 100644 (file)
@@ -465,7 +465,7 @@ int snd_ac97_get_enum_double(snd_kcontrol_t * kcontrol, snd_ctl_elem_value_t * u
        struct ac97_enum *e = (struct ac97_enum *)kcontrol->private_value;
        unsigned short val, bitmask;
        
-       for (bitmask = 1; bitmask > e->mask; bitmask <<= 1)
+       for (bitmask = 1; bitmask < e->mask; bitmask <<= 1)
                ;
        val = snd_ac97_read_cache(ac97, e->reg);
        ucontrol->value.enumerated.item[0] = (val >> e->shift_l) & (bitmask - 1);
@@ -482,7 +482,7 @@ int snd_ac97_put_enum_double(snd_kcontrol_t * kcontrol, snd_ctl_elem_value_t * u
        unsigned short val;
        unsigned short mask, bitmask;
        
-       for (bitmask = 1; bitmask > e->mask; bitmask <<= 1)
+       for (bitmask = 1; bitmask < e->mask; bitmask <<= 1)
                ;
        if (ucontrol->value.enumerated.item[0] > e->mask - 1)
                return -EINVAL;