Re: [PATCH] ALSA: usb-audio: MOTU MicroBook II playback fix

Re: [PATCH] ALSA: usb-audio: MOTU MicroBook II playback fix

[Sally Marcher]

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

Hi,
patch is attached to this email.

Thanks,
Sally Marcher

On 10/26/26 2:57 PM, Alexander Tsoy wrote:
> В Пн, 26/10/2026 в 14:40 +0100, Sally Marcher пишет:
>>   From cd7f80a8beaa40bcb2b313586a3c698d35982c21 Mon Sep 17 00:00:00
>> 2001
>> From: Sally Marcher <me@gry.sh>
>> Date: Mon, 26 Oct 2026 14:29:34 +0100
>> Subject: [PATCH] Add correct datainterval in the quirks-table for
>> Microbook II
>>
>> This patch fixes playback issues on the Microbook II.
>> The correct value was in the older driver for this device
>> (
>> https://github.com/manuel-rhdt/snd-microbookii/blob/dd304c097bb6a9d522
>> 76ac29c6969fa02bc7674a/audio.c#L914).
>>
>> Datainterval needs to be set explicitly.
>>
>> Signed-off-by: Sally Marcher <me@gry.sh>
>> ---
>>    sound/usb/quirks-table.h | 2 ++
>>    1 file changed, 2 insertions(+)
>>
>> diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
>> index 71444c2898b4..ac7861c3d26c 100644
>> --- a/sound/usb/quirks-table.h
>> +++ b/sound/usb/quirks-table.h
>> @@ -2781,6 +2781,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
>>                        .rates = SNDRV_PCM_RATE_96000,
>>                        .ep_attr = USB_ENDPOINT_XFER_ISOC |
>>                               USB_ENDPOINT_SYNC_ASYNC,
>> +                    .datainterval = 1,
>>                        .rate_min = 96000,
>>                        .rate_max = 96000,
>>                        .nr_rates = 1,
>> @@ -2803,6 +2804,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
>>                        .rates = SNDRV_PCM_RATE_96000,
>>                        .ep_attr = USB_ENDPOINT_XFER_ISOC |
>>                               USB_ENDPOINT_SYNC_ASYNC,
>> +                    .datainterval = 1,
>>                        .rate_min = 96000,
>>                        .rate_max = 96000,
>>                        .nr_rates = 1,
> Hi!
>
> You should send this path to linux-sound@vger.kernel.org and CC
> mainterners: Takashi Iwai <tiwai@suse.com>, Jaroslav Kysela
> <perex@perex.cz>
>
> Also your email client garbles the patch. For example tabs are replaced
> by spaces. If you cannot fix that, then it's better to just attach the
> patch as a text file.

[-- Attachment #2: 0001-Add-correct-datainterval-in-the-quirks-table-for-Mic.patch --]
[-- Type: text/x-patch, Size: 1302 bytes --]

From cd7f80a8beaa40bcb2b313586a3c698d35982c21 Mon Sep 17 00:00:00 2001
From: Sally Marcher <me@gry.sh>
Date: Mon, 26 Oct 2026 14:29:34 +0100
Subject: [PATCH] Add correct datainterval in the quirks-table for Microbook II

This patch fixes playback issues on the Microbook II.
The correct value was in the older driver for this device (https://github.com/manuel-rhdt/snd-microbookii/blob/dd304c097bb6a9d52276ac29c6969fa02bc7674a/audio.c#L914).

Datainterval needs to be set explicitly.

Signed-off-by: Sally Marcher <me@gry.sh>
---
 sound/usb/quirks-table.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
index 71444c2898b4..ac7861c3d26c 100644
--- a/sound/usb/quirks-table.h
+++ b/sound/usb/quirks-table.h
@@ -2781,6 +2781,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
 					.rates = SNDRV_PCM_RATE_96000,
 					.ep_attr = USB_ENDPOINT_XFER_ISOC |
 						   USB_ENDPOINT_SYNC_ASYNC,
+					.datainterval = 1,
 					.rate_min = 96000,
 					.rate_max = 96000,
 					.nr_rates = 1,
@@ -2803,6 +2804,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
 					.rates = SNDRV_PCM_RATE_96000,
 					.ep_attr = USB_ENDPOINT_XFER_ISOC |
 						   USB_ENDPOINT_SYNC_ASYNC,
+					.datainterval = 1,
 					.rate_min = 96000,
 					.rate_max = 96000,
 					.nr_rates = 1,
-- 
2.56.0

Re: [PATCH] ALSA: usb-audio: MOTU MicroBook II playback fix

[Takashi Iwai]

On Mon, 26 Oct 2026 17:01:32 +0100,
Sally Marcher wrote:
> 
> Hi,
> patch is attached to this email.

OK, I applied manually now.


thanks,

Takashi

> 
> Thanks,
> Sally Marcher
> 
> On 10/26/26 2:57 PM, Alexander Tsoy wrote:
> > В Пн, 26/10/2026 в 14:40 +0100, Sally Marcher пишет:
> >>   From cd7f80a8beaa40bcb2b313586a3c698d35982c21 Mon Sep 17 00:00:00
> >> 2001
> >> From: Sally Marcher <me@gry.sh>
> >> Date: Mon, 26 Oct 2026 14:29:34 +0100
> >> Subject: [PATCH] Add correct datainterval in the quirks-table for
> >> Microbook II
> >> 
> >> This patch fixes playback issues on the Microbook II.
> >> The correct value was in the older driver for this device
> >> (
> >> https://github.com/manuel-rhdt/snd-microbookii/blob/dd304c097bb6a9d522
> >> 76ac29c6969fa02bc7674a/audio.c#L914).
> >> 
> >> Datainterval needs to be set explicitly.
> >> 
> >> Signed-off-by: Sally Marcher <me@gry.sh>
> >> ---
> >>    sound/usb/quirks-table.h | 2 ++
> >>    1 file changed, 2 insertions(+)
> >> 
> >> diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
> >> index 71444c2898b4..ac7861c3d26c 100644
> >> --- a/sound/usb/quirks-table.h
> >> +++ b/sound/usb/quirks-table.h
> >> @@ -2781,6 +2781,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
> >>                        .rates = SNDRV_PCM_RATE_96000,
> >>                        .ep_attr = USB_ENDPOINT_XFER_ISOC |
> >>                               USB_ENDPOINT_SYNC_ASYNC,
> >> +                    .datainterval = 1,
> >>                        .rate_min = 96000,
> >>                        .rate_max = 96000,
> >>                        .nr_rates = 1,
> >> @@ -2803,6 +2804,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
> >>                        .rates = SNDRV_PCM_RATE_96000,
> >>                        .ep_attr = USB_ENDPOINT_XFER_ISOC |
> >>                               USB_ENDPOINT_SYNC_ASYNC,
> >> +                    .datainterval = 1,
> >>                        .rate_min = 96000,
> >>                        .rate_max = 96000,
> >>                        .nr_rates = 1,
> > Hi!
> > 
> > You should send this path to linux-sound@vger.kernel.org and CC
> > mainterners: Takashi Iwai <tiwai@suse.com>, Jaroslav Kysela
> > <perex@perex.cz>
> > 
> > Also your email client garbles the patch. For example tabs are replaced
> > by spaces. If you cannot fix that, then it's better to just attach the
> > patch as a text file.
> From cd7f80a8beaa40bcb2b313586a3c698d35982c21 Mon Sep 17 00:00:00 2001
> From: Sally Marcher <me@gry.sh>
> Date: Mon, 26 Oct 2026 14:29:34 +0100
> Subject: [PATCH] Add correct datainterval in the quirks-table for Microbook II
> 
> This patch fixes playback issues on the Microbook II.
> The correct value was in the older driver for this device (https://github.com/manuel-rhdt/snd-microbookii/blob/dd304c097bb6a9d52276ac29c6969fa02bc7674a/audio.c#L914).
> 
> Datainterval needs to be set explicitly.
> 
> Signed-off-by: Sally Marcher <me@gry.sh>
> ---
>  sound/usb/quirks-table.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/sound/usb/quirks-table.h b/sound/usb/quirks-table.h
> index 71444c2898b4..ac7861c3d26c 100644
> --- a/sound/usb/quirks-table.h
> +++ b/sound/usb/quirks-table.h
> @@ -2781,6 +2781,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
>  					.rates = SNDRV_PCM_RATE_96000,
>  					.ep_attr = USB_ENDPOINT_XFER_ISOC |
>  						   USB_ENDPOINT_SYNC_ASYNC,
> +					.datainterval = 1,
>  					.rate_min = 96000,
>  					.rate_max = 96000,
>  					.nr_rates = 1,
> @@ -2803,6 +2804,7 @@ YAMAHA_DEVICE(0x7010, "UB99"),
>  					.rates = SNDRV_PCM_RATE_96000,
>  					.ep_attr = USB_ENDPOINT_XFER_ISOC |
>  						   USB_ENDPOINT_SYNC_ASYNC,
> +					.datainterval = 1,
>  					.rate_min = 96000,
>  					.rate_max = 96000,
>  					.nr_rates = 1,
> -- 
> 2.56.0
> 

Re: [PATCH] ALSA: usb-audio: MOTU MicroBook II playback fix

[Alexander Tsoy]

В Вт, 27/10/2026 в 08:33 +0100, Takashi Iwai пишет:
> On Mon, 26 Oct 2026 17:01:32 +0100,
> Sally Marcher wrote:
> > 
> > Hi,
> > patch is attached to this email.
> 
> OK, I applied manually now.

Thanks!

I just want to add some context here for the history. I replied to
original thread on alsa-devel ML, but it seems MTA on alsa-project.org
is currently down.

This device is not class compliant and all endpoints are contained in a
single interface. snd_usb_parse_datainterval() probably returns 
bInterval of the first endpoint which is an Interrupt endpoint with 
bInterval=4. That's why datainterval needs to be set explicitly.