]> git.openfabrics.org - ~shefty/rdma-dev.git/commitdiff
ALSA: usb-audio: Fix comment
authorMark Hills <mark@pogo.org.uk>
Fri, 11 May 2012 17:31:55 +0000 (18:31 +0100)
committerTakashi Iwai <tiwai@suse.de>
Fri, 11 May 2012 19:27:36 +0000 (21:27 +0200)
Explained by Takashi in <s5hfwbtmz0q.wl%tiwai@suse.de>

> The reason is because get_min_max*() isn't called in the place you
> created these controls, and get_min_max() would be called only for
> integer volumes later even if uninitialized.  A short cut for booleans.

Signed-off-by: Mark Hills <mark@pogo.org.uk>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/mixer_quirks.c

index 12125ca7167fb820d7352e1de8d0f90d9a792520..41f4b69119205d04f9b50e1e6ccc3bd5d16d980f 100644 (file)
@@ -79,16 +79,8 @@ static int snd_create_std_mono_ctl(struct usb_mixer_interface *mixer,
        cval->control = control;
        cval->cmask = cmask;
 
-       /* FIXME: Do we need this?
-        * The following values are for compatibility with
-        * Ebox-44 mixer.
-        * But the corresponding ebox-44 function says:
-        *    "Volume controls will override these values"
-        *
-        * These values don't have any effect at all for
-        * M-Audio FTUs.
-        * So I think, we can safely omit the range settings here.
-        */
+       /* get_min_max() is called only for integer volumes later,
+        * so provide a short-cut for booleans */
        cval->min = 0;
        cval->max = 1;
        cval->res = 0;