Re: [PATCH] [ALSA] usb-audio - Support for Roland GAIA SH-01 Synthesizer

[Daniel Mack <zonque@gmail.com>]

On 11/27/2011 03:34 PM, John F Leach wrote:
> Added table quirks entry for Roland GAIA SH-01 Synthesizer based upon
> Roland SH-201 table entry as template.  USB MIDI and audio was tested
> with Muse and Audacity.
> 
> Signed-off-by: John F Leach <jfleach@jfleach.com>
> Cc: Daniel Mack <zonque@gmail.com>
> Cc: Clemens Ladisch <clemens@ladisch.de>
> Cc: Takashi Iwai <tiwai@suse.de>
> ---
>  sound/usb/quirks-table.h |   31 +++++++++++++++++++++++++++++++
>  1 files changed, 31 insertions(+), 0 deletions(-)
> 
> diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
> index b61945f..6380665 100644
> --- a/sound/usb/quirks-table.h
> +++ b/sound/usb/quirks-table.h
> @@ -1400,6 +1400,37 @@ YAMAHA_DEVICE(0x7010, "UB99"),
>  	}
>  },
>  {
> +	/* Roland GAIA SH-01 */
> +	USB_DEVICE(0x0582, 0x0111),

Entries in this list should be ordered by the USB vendor/product ID,
which is not the case judging from the next block:

>  	/* Roland SH-201 */
>  	USB_DEVICE(0x0582, 0x00ad),
>  	.driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {


Otherwise looks good.

Thanks for sending patches!


Daniel


> +	.driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {
> +		.vendor_name = "Roland",
> +		.product_name = "GAIA",
> +		.ifnum = QUIRK_ANY_INTERFACE,
> +		.type = QUIRK_COMPOSITE,
> +		.data = (const struct snd_usb_audio_quirk[]) {
> +			{
> +				.ifnum = 0,
> +				.type = QUIRK_AUDIO_STANDARD_INTERFACE
> +			},
> +			{
> +				.ifnum = 1,
> +				.type = QUIRK_AUDIO_STANDARD_INTERFACE
> +			},
> +			{
> +				.ifnum = 2,
> +                                .type = QUIRK_MIDI_FIXED_ENDPOINT,
> +                                .data = & (const struct
> snd_usb_midi_endpoint_info) {
> +                                       .out_cables = 0x0003,
> +                                       .in_cables  = 0x0003
> +				}
> +			},
> +			{
> +				.ifnum = -1
> +			}
> +		}
> +	}
> +},
> +{
>  	/* Roland SH-201 */
>  	USB_DEVICE(0x0582, 0x00ad),
>  	.driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {