diff options
| author | Takashi Iwai <tiwai@suse.de> | 2016-02-29 14:26:43 +0100 |
|---|---|---|
| committer | Jiri Slaby <jslaby@suse.cz> | 2016-03-07 16:05:26 +0100 |
| commit | 6cc1e98de81b68a4e15088ac504ea6677288e9aa (patch) | |
| tree | 5633f01a811edc689af70b8c1f5c6606f775f9ce | |
| parent | 4e4ac2043c0519274753ef4c3b455946f4bbd2e3 (diff) | |
| download | lwn-6cc1e98de81b68a4e15088ac504ea6677288e9aa.tar.gz lwn-6cc1e98de81b68a4e15088ac504ea6677288e9aa.zip | |
ALSA: hdsp: Fix wrong boolean ctl value accesses
commit eab3c4db193f5fcccf70e884de9a922ca2c63d80 upstream.
snd-hdsp driver accesses enum item values (int) instead of boolean
values (long) wrongly for some ctl elements. This patch fixes them.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
| -rw-r--r-- | sound/pci/rme9652/hdsp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/pci/rme9652/hdsp.c b/sound/pci/rme9652/hdsp.c index f59a321a6d6a..a15e4e9b2774 100644 --- a/sound/pci/rme9652/hdsp.c +++ b/sound/pci/rme9652/hdsp.c @@ -2923,7 +2923,7 @@ static int snd_hdsp_get_dds_offset(struct snd_kcontrol *kcontrol, struct snd_ctl { struct hdsp *hdsp = snd_kcontrol_chip(kcontrol); - ucontrol->value.enumerated.item[0] = hdsp_dds_offset(hdsp); + ucontrol->value.integer.value[0] = hdsp_dds_offset(hdsp); return 0; } @@ -2935,7 +2935,7 @@ static int snd_hdsp_put_dds_offset(struct snd_kcontrol *kcontrol, struct snd_ctl if (!snd_hdsp_use_is_exclusive(hdsp)) return -EBUSY; - val = ucontrol->value.enumerated.item[0]; + val = ucontrol->value.integer.value[0]; spin_lock_irq(&hdsp->lock); if (val != hdsp_dds_offset(hdsp)) change = (hdsp_set_dds_offset(hdsp, val) == 0) ? 1 : 0; |