Techwizz/kernel-fxtec-pro1x
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

ALSA: snd-usb: mixer: ignore -EINVAL in snd_usb_mixer_controls()

Browse source 
Creation of individual mixer controls may fail, but that shouldn't cause
the entire mixer creation to fail. Even worse, if the mixer creation
fails, that will error out the entire device probing.

All the functions called by parse_audio_unit() should return -EINVAL if
they find descriptors that are unsupported or believed to be malformed,
so we can safely handle this error code as a non-fatal condition in
snd_usb_mixer_controls().

That fixes a long standing bug which is commonly worked around by
adding quirks which make the driver ignore entire interfaces. Some of
them might now be unnecessary.

Signed-off-by: Daniel Mack <zonque@gmail.com>
Reported-and-tested-by: Rodolfo Thomazelli <pe.soberbo@gmail.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Daniel Mack 2013-03-19 21:09:25 +01:00 committed by Takashi Iwai
parent 4d7b86c98e
commit 83ea5d18d7
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
sound/usb/mixer.c
View file

@ -2123,7 +2123,7 @@ static int snd_usb_mixer_controls(struct usb_mixer_interface *mixer)
state.oterm.type = le16_to_cpu(desc->wTerminalType); state.oterm.type = le16_to_cpu(desc->wTerminalType);
state.oterm.name = desc->iTerminal; state.oterm.name = desc->iTerminal;
err = parse_audio_unit(&state, desc->bSourceID); err = parse_audio_unit(&state, desc->bSourceID);
if (err < 0) if (err < 0 && err != -EINVAL)
return err; return err;
} else { /* UAC_VERSION_2 */ } else { /* UAC_VERSION_2 */
struct uac2_output_terminal_descriptor *desc = p; struct uac2_output_terminal_descriptor *desc = p;
@ -2135,12 +2135,12 @@ static int snd_usb_mixer_controls(struct usb_mixer_interface *mixer)
state.oterm.type = le16_to_cpu(desc->wTerminalType); state.oterm.type = le16_to_cpu(desc->wTerminalType);
state.oterm.name = desc->iTerminal; state.oterm.name = desc->iTerminal;
err = parse_audio_unit(&state, desc->bSourceID); err = parse_audio_unit(&state, desc->bSourceID);
if (err < 0) if (err < 0 && err != -EINVAL)
return err; return err;
/* for UAC2, use the same approach to also add the clock selectors */ /* for UAC2, use the same approach to also add the clock selectors */
err = parse_audio_unit(&state, desc->bCSourceID); err = parse_audio_unit(&state, desc->bCSourceID);
if (err < 0) if (err < 0 && err != -EINVAL)
return err; return err;
} }
} }