Re: [PATCH] reenable MIDI for usx2y

[Takashi Iwai <tiwai@suse.de>]

At Sun, 27 Mar 2005 14:00:12 +0200 (CEST),
karsten wiese wrote:
> 
> Hi Clemens,
> 
> usx2y needs to specify exactly what usb-endpoints to use.
> this feature got lost lately and is restored by attached
> patch. Please apply!

IMO, it's better to fix snd_usbmidi_detect_endpoints() to check
in_ep and out_ep rather than its caller.


Takashi

> 
> rgds,
> Karsten
> 
> Canonical Patch:
> ------------------------------------------------------
> Don't detect MIDI usb-endpoint, if quirk specifies it
> 
> If for QUIRK_MIDI_FIXED_ENDPOINT a struct
> snd_usb_midi_endpoint_info's
> members out_ep or in_ep are != 0,
> they shall be treated as exactly what usb-endpoints to use.
> This rule was lost due to a recent change in usbmidi.c
> and is reestablished by attached patch.
> 
> signed-of-by: Karsten Wiese <annabellesgarden@yahoo.de>
> 
> 
> 	
> 
> 	
> 		
> ___________________________________________________________ 
> Gesendet von Yahoo! Mail - Jetzt mit 250MB Speicher kostenlos - Hier anmelden: http://mail.yahoo.de
> [2 usbmidi.c.patch <text/x-diff (7bit)>]
> Index: usbmidi.c
> ===================================================================
> RCS file: /cvsroot/alsa/alsa-kernel/usb/usbmidi.c,v
> retrieving revision 1.45
> diff -U3 -r1.45 usbmidi.c
> --- usbmidi.c	24 Mar 2005 07:17:17 -0000	1.45
> +++ usbmidi.c	25 Mar 2005 13:46:50 -0000
> @@ -1492,7 +1492,8 @@
>  		case QUIRK_MIDI_FIXED_ENDPOINT:
>  			memcpy(&endpoints[0], quirk->data,
>  			       sizeof(snd_usb_midi_endpoint_info_t));
> -			err = snd_usbmidi_detect_endpoints(umidi, &endpoints[0], 1);
> +			err = (endpoints[0].in_ep || endpoints[0].out_ep) ? 0 :
> +				snd_usbmidi_detect_endpoints(umidi, &endpoints[0], 1);
>  			break;
>  		case QUIRK_MIDI_YAMAHA:
>  			err = snd_usbmidi_detect_yamaha(umidi, &endpoints[0]);


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click