Linux Sound subsystem development
 help / color / mirror / Atom feed
* [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti
@ 2026-01-16  6:33 胡连勤
  2026-01-16  6:36 ` Cryolitia PukNgae
  2026-01-19  8:12 ` Takashi Iwai
  0 siblings, 2 replies; 5+ messages in thread
From: 胡连勤 @ 2026-01-16  6:33 UTC (permalink / raw)
  To: perex@perex.cz, tiwai@suse.com, Cryolitia PukNgae, pav@iki.fi,
	jussi@sonarnerd.net, roy.vegard.ovesen@gmail.com
  Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org,
	胡连勤

Audio control requests that sets sampling frequency sometimes fail on
this card. Adding delay between control messages eliminates that problem.

usb 1-1: New USB device found, idVendor=2fc6, idProduct=f06b
usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
usb 1-1: Product: MOONDROP Moonriver2 Ti
usb 1-1: Manufacturer: MOONDROP
usb 1-1: SerialNumber: MOONDROP Moonriver2 Ti

Signed-off-by: Lianqin Hu <hulianqin@vivo.com>
---

Changes in v2:
 - Reordered by PID
 - Link to v1: https://lore.kernel.org/all/TYUPR06MB6217CD0C82B11AD3E659D174D28DA@TYUPR06MB6217.apcprd06.prod.outlook.com/

 sound/usb/quirks.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index f38330b095e9..2d9f28558874 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -2390,6 +2390,8 @@ static const struct usb_audio_quirk_flags_table quirk_flags_table[] = {
 		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
 	DEVICE_FLG(0x2d99, 0x0026, /* HECATE G2 GAMING HEADSET */
 		   QUIRK_FLAG_MIXER_PLAYBACK_MIN_MUTE),
+	DEVICE_FLG(0x2fc6, 0xf06b, /* MOONDROP Moonriver2 Ti */
+		   QUIRK_FLAG_CTL_MSG_DELAY),
 	DEVICE_FLG(0x2fc6, 0xf0b7, /* iBasso DC07 Pro */
 		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
 	DEVICE_FLG(0x30be, 0x0101, /* Schiit Hel */
-- 
2.39.0


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

* Re: [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti
  2026-01-16  6:33 [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti 胡连勤
@ 2026-01-16  6:36 ` Cryolitia PukNgae
  2026-01-16 14:16   ` Takashi Iwai
  2026-01-19  8:12 ` Takashi Iwai
  1 sibling, 1 reply; 5+ messages in thread
From: Cryolitia PukNgae @ 2026-01-16  6:36 UTC (permalink / raw)
  To: 胡连勤, perex@perex.cz, tiwai@suse.com,
	pav@iki.fi, jussi@sonarnerd.net, roy.vegard.ovesen@gmail.com
  Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org



On 1/16/26 14:33, 胡连勤 wrote:
> Audio control requests that sets sampling frequency sometimes fail on
> this card. Adding delay between control messages eliminates that problem.
> 
> usb 1-1: New USB device found, idVendor=2fc6, idProduct=f06b
> usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> usb 1-1: Product: MOONDROP Moonriver2 Ti
> usb 1-1: Manufacturer: MOONDROP
> usb 1-1: SerialNumber: MOONDROP Moonriver2 Ti
> 
> Signed-off-by: Lianqin Hu <hulianqin@vivo.com>
> ---
> 
> Changes in v2:
>  - Reordered by PID
>  - Link to v1: https://lore.kernel.org/all/TYUPR06MB6217CD0C82B11AD3E659D174D28DA@TYUPR06MB6217.apcprd06.prod.outlook.com/
> 
>  sound/usb/quirks.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> index f38330b095e9..2d9f28558874 100644
> --- a/sound/usb/quirks.c
> +++ b/sound/usb/quirks.c
> @@ -2390,6 +2390,8 @@ static const struct usb_audio_quirk_flags_table quirk_flags_table[] = {
>  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
>  	DEVICE_FLG(0x2d99, 0x0026, /* HECATE G2 GAMING HEADSET */
>  		   QUIRK_FLAG_MIXER_PLAYBACK_MIN_MUTE),
> +	DEVICE_FLG(0x2fc6, 0xf06b, /* MOONDROP Moonriver2 Ti */
> +		   QUIRK_FLAG_CTL_MSG_DELAY),
>  	DEVICE_FLG(0x2fc6, 0xf0b7, /* iBasso DC07 Pro */
>  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
>  	DEVICE_FLG(0x30be, 0x0101, /* Schiit Hel */

LGTM

Approved-by: Cryolitia PukNgae <cryolitia@uniontech.com>

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

* Re: [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti
  2026-01-16  6:36 ` Cryolitia PukNgae
@ 2026-01-16 14:16   ` Takashi Iwai
  2026-01-19  7:52     ` Cryolitia PukNgae
  0 siblings, 1 reply; 5+ messages in thread
From: Takashi Iwai @ 2026-01-16 14:16 UTC (permalink / raw)
  To: Cryolitia PukNgae
  Cc: 胡连勤, perex@perex.cz, tiwai@suse.com,
	pav@iki.fi, jussi@sonarnerd.net, roy.vegard.ovesen@gmail.com,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org

On Fri, 16 Jan 2026 07:36:42 +0100,
Cryolitia PukNgae wrote:
> 
> 
> 
> On 1/16/26 14:33, 胡连勤 wrote:
> > Audio control requests that sets sampling frequency sometimes fail on
> > this card. Adding delay between control messages eliminates that problem.
> > 
> > usb 1-1: New USB device found, idVendor=2fc6, idProduct=f06b
> > usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> > usb 1-1: Product: MOONDROP Moonriver2 Ti
> > usb 1-1: Manufacturer: MOONDROP
> > usb 1-1: SerialNumber: MOONDROP Moonriver2 Ti
> > 
> > Signed-off-by: Lianqin Hu <hulianqin@vivo.com>
> > ---
> > 
> > Changes in v2:
> >  - Reordered by PID
> >  - Link to v1: https://lore.kernel.org/all/TYUPR06MB6217CD0C82B11AD3E659D174D28DA@TYUPR06MB6217.apcprd06.prod.outlook.com/
> > 
> >  sound/usb/quirks.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> > index f38330b095e9..2d9f28558874 100644
> > --- a/sound/usb/quirks.c
> > +++ b/sound/usb/quirks.c
> > @@ -2390,6 +2390,8 @@ static const struct usb_audio_quirk_flags_table quirk_flags_table[] = {
> >  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
> >  	DEVICE_FLG(0x2d99, 0x0026, /* HECATE G2 GAMING HEADSET */
> >  		   QUIRK_FLAG_MIXER_PLAYBACK_MIN_MUTE),
> > +	DEVICE_FLG(0x2fc6, 0xf06b, /* MOONDROP Moonriver2 Ti */
> > +		   QUIRK_FLAG_CTL_MSG_DELAY),
> >  	DEVICE_FLG(0x2fc6, 0xf0b7, /* iBasso DC07 Pro */
> >  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
> >  	DEVICE_FLG(0x30be, 0x0101, /* Schiit Hel */
> 
> LGTM
> 
> Approved-by: Cryolitia PukNgae <cryolitia@uniontech.com>

I guess you meant Reviewed-by?


thanks,

Takashi

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

* Re: [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti
  2026-01-16 14:16   ` Takashi Iwai
@ 2026-01-19  7:52     ` Cryolitia PukNgae
  0 siblings, 0 replies; 5+ messages in thread
From: Cryolitia PukNgae @ 2026-01-19  7:52 UTC (permalink / raw)
  To: Takashi Iwai
  Cc: 胡连勤, perex@perex.cz, tiwai@suse.com,
	pav@iki.fi, jussi@sonarnerd.net, roy.vegard.ovesen@gmail.com,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org



On 1/16/26 22:16, Takashi Iwai wrote:
> On Fri, 16 Jan 2026 07:36:42 +0100,
> Cryolitia PukNgae wrote:
>>
>>
>>
>> On 1/16/26 14:33, 胡连勤 wrote:
>>> Audio control requests that sets sampling frequency sometimes fail on
>>> this card. Adding delay between control messages eliminates that problem.
>>>
>>> usb 1-1: New USB device found, idVendor=2fc6, idProduct=f06b
>>> usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
>>> usb 1-1: Product: MOONDROP Moonriver2 Ti
>>> usb 1-1: Manufacturer: MOONDROP
>>> usb 1-1: SerialNumber: MOONDROP Moonriver2 Ti
>>>
>>> Signed-off-by: Lianqin Hu <hulianqin@vivo.com>
>>> ---
>>>
>>> Changes in v2:
>>>  - Reordered by PID
>>>  - Link to v1: https://lore.kernel.org/all/TYUPR06MB6217CD0C82B11AD3E659D174D28DA@TYUPR06MB6217.apcprd06.prod.outlook.com/
>>>
>>>  sound/usb/quirks.c | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
>>> index f38330b095e9..2d9f28558874 100644
>>> --- a/sound/usb/quirks.c
>>> +++ b/sound/usb/quirks.c
>>> @@ -2390,6 +2390,8 @@ static const struct usb_audio_quirk_flags_table quirk_flags_table[] = {
>>>  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
>>>  	DEVICE_FLG(0x2d99, 0x0026, /* HECATE G2 GAMING HEADSET */
>>>  		   QUIRK_FLAG_MIXER_PLAYBACK_MIN_MUTE),
>>> +	DEVICE_FLG(0x2fc6, 0xf06b, /* MOONDROP Moonriver2 Ti */
>>> +		   QUIRK_FLAG_CTL_MSG_DELAY),
>>>  	DEVICE_FLG(0x2fc6, 0xf0b7, /* iBasso DC07 Pro */
>>>  		   QUIRK_FLAG_CTL_MSG_DELAY_1M),
>>>  	DEVICE_FLG(0x30be, 0x0101, /* Schiit Hel */
>>
>> LGTM
>>
>> Approved-by: Cryolitia PukNgae <cryolitia@uniontech.com>
> 
> I guess you meant Reviewed-by?
> 

sorry for the mess

Reviewed-by: Cryolitia PukNgae <cryolitia@uniontech.com>

> 
> thanks,
> 
> Takashi
> 


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

* Re: [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti
  2026-01-16  6:33 [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti 胡连勤
  2026-01-16  6:36 ` Cryolitia PukNgae
@ 2026-01-19  8:12 ` Takashi Iwai
  1 sibling, 0 replies; 5+ messages in thread
From: Takashi Iwai @ 2026-01-19  8:12 UTC (permalink / raw)
  To: 胡连勤
  Cc: perex@perex.cz, tiwai@suse.com, Cryolitia PukNgae, pav@iki.fi,
	jussi@sonarnerd.net, roy.vegard.ovesen@gmail.com,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org

On Fri, 16 Jan 2026 07:33:03 +0100,
胡连勤 wrote:
> 
> Audio control requests that sets sampling frequency sometimes fail on
> this card. Adding delay between control messages eliminates that problem.
> 
> usb 1-1: New USB device found, idVendor=2fc6, idProduct=f06b
> usb 1-1: New USB device strings: Mfr=1, Product=2, SerialNumber=3
> usb 1-1: Product: MOONDROP Moonriver2 Ti
> usb 1-1: Manufacturer: MOONDROP
> usb 1-1: SerialNumber: MOONDROP Moonriver2 Ti
> 
> Signed-off-by: Lianqin Hu <hulianqin@vivo.com>

Applied now.  Thanks.


Takashi

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

end of thread, other threads:[~2026-01-19  8:12 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-16  6:33 [PATCH v2] ALSA: usb-audio: Add delay quirk for MOONDROP Moonriver2 Ti 胡连勤
2026-01-16  6:36 ` Cryolitia PukNgae
2026-01-16 14:16   ` Takashi Iwai
2026-01-19  7:52     ` Cryolitia PukNgae
2026-01-19  8:12 ` Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox