ALSA: snd-usb-audio: set the timeout for usb control set messages to 5000 ms
authorDaniel Schürmann <daschuer@mixxx.org>
Sat, 20 Apr 2013 21:06:17 +0000 (23:06 +0200)
committerTakashi Iwai <tiwai@suse.de>
Mon, 22 Apr 2013 08:45:02 +0000 (10:45 +0200)
Set the timeout for USB control set messages according to the USB 2
spec, using the macros from include/linux/usb.h.
The get timout becomes 5000 ms even though it is 500 ms in the
spec. This patch is required to run the Hercules RMX2 which needs a
timeout of 1240 ms.

More notes from author:
I still distinguish between set and get but as long both are 5000 ms
GCC will remove it anyway. IMHO this is more easy read and there is no
need to explain why we use a get timeout for set messages.

Signed-off-by: Daniel Schürmann <daschuer@mixxx.org>
Acked-by: Daniel Mack <zonque@gmail.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/helper.c

index c1db28f874c285d12f2ee310a57ce34daa54e301..620902463c6ea2d5b9ba8eec496c33b5c8e78c2a 100644 (file)
@@ -86,14 +86,22 @@ int snd_usb_ctl_msg(struct usb_device *dev, unsigned int pipe, __u8 request,
 {
        int err;
        void *buf = NULL;
+       int timeout;
 
        if (size > 0) {
                buf = kmemdup(data, size, GFP_KERNEL);
                if (!buf)
                        return -ENOMEM;
        }
+
+       if (requesttype & USB_DIR_IN)
+               timeout = USB_CTRL_GET_TIMEOUT;
+       else
+               timeout = USB_CTRL_SET_TIMEOUT;
+
        err = usb_control_msg(dev, pipe, request, requesttype,
-                             value, index, buf, size, 1000);
+                             value, index, buf, size, timeout);
+
        if (size > 0) {
                memcpy(data, buf, size);
                kfree(buf);