All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Support for Yamaha MOXF
@ 2014-01-09  5:09 Michael Slade
  2014-01-09  6:38 ` Takashi Iwai
  2014-01-09  7:52 ` Clemens Ladisch
  0 siblings, 2 replies; 4+ messages in thread
From: Michael Slade @ 2014-01-09  5:09 UTC (permalink / raw)
  To: alsa-devel

[-- Attachment #1: Type: text/plain, Size: 777 bytes --]

My apologies if I'm doing this wrong.

I have managed to get a Yamaha MOXF8 working with the attached patch.  
Presumably also works with a MOXF6.

The new Yamaha MOXF6/8 are successors to the MOX6/8.  Apart from things 
like new voices, a flash expansion port, and a different colour at the 
ends, they are pretty much identical in operation.

The MOXF has a USB ID of 0499:150f.  So I needed to add a new 
quirks-table.h stanza for it, by copy-pasting the MOX one and changing 
he ID.

I haven't tested it to death or anything,  but I am having no trouble 
recording and playing audio and midi.

Let me know if this patch is acceptable or of there is more that is 
needed from me.

Note, this patch is against Ubuntu Raring's 3.8.0 kernel, but it's a 
trivial patch.

Mick.


[-- Attachment #2: alsa-yamaha-moxf.patch --]
[-- Type: text/x-patch, Size: 758 bytes --]

--- linux-3.8.0.orig/sound/usb/quirks-table.h
+++ linux-3.8.0/sound/usb/quirks-table.h
@@ -353,6 +359,32 @@
 		}
 	}
 },
+{
+	USB_DEVICE(0x0499, 0x150f),
+	.driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {
+		/* .vendor_name = "Yamaha", */
+		/* .product_name = "MOXF6/MOXF8", */
+		.ifnum = QUIRK_ANY_INTERFACE,
+		.type = QUIRK_COMPOSITE,
+		.data = (const struct snd_usb_audio_quirk[]) {
+			{
+				.ifnum = 1,
+				.type = QUIRK_AUDIO_STANDARD_INTERFACE
+			},
+			{
+				.ifnum = 2,
+				.type = QUIRK_AUDIO_STANDARD_INTERFACE
+			},
+			{
+				.ifnum = 3,
+				.type = QUIRK_MIDI_YAMAHA
+			},
+			{
+				.ifnum = -1
+			}
+		}
+	}
+},
 YAMAHA_DEVICE(0x2000, "DGP-7"),
 YAMAHA_DEVICE(0x2001, "DGP-5"),
 YAMAHA_DEVICE(0x2002, NULL),

[-- Attachment #3: Type: text/plain, Size: 0 bytes --]



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Support for Yamaha MOXF
  2014-01-09  5:09 [PATCH] Support for Yamaha MOXF Michael Slade
@ 2014-01-09  6:38 ` Takashi Iwai
  2014-01-09  7:52 ` Clemens Ladisch
  1 sibling, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2014-01-09  6:38 UTC (permalink / raw)
  To: Michael Slade; +Cc: alsa-devel

At Thu, 09 Jan 2014 16:09:10 +1100,
Michael Slade wrote:
> 
> My apologies if I'm doing this wrong.
> 
> I have managed to get a Yamaha MOXF8 working with the attached patch.  
> Presumably also works with a MOXF6.
> 
> The new Yamaha MOXF6/8 are successors to the MOX6/8.  Apart from things 
> like new voices, a flash expansion port, and a different colour at the 
> ends, they are pretty much identical in operation.
> 
> The MOXF has a USB ID of 0499:150f.  So I needed to add a new 
> quirks-table.h stanza for it, by copy-pasting the MOX one and changing 
> he ID.
> 
> I haven't tested it to death or anything,  but I am having no trouble 
> recording and playing audio and midi.
> 
> Let me know if this patch is acceptable or of there is more that is 
> needed from me.
> 
> Note, this patch is against Ubuntu Raring's 3.8.0 kernel, but it's a 
> trivial patch.

The code change looks OK, but please reformat the patch in the way to
follow Documentation/SubmittigPatches and resubmit to ML.


thanks,

Takashi

> 
> Mick.
> 
> [2 alsa-yamaha-moxf.patch <text/x-patch (7bit)>]
> --- linux-3.8.0.orig/sound/usb/quirks-table.h
> +++ linux-3.8.0/sound/usb/quirks-table.h
> @@ -353,6 +359,32 @@
>  		}
>  	}
>  },
> +{
> +	USB_DEVICE(0x0499, 0x150f),
> +	.driver_info = (unsigned long) & (const struct snd_usb_audio_quirk) {
> +		/* .vendor_name = "Yamaha", */
> +		/* .product_name = "MOXF6/MOXF8", */
> +		.ifnum = QUIRK_ANY_INTERFACE,
> +		.type = QUIRK_COMPOSITE,
> +		.data = (const struct snd_usb_audio_quirk[]) {
> +			{
> +				.ifnum = 1,
> +				.type = QUIRK_AUDIO_STANDARD_INTERFACE
> +			},
> +			{
> +				.ifnum = 2,
> +				.type = QUIRK_AUDIO_STANDARD_INTERFACE
> +			},
> +			{
> +				.ifnum = 3,
> +				.type = QUIRK_MIDI_YAMAHA
> +			},
> +			{
> +				.ifnum = -1
> +			}
> +		}
> +	}
> +},
>  YAMAHA_DEVICE(0x2000, "DGP-7"),
>  YAMAHA_DEVICE(0x2001, "DGP-5"),
>  YAMAHA_DEVICE(0x2002, NULL),
> [3  <text/plain; us-ascii (7bit)>]
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Support for Yamaha MOXF
  2014-01-09  5:09 [PATCH] Support for Yamaha MOXF Michael Slade
  2014-01-09  6:38 ` Takashi Iwai
@ 2014-01-09  7:52 ` Clemens Ladisch
  2014-01-09  8:14   ` Takashi Iwai
  1 sibling, 1 reply; 4+ messages in thread
From: Clemens Ladisch @ 2014-01-09  7:52 UTC (permalink / raw)
  To: Michael Slade, alsa-devel

Michael Slade wrote:
> I have managed to get a Yamaha MOXF8 working with the attached patch.

In theory, it should already work with kernel 3.11.


Regards,
Clemens

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Support for Yamaha MOXF
  2014-01-09  7:52 ` Clemens Ladisch
@ 2014-01-09  8:14   ` Takashi Iwai
  0 siblings, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2014-01-09  8:14 UTC (permalink / raw)
  To: Clemens Ladisch; +Cc: alsa-devel, Michael Slade

At Thu, 09 Jan 2014 08:52:40 +0100,
Clemens Ladisch wrote:
> 
> Michael Slade wrote:
> > I have managed to get a Yamaha MOXF8 working with the attached patch.
> 
> In theory, it should already work with kernel 3.11.

Ah, good point.  I forgot your recent rework.

Michael, please check whether 3.11 and later kernel works without your
patch.  If it doesn't, please send the fix patch again.


Takashi

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-01-09  8:14 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-09  5:09 [PATCH] Support for Yamaha MOXF Michael Slade
2014-01-09  6:38 ` Takashi Iwai
2014-01-09  7:52 ` Clemens Ladisch
2014-01-09  8:14   ` Takashi Iwai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.