Commit ac774236 authored by Keith A. Milner's avatar Keith A. Milner Committed by Takashi Iwai

ALSA: usb-audio: Allow any MIDI endpoint to drive use of interrupt transfer on newer Roland devices

This patch enables interrupt transfer mode for MIDI ports on newer
Boss/Roland devices such as the GT-100/001 which support interrupt
transfer on both IN and OUT MIDI endpoints. Previously this wasn't being
enabled for these devices as the code was specifically looking for the
scenario where the IN endpoint supported interrupt transfer and the OUT
endpoint was bulk transfer. Newer devices support interrupt transfer for
both endpoints.

This has been tested on Boss devices GT-001, BR-80 and JS-8 and Roland
VS-20.

It would benefit from some regresison testing with other devices if
possible.
Signed-off-by: default avatarKeith A. Milner <maillist@superlative.org>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 2a7e1713
...@@ -1903,11 +1903,14 @@ static void snd_usbmidi_switch_roland_altsetting(struct snd_usb_midi *umidi) ...@@ -1903,11 +1903,14 @@ static void snd_usbmidi_switch_roland_altsetting(struct snd_usb_midi *umidi)
hostif = &intf->altsetting[1]; hostif = &intf->altsetting[1];
intfd = get_iface_desc(hostif); intfd = get_iface_desc(hostif);
/* If either or both of the endpoints support interrupt transfer,
* then use the alternate setting
*/
if (intfd->bNumEndpoints != 2 || if (intfd->bNumEndpoints != 2 ||
(get_endpoint(hostif, 0)->bmAttributes & !((get_endpoint(hostif, 0)->bmAttributes &
USB_ENDPOINT_XFERTYPE_MASK) != USB_ENDPOINT_XFER_BULK || USB_ENDPOINT_XFERTYPE_MASK) == USB_ENDPOINT_XFER_INT ||
(get_endpoint(hostif, 1)->bmAttributes & (get_endpoint(hostif, 1)->bmAttributes &
USB_ENDPOINT_XFERTYPE_MASK) != USB_ENDPOINT_XFER_INT) USB_ENDPOINT_XFERTYPE_MASK) == USB_ENDPOINT_XFER_INT))
return; return;
dev_dbg(&umidi->dev->dev, "switching to altsetting %d with int ep\n", dev_dbg(&umidi->dev->dev, "switching to altsetting %d with int ep\n",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment