Re: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

newer
[alsa-devel] [PATCH 1/2] ASoC:...

older
Re: [alsa-devel] [PATCH] sound:...

Damien Zammit

23 Dec 2012 23 Dec '12

5:54 a.m.

On 23 December 2012 14:36, Damien Zammit damien.zammit@gmail.com wrote:

...

I have fixed both the kernel panic and the broken pipe issue with midi. I believe this completes the patch, I just don't know how to submit it to you.

Please find attached the patch you accepted previously: (Version 5) Plus an incremental patch (Version 5-2) ***Please ignore patch (Version 6)***

I hope this is the easiest way for you to review it.

It compiles, runs, and works on the hardware now without errors.

Thanks, Damien

Attachments:

patch_for-next_dz5.txt (text/plain — 8.1 KB)
patch_for-next_dz5-2.txt (text/plain — 7.6 KB)

Show replies by date

Damien Zammit

23 Dec 23 Dec

6:12 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

On 23 December 2012 15:54, Damien Zammit damien.zammit@gmail.com wrote:

...

Plus an incremental patch (Version 5-2)

Sorry I forgot the commit message:

Support for Digidesign Mbox 2 USB sound card:

This patch is the result of a lot of trial and error, since there are no specs available for the device.

This is part two (2) of patch_for_next_dz5.txt and should be applied after it.

This patch fixes two things: 1) A kernel panic caused by incorrect allocation of a u8 variable "bootresponse". 2) A noisy dmesg (urb status -32) caused by broken pipe to an invalid midi endpoint.

It is also a little cleaner because there is no need for a new QUIRK_MIDI type as suggested by kernel developers.

Signed-off-by: Damien Zammit damien@zamaudio.com

---

See previous message for the actual patch (version 5-2)

Takashi Iwai

24 Dec 24 Dec

10:38 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

At Sun, 23 Dec 2012 15:54:48 +1100, Damien Zammit wrote:

...

[1 <text/plain; ISO-8859-1 (7bit)>] On 23 December 2012 14:36, Damien Zammit damien.zammit@gmail.com wrote:

...
I have fixed both the kernel panic and the broken pipe issue with midi. I believe this completes the patch, I just don't know how to submit it to you.

Please find attached the patch you accepted previously: (Version 5) Plus an incremental patch (Version 5-2) ***Please ignore patch (Version 6)***

I hope this is the easiest way for you to review it.

Well, the standard way is to send an incremental patch against the latest upstream. That is, just give a diff against 3.8-rc1 with a patch description about only the incremental patch -- i.e. which problems in previous commits are fixed by the new patch.

BTW, I'm going to be on vacation until Jan. 2, so the reply will be delayed.

Have a merry Christmas and new year.

thanks,

Takashi

...

It compiles, runs, and works on the hardware now without errors.

Thanks, Damien [2 patch_for-next_dz5.txt <text/plain; US-ASCII (base64)>] Support for Digidesign Mbox 2 USB sound card:

This patch is the result of a lot of trial and error, since there are no specs available for the device.

Full duplex support is provided, i.e. playback and recording in stereo. The format is hardcoded at 48000Hz @ 24 bit, which is the maximum that the device supports. Also, MIDI in and MIDI out both work.

Users will notice that the S/PDIF light also flashes when playback or recording is active. I believe this means that S/PDIF input/output is simultaneously activated with the analogue i/o during use. But this particular functionality remains untested.

Note that this particular version of the patch is so far untested on the physical hardware because I have not compiled a full kernel with the changes. However, extensive testing has been done by many users of the hardware who believe other versions of my patch have worked since circa 2009.

Signed-off-by: Damien Zammit damien@zamaudio.com

diff --git a/sound/usb/midi.c b/sound/usb/midi.c index 34b9bb7..c183d34 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -2181,6 +2181,10 @@ int snd_usbmidi_create(struct snd_card *card, umidi->usb_protocol_ops = &snd_usbmidi_novation_ops; err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints); break;
case QUIRK_MIDI_MBOX2:
umidi->usb_protocol_ops = &snd_usbmidi_midiman_ops;
err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints);
break;
case QUIRK_MIDI_RAW_BYTES: umidi->usb_protocol_ops = &snd_usbmidi_raw_ops; /*
diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index 579cf6f..cdcf6b4 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -2921,6 +2921,93 @@ YAMAHA_DEVICE(0x7010, "UB99"),

} },

+/* DIGIDESIGN MBOX 2 */ +{
USB_DEVICE(0x0dba, 0x3000),

.driver_info = (unsigned long) &(const struct snd_usb_audio_quirk) {
.vendor_name = "Digidesign",
.product_name = "Mbox 2",
.ifnum = QUIRK_ANY_INTERFACE,
.type = QUIRK_COMPOSITE,
.data = (const struct snd_usb_audio_quirk[]) {
	{
		.ifnum = 0,
		.type = QUIRK_IGNORE_INTERFACE
	},
	{
		.ifnum = 1,
		.type = QUIRK_IGNORE_INTERFACE
	},
	{
		.ifnum = 2,
		.type = QUIRK_AUDIO_FIXED_ENDPOINT,
		.data = &(const struct audioformat) {
			.formats = SNDRV_PCM_FMTBIT_S24_3BE,
			.channels = 2,
			.iface = 2,
			.altsetting = 2,
			.altset_idx = 1,
			.attributes = 0x00,
			.endpoint = 0x03,
			.ep_attr = USB_ENDPOINT_SYNC_ASYNC,
			.maxpacksize = 0x128,
			.rates = SNDRV_PCM_RATE_48000,
			.rate_min = 48000,
			.rate_max = 48000,
			.nr_rates = 1,
			.rate_table = (unsigned int[]) {
				48000
			}
		}
	},
	{
		.ifnum = 3,
		.type = QUIRK_IGNORE_INTERFACE
	},
	{
		.ifnum = 4,
		.type = QUIRK_AUDIO_FIXED_ENDPOINT,
		.data = &(const struct audioformat) {
		.formats = SNDRV_PCM_FMTBIT_S24_3BE,
			.channels = 2,
			.iface = 4,
			.altsetting = 2,
			.altset_idx = 1,
			.attributes = UAC_EP_CS_ATTR_SAMPLE_RATE,
			.endpoint = 0x85,
			.ep_attr = USB_ENDPOINT_SYNC_SYNC,
			.maxpacksize = 0x128,
			.rates = SNDRV_PCM_RATE_48000,
			.rate_min = 48000,
			.rate_max = 48000,
			.nr_rates = 1,
			.rate_table = (unsigned int[]) {
				48000
			}
		}
	},
	{
		.ifnum = 5,
		.type = QUIRK_IGNORE_INTERFACE
	},
	{
		.ifnum = 6,
		.type = QUIRK_MIDI_MBOX2,
		.data = &(const struct snd_usb_midi_endpoint_info) {
			.out_ep =  0x02,
			.out_cables = 0x0001,
			.in_ep = 0x81,
			.in_interval = 0x01,
			.in_cables = 0x0001
		}
	},
	{
		.ifnum = -1
	}
}
}
+}, { /* Tascam US122 MKII - playback-only support */ .match_flags = USB_DEVICE_ID_MATCH_DEVICE, diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index 0f58b4b..8f23af5 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -306,6 +306,7 @@ int snd_usb_create_quirk(struct snd_usb_audio *chip, [QUIRK_MIDI_YAMAHA] = create_any_midi_quirk, [QUIRK_MIDI_MIDIMAN] = create_any_midi_quirk, [QUIRK_MIDI_NOVATION] = create_any_midi_quirk,
[QUIRK_MIDI_MBOX2] = create_any_midi_quirk,
[QUIRK_MIDI_RAW_BYTES] = create_any_midi_quirk, [QUIRK_MIDI_EMAGIC] = create_any_midi_quirk, [QUIRK_MIDI_CME] = create_any_midi_quirk,
@@ -655,6 +656,10 @@ int snd_usb_apply_boot_quirk(struct usb_device *dev, case USB_ID(0x0ccd, 0x00b1): /* Terratec Aureon 7.1 USB */ return snd_usb_cm6206_boot_quirk(dev);
case USB_ID(0x0dba, 0x3000):
/* Digidesign Mbox 2 */
return snd_usb_mbox2_boot_quirk(dev);
case USB_ID(0x133e, 0x0815): /* Access Music VirusTI Desktop */ return snd_usb_accessmusic_boot_quirk(dev);
@@ -761,6 +766,92 @@ void snd_usb_set_format_quirk(struct snd_usb_substream *subs, } }

+static void mbox2_setup_48_24_magic(struct usb_device *dev) +{
u8 srate[3];

u8 temp[12];

/* Choose 48000Hz permanently */

srate[0] = 0x80;

srate[1] = 0xbb;

srate[2] = 0x00;

/* Send the magic! */

snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
0x01, 0x22, 0x0100, 0x0085, &temp, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0085, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0086, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0003, &srate, 0x0003);
return;
+}

+/* Digidesign Mbox 2 needs to load firmware onboard

and driver must wait a few seconds for initialisation.

*/

+#define MBOX2_FIRMWARE_SIZE 646 +#define MBOX2_BOOT_LOADING 0x01 /* Hard coded into the device */ +#define MBOX2_BOOT_READY 0x02 /* Hard coded into the device */

+int snd_usb_mbox2_boot_quirk(struct usb_device *dev) +{
struct usb_host_config *config = dev->actconfig;

int err;

u8 bootresponse;

int fwsize;

int count;

fwsize = le16_to_cpu(get_cfg_desc(config)->wTotalLength);

if (fwsize != MBOX2_FIRMWARE_SIZE) {
snd_printk(KERN_ERR "usb-audio: Invalid firmware size=%d.\n", fwsize);
return -ENODEV;
}

snd_printd("usb-audio: Sending Digidesign Mbox 2 boot sequence...\n");

count = 0;

bootresponse = MBOX2_BOOT_LOADING;

while ((bootresponse == MBOX2_BOOT_LOADING) && (count < 10)) {
msleep(500); /* 0.5 second delay */
snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
	/* Control magic - load onboard firmware */
	0x85, 0xc0, 0x0001, 0x0000, &bootresponse, 0x0012);
if (bootresponse == MBOX2_BOOT_READY)
	break;
snd_printd("usb-audio: device not ready, resending boot sequence...\n");
count++;
}

if (bootresponse != MBOX2_BOOT_READY) {
snd_printk(KERN_ERR "usb-audio: Unknown bootresponse=%d, or timed out, ignoring device.\n", bootresponse);
return -ENODEV;
}

snd_printdd("usb-audio: device initialised!\n");

err = usb_get_descriptor(dev, USB_DT_DEVICE, 0,
&dev->descriptor, sizeof(dev->descriptor));
config = dev->actconfig;

if (err < 0)
snd_printd("error usb_get_descriptor: %d\n", err);
err = usb_reset_configuration(dev);

if (err < 0)
snd_printd("error usb_reset_configuration: %d\n", err);
snd_printdd("mbox2_boot: new boot length = %d\n",
le16_to_cpu(get_cfg_desc(config)->wTotalLength));
mbox2_setup_48_24_magic(dev);

snd_printk(KERN_INFO "usb-audio: Digidesign Mbox 2: 24bit 48kHz");

return 0; /* Successful boot */
+}

void snd_usb_endpoint_start_quirk(struct snd_usb_endpoint *ep) { /* diff --git a/sound/usb/quirks.h b/sound/usb/quirks.h index 0ca9e91..e7e9530 100644 --- a/sound/usb/quirks.h +++ b/sound/usb/quirks.h @@ -24,6 +24,8 @@ void snd_usb_set_format_quirk(struct snd_usb_substream *subs, int snd_usb_is_big_endian_format(struct snd_usb_audio *chip, struct audioformat *fp);

+int snd_usb_mbox2_boot_quirk(struct usb_device *dev);

void snd_usb_endpoint_start_quirk(struct snd_usb_endpoint *ep);

void snd_usb_ctl_msg_quirk(struct usb_device *dev, unsigned int pipe, diff --git a/sound/usb/usbaudio.h b/sound/usb/usbaudio.h index 1ac3fd9..a8172c1 100644 --- a/sound/usb/usbaudio.h +++ b/sound/usb/usbaudio.h @@ -76,6 +76,7 @@ enum quirk_type { QUIRK_MIDI_YAMAHA, QUIRK_MIDI_MIDIMAN, QUIRK_MIDI_NOVATION,

QUIRK_MIDI_MBOX2, QUIRK_MIDI_RAW_BYTES, QUIRK_MIDI_EMAGIC, QUIRK_MIDI_CME,

[3 patch_for-next_dz5-2.txt <text/plain; US-ASCII (base64)>] diff --git a/sound/usb/midi.c b/sound/usb/midi.c index c183d34..34b9bb7 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -2181,10 +2181,6 @@ int snd_usbmidi_create(struct snd_card *card, umidi->usb_protocol_ops = &snd_usbmidi_novation_ops; err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints); break;
case QUIRK_MIDI_MBOX2:
umidi->usb_protocol_ops = &snd_usbmidi_midiman_ops;
err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints);
break;
case QUIRK_MIDI_RAW_BYTES: umidi->usb_protocol_ops = &snd_usbmidi_raw_ops; /*
diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index cdcf6b4..95e3326 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -2993,7 +2993,7 @@ YAMAHA_DEVICE(0x7010, "UB99"), }, { .ifnum = 6,
		.type = QUIRK_MIDI_MBOX2,
		.type = QUIRK_MIDI_MIDIMAN,
	.data = &(const struct snd_usb_midi_endpoint_info) {
		.out_ep =  0x02,
		.out_cables = 0x0001,
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index 8f23af5..dc96696 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -306,7 +306,6 @@ int snd_usb_create_quirk(struct snd_usb_audio *chip, [QUIRK_MIDI_YAMAHA] = create_any_midi_quirk, [QUIRK_MIDI_MIDIMAN] = create_any_midi_quirk, [QUIRK_MIDI_NOVATION] = create_any_midi_quirk,
[QUIRK_MIDI_MBOX2] = create_any_midi_quirk,
[QUIRK_MIDI_RAW_BYTES] = create_any_midi_quirk, [QUIRK_MIDI_EMAGIC] = create_any_midi_quirk, [QUIRK_MIDI_CME] = create_any_midi_quirk,
@@ -498,6 +497,92 @@ static int snd_usb_nativeinstruments_boot_quirk(struct usb_device *dev) return -EAGAIN; }

+static void mbox2_setup_48_24_magic(struct usb_device *dev) +{
u8 srate[3];

u8 temp[12];

/* Choose 48000Hz permanently */

srate[0] = 0x80;

srate[1] = 0xbb;

srate[2] = 0x00;

/* Send the magic! */

snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
0x01, 0x22, 0x0100, 0x0085, &temp, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0085, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0086, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0003, &srate, 0x0003);
return;
+}

+/* Digidesign Mbox 2 needs to load firmware onboard

and driver must wait a few seconds for initialisation.

*/

+#define MBOX2_FIRMWARE_SIZE 646 +#define MBOX2_BOOT_LOADING 0x01 /* Hard coded into the device */ +#define MBOX2_BOOT_READY 0x02 /* Hard coded into the device */

+static int snd_usb_mbox2_boot_quirk(struct usb_device *dev) +{
struct usb_host_config *config = dev->actconfig;

int err;

u8 bootresponse[12];

int fwsize;

int count;

fwsize = le16_to_cpu(get_cfg_desc(config)->wTotalLength);

if (fwsize != MBOX2_FIRMWARE_SIZE) {
snd_printk(KERN_ERR "usb-audio: Invalid firmware size=%d.\n", fwsize);
return -ENODEV;
}

snd_printd("usb-audio: Sending Digidesign Mbox 2 boot sequence...\n");

count = 0;

bootresponse[0] = MBOX2_BOOT_LOADING;

while ((bootresponse[0] == MBOX2_BOOT_LOADING) && (count < 10)) {
msleep(500); /* 0.5 second delay */
snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
	/* Control magic - load onboard firmware */
	0x85, 0xc0, 0x0001, 0x0000, &bootresponse, 0x0012);
if (bootresponse[0] == MBOX2_BOOT_READY)
	break;
snd_printd("usb-audio: device not ready, resending boot sequence...\n");
count++;
}

if (bootresponse[0] != MBOX2_BOOT_READY) {
snd_printk(KERN_ERR "usb-audio: Unknown bootresponse=%d, or timed out, ignoring device.\n", bootresponse[0]);
return -ENODEV;
}

snd_printdd("usb-audio: device initialised!\n");

err = usb_get_descriptor(dev, USB_DT_DEVICE, 0,
&dev->descriptor, sizeof(dev->descriptor));
config = dev->actconfig;

if (err < 0)
snd_printd("error usb_get_descriptor: %d\n", err);
err = usb_reset_configuration(dev);

if (err < 0)
snd_printd("error usb_reset_configuration: %d\n", err);
snd_printdd("mbox2_boot: new boot length = %d\n",
le16_to_cpu(get_cfg_desc(config)->wTotalLength));
mbox2_setup_48_24_magic(dev);

snd_printk(KERN_INFO "usb-audio: Digidesign Mbox 2: 24bit 48kHz");

return 0; /* Successful boot */
+}

/*

Setup quirks

*/ @@ -574,7 +659,6 @@ static int audiophile_skip_setting_quirk(struct snd_usb_audio *chip, return 0; /* keep this altsetting */ }

static int fasttrackpro_skip_setting_quirk(struct snd_usb_audio *chip, int iface, int altno) { @@ -766,92 +850,6 @@ void snd_usb_set_format_quirk(struct snd_usb_substream *subs, } }

-static void mbox2_setup_48_24_magic(struct usb_device *dev) -{
u8 srate[3];

u8 temp[12];

/* Choose 48000Hz permanently */

srate[0] = 0x80;

srate[1] = 0xbb;

srate[2] = 0x00;

/* Send the magic! */

snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
0x01, 0x22, 0x0100, 0x0085, &temp, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0085, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0086, &srate, 0x0003);
snd_usb_ctl_msg(dev, usb_sndctrlpipe(dev, 0),
0x81, 0xa2, 0x0100, 0x0003, &srate, 0x0003);
return;
-}

-/* Digidesign Mbox 2 needs to load firmware onboard

and driver must wait a few seconds for initialisation.

*/

-#define MBOX2_FIRMWARE_SIZE 646 -#define MBOX2_BOOT_LOADING 0x01 /* Hard coded into the device */ -#define MBOX2_BOOT_READY 0x02 /* Hard coded into the device */

-int snd_usb_mbox2_boot_quirk(struct usb_device *dev) -{
struct usb_host_config *config = dev->actconfig;

int err;

u8 bootresponse;

int fwsize;

int count;

fwsize = le16_to_cpu(get_cfg_desc(config)->wTotalLength);

if (fwsize != MBOX2_FIRMWARE_SIZE) {
snd_printk(KERN_ERR "usb-audio: Invalid firmware size=%d.\n", fwsize);
return -ENODEV;
}

snd_printd("usb-audio: Sending Digidesign Mbox 2 boot sequence...\n");

count = 0;

bootresponse = MBOX2_BOOT_LOADING;

while ((bootresponse == MBOX2_BOOT_LOADING) && (count < 10)) {
msleep(500); /* 0.5 second delay */
snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0),
	/* Control magic - load onboard firmware */
	0x85, 0xc0, 0x0001, 0x0000, &bootresponse, 0x0012);
if (bootresponse == MBOX2_BOOT_READY)
	break;
snd_printd("usb-audio: device not ready, resending boot sequence...\n");
count++;
}

if (bootresponse != MBOX2_BOOT_READY) {
snd_printk(KERN_ERR "usb-audio: Unknown bootresponse=%d, or timed out, ignoring device.\n", bootresponse);
return -ENODEV;
}

snd_printdd("usb-audio: device initialised!\n");

err = usb_get_descriptor(dev, USB_DT_DEVICE, 0,
&dev->descriptor, sizeof(dev->descriptor));
config = dev->actconfig;

if (err < 0)
snd_printd("error usb_get_descriptor: %d\n", err);
err = usb_reset_configuration(dev);

if (err < 0)
snd_printd("error usb_reset_configuration: %d\n", err);
snd_printdd("mbox2_boot: new boot length = %d\n",
le16_to_cpu(get_cfg_desc(config)->wTotalLength));
mbox2_setup_48_24_magic(dev);

snd_printk(KERN_INFO "usb-audio: Digidesign Mbox 2: 24bit 48kHz");

return 0; /* Successful boot */
-}

void snd_usb_endpoint_start_quirk(struct snd_usb_endpoint *ep) { /* diff --git a/sound/usb/quirks.h b/sound/usb/quirks.h index e7e9530..0ca9e91 100644 --- a/sound/usb/quirks.h +++ b/sound/usb/quirks.h @@ -24,8 +24,6 @@ void snd_usb_set_format_quirk(struct snd_usb_substream *subs, int snd_usb_is_big_endian_format(struct snd_usb_audio *chip, struct audioformat *fp);

-int snd_usb_mbox2_boot_quirk(struct usb_device *dev);

void snd_usb_endpoint_start_quirk(struct snd_usb_endpoint *ep);

void snd_usb_ctl_msg_quirk(struct usb_device *dev, unsigned int pipe,

Damien Zammit

26 Dec 26 Dec

12:21 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

On 24 December 2012 20:38, Takashi Iwai tiwai@suse.de wrote:

...

Well, the standard way is to send an incremental patch against the latest upstream. That is, just give a diff against 3.8-rc1 with a patch description about only the incremental patch -- i.e. which problems in previous commits are fixed by the new patch.

Okay, in that case if you could ignore version 5-2 and I will send through a diff based on 3.8-rc1 with a proper comment instead. I will name it "patch_3.8-rc1_dz1.txt" in my next email.

...

BTW, I'm going to be on vacation until Jan. 2, so the reply will be delayed. Have a merry Christmas and new year.

You too, and have a great holiday!

Regards, Damien

Takashi Iwai

3 Jan 3 Jan

11:11 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

At Wed, 26 Dec 2012 10:21:23 +1100, Damien Zammit wrote:

...

On 24 December 2012 20:38, Takashi Iwai tiwai@suse.de wrote:

...
Well, the standard way is to send an incremental patch against the latest upstream. That is, just give a diff against 3.8-rc1 with a patch description about only the incremental patch -- i.e. which problems in previous commits are fixed by the new patch.

Okay, in that case if you could ignore version 5-2 and I will send through a diff based on 3.8-rc1 with a proper comment instead. I will name it "patch_3.8-rc1_dz1.txt" in my next email.

Yes, a patch against 3.8-rc1 will be helpful. Did you already post it?

thanks,

Takashi

Damien Zammit

4 Jan 4 Jan

2:29 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

On 3 January 2013 21:11, Takashi Iwai tiwai@suse.de wrote:

...

Yes, a patch against 3.8-rc1 will be helpful.

Here it is. I generated the patch, checked out v3.8-rc1 tag, patched it with this patch and compiled it out of tree for my running 3.8-rc1 kernel. Tested playback with jackd. Works well.

Regards, Damien

Takashi Iwai

10:57 a.m.

New subject: [alsa-devel] sound: usb-audio: [PATCH]x2 full capture/playback/spdif support for Digidesign Mbox 2

At Fri, 4 Jan 2013 12:29:18 +1100, Damien Zammit wrote:

...

On 3 January 2013 21:11, Takashi Iwai tiwai@suse.de wrote:

...
Yes, a patch against 3.8-rc1 will be helpful.

Here it is. I generated the patch, checked out v3.8-rc1 tag, patched it with this patch and compiled it out of tree for my running 3.8-rc1 kernel. Tested playback with jackd. Works well.

Thanks, applied now with a bit modification of subject and patch description.

Takashi

...

Regards, Damien [2 patch_3.8-rc1_dz1.txt <text/plain; US-ASCII (base64)>] Support for Digidesign Mbox 2 USB sound card:

This patch is the result of a lot of trial and error, since there are no specs available for the device.

This patch is based on 3.8-rc1. It fixes two things:

A kernel panic caused by incorrect allocation of a u8 variable "bootresponse".

A noisy dmesg (urb status -32) caused by broken pipe to an invalid midi endpoint.

It is also a little cleaner because there is no need for a new QUIRK_MIDI type as suggested by kernel developers, since the device follows exactly the MIDIMAN protocol.

Signed-off-by: Damien Zammit damien@zamaudio.com

diff --git a/sound/usb/midi.c b/sound/usb/midi.c index c183d34..34b9bb7 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -2181,10 +2181,6 @@ int snd_usbmidi_create(struct snd_card *card, umidi->usb_protocol_ops = &snd_usbmidi_novation_ops; err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints); break;
case QUIRK_MIDI_MBOX2:
umidi->usb_protocol_ops = &snd_usbmidi_midiman_ops;
err = snd_usbmidi_detect_per_port_endpoints(umidi, endpoints);
break;
case QUIRK_MIDI_RAW_BYTES: umidi->usb_protocol_ops = &snd_usbmidi_raw_ops; /*
diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h index cdcf6b4..95e3326 100644 --- a/sound/usb/quirks-table.h +++ b/sound/usb/quirks-table.h @@ -2993,7 +2993,7 @@ YAMAHA_DEVICE(0x7010, "UB99"), }, { .ifnum = 6,
		.type = QUIRK_MIDI_MBOX2,
		.type = QUIRK_MIDI_MIDIMAN,
	.data = &(const struct snd_usb_midi_endpoint_info) {
		.out_ep =  0x02,
		.out_cables = 0x0001,
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c index f104c68..acc12f0 100644 --- a/sound/usb/quirks.c +++ b/sound/usb/quirks.c @@ -306,7 +306,6 @@ int snd_usb_create_quirk(struct snd_usb_audio *chip, [QUIRK_MIDI_YAMAHA] = create_any_midi_quirk, [QUIRK_MIDI_MIDIMAN] = create_any_midi_quirk, [QUIRK_MIDI_NOVATION] = create_any_midi_quirk,
[QUIRK_MIDI_MBOX2] = create_any_midi_quirk,
[QUIRK_MIDI_RAW_BYTES] = create_any_midi_quirk, [QUIRK_MIDI_EMAGIC] = create_any_midi_quirk, [QUIRK_MIDI_CME] = create_any_midi_quirk,
@@ -528,11 +527,11 @@ static void mbox2_setup_48_24_magic(struct usb_device *dev) #define MBOX2_BOOT_LOADING 0x01 /* Hard coded into the device */ #define MBOX2_BOOT_READY 0x02 /* Hard coded into the device */

-int snd_usb_mbox2_boot_quirk(struct usb_device *dev) +static int snd_usb_mbox2_boot_quirk(struct usb_device *dev) { struct usb_host_config *config = dev->actconfig; int err;

u8 bootresponse;

u8 bootresponse[12]; int fwsize; int count;

@@ -546,20 +545,20 @@ int snd_usb_mbox2_boot_quirk(struct usb_device *dev) snd_printd("usb-audio: Sending Digidesign Mbox 2 boot sequence...\n");

count = 0;

bootresponse = MBOX2_BOOT_LOADING;

while ((bootresponse == MBOX2_BOOT_LOADING) && (count < 10)) {

bootresponse[0] = MBOX2_BOOT_LOADING;

while ((bootresponse[0] == MBOX2_BOOT_LOADING) && (count < 10)) { msleep(500); /* 0.5 second delay */ snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), /* Control magic - load onboard firmware */ 0x85, 0xc0, 0x0001, 0x0000, &bootresponse, 0x0012);
if (bootresponse == MBOX2_BOOT_READY)
if (bootresponse[0] == MBOX2_BOOT_READY)
break;
snd_printd("usb-audio: device not ready, resending boot sequence...\n"); count++; }
if (bootresponse != MBOX2_BOOT_READY) {
snd_printk(KERN_ERR "usb-audio: Unknown bootresponse=%d, or timed out, ignoring device.\n", bootresponse);
if (bootresponse[0] != MBOX2_BOOT_READY) {
snd_printk(KERN_ERR "usb-audio: Unknown bootresponse=%d, or timed out, ignoring device.\n", bootresponse[0]);
return -ENODEV; }
@@ -660,7 +659,6 @@ static int audiophile_skip_setting_quirk(struct snd_usb_audio *chip, return 0; /* keep this altsetting */ }

static int fasttrackpro_skip_setting_quirk(struct snd_usb_audio *chip, int iface, int altno) { diff --git a/sound/usb/usbaudio.h b/sound/usb/usbaudio.h index a8172c1..1ac3fd9 100644 --- a/sound/usb/usbaudio.h +++ b/sound/usb/usbaudio.h @@ -76,7 +76,6 @@ enum quirk_type { QUIRK_MIDI_YAMAHA, QUIRK_MIDI_MIDIMAN, QUIRK_MIDI_NOVATION,

QUIRK_MIDI_MBOX2, QUIRK_MIDI_RAW_BYTES, QUIRK_MIDI_EMAGIC, QUIRK_MIDI_CME,

4370

Age (days ago)

4382

Last active (days ago)

List overview

Download

6 comments

2 participants

tags (0)

participants (2)

Damien Zammit
Takashi Iwai