* [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610
@ 2017-10-05 18:04 Kai-Heng Feng
2017-10-05 18:22 ` Takashi Iwai
0 siblings, 1 reply; 4+ messages in thread
From: Kai-Heng Feng @ 2017-10-05 18:04 UTC (permalink / raw)
To: tiwai, perex; +Cc: alsa-devel, linux-kernel, Kai-Heng Feng
Like other Plantronics devices, P610 does not support sample
rate reading. Apply sample rate quirk to it.
BugLink: https://bugs.launchpad.net/bugs/1719853
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
---
sound/usb/quirks.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index b8cb57aeec77..9ddaae3784f5 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1138,6 +1138,7 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
case USB_ID(0x047F, 0x0415): /* Plantronics BT-300 */
case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
case USB_ID(0x047F, 0xC022): /* Plantronics C310 */
+ case USB_ID(0x047F, 0xC02F): /* Plantronics P610 */
case USB_ID(0x047F, 0xC036): /* Plantronics C520-M */
case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
case USB_ID(0x0556, 0x0014): /* Phoenix Audio TMX320VC */
--
2.14.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610
2017-10-05 18:04 [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610 Kai-Heng Feng
@ 2017-10-05 18:22 ` Takashi Iwai
2017-10-06 4:27 ` Kai-Heng Feng
0 siblings, 1 reply; 4+ messages in thread
From: Takashi Iwai @ 2017-10-05 18:22 UTC (permalink / raw)
To: Kai-Heng Feng; +Cc: perex, alsa-devel, linux-kernel
On Thu, 05 Oct 2017 20:04:06 +0200,
Kai-Heng Feng wrote:
>
> Like other Plantronics devices, P610 does not support sample
> rate reading. Apply sample rate quirk to it.
>
> BugLink: https://bugs.launchpad.net/bugs/1719853
>
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Hrm, maybe we should ignore all Plantronics devices?
Also MS Lifecam and Phoenix devices are such candidates.
So something like below.
Takashi
-- 8< --
From: Takashi Iwai <tiwai@suse.de>
Subject: [PATCH] ALSA: usb-audio: Apply vendor ID matching for sample rate
quirk
So far, lots of Plantronics, MS and Phoenix Audio devices need the
quirk not to read sample rate back, and the list just grows.
In this patch, instead of adding each device, apply the quirk by
matching with these vendors.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
sound/usb/quirks.c | 23 +++++++++--------------
1 file changed, 9 insertions(+), 14 deletions(-)
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index b8cb57aeec77..9aeb05f61f78 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1128,29 +1128,24 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
/* devices which do not support reading the sample rate. */
switch (chip->usb_id) {
case USB_ID(0x041E, 0x4080): /* Creative Live Cam VF0610 */
- case USB_ID(0x045E, 0x075D): /* MS Lifecam Cinema */
- case USB_ID(0x045E, 0x076D): /* MS Lifecam HD-5000 */
- case USB_ID(0x045E, 0x076E): /* MS Lifecam HD-5001 */
- case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
- case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
- case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
- case USB_ID(0x047F, 0x02F7): /* Plantronics BT-600 */
- case USB_ID(0x047F, 0x0415): /* Plantronics BT-300 */
- case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
- case USB_ID(0x047F, 0xC022): /* Plantronics C310 */
- case USB_ID(0x047F, 0xC036): /* Plantronics C520-M */
case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
case USB_ID(0x0556, 0x0014): /* Phoenix Audio TMX320VC */
case USB_ID(0x05A3, 0x9420): /* ELP HD USB Camera */
case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
case USB_ID(0x1395, 0x740a): /* Sennheiser DECT */
case USB_ID(0x1901, 0x0191): /* GE B850V3 CP2114 audio interface */
- case USB_ID(0x1de7, 0x0013): /* Phoenix Audio MT202exe */
- case USB_ID(0x1de7, 0x0014): /* Phoenix Audio TMX320 */
- case USB_ID(0x1de7, 0x0114): /* Phoenix Audio MT202pcs */
case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
return true;
}
+
+ /* devices of these vendors don't support reading rate, either */
+ switch (USB_ID_VENDOR(chip->usb_id)) {
+ case 0x045E: /* MS Lifecam */
+ case 0x047F: /* Plantronics */
+ case 0x1de7: /* Phoenix Audio */
+ return true;
+ }
+
return false;
}
--
2.14.2
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610
2017-10-05 18:22 ` Takashi Iwai
@ 2017-10-06 4:27 ` Kai-Heng Feng
2017-10-06 8:42 ` Takashi Iwai
0 siblings, 1 reply; 4+ messages in thread
From: Kai-Heng Feng @ 2017-10-06 4:27 UTC (permalink / raw)
To: Takashi Iwai; +Cc: perex, alsa-devel, LKML
Hi,
On Fri, Oct 6, 2017 at 2:22 AM, Takashi Iwai <tiwai@suse.de> wrote:
> On Thu, 05 Oct 2017 20:04:06 +0200,
> Kai-Heng Feng wrote:
>>
>> Like other Plantronics devices, P610 does not support sample
>> rate reading. Apply sample rate quirk to it.
>>
>> BugLink: https://bugs.launchpad.net/bugs/1719853
>>
>> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
>
> Hrm, maybe we should ignore all Plantronics devices?
> Also MS Lifecam and Phoenix devices are such candidates.
>
> So something like below.
>
> Takashi
>
> -- 8< --
> From: Takashi Iwai <tiwai@suse.de>
> Subject: [PATCH] ALSA: usb-audio: Apply vendor ID matching for sample rate
> quirk
>
> So far, lots of Plantronics, MS and Phoenix Audio devices need the
> quirk not to read sample rate back, and the list just grows.
> In this patch, instead of adding each device, apply the quirk by
> matching with these vendors.
>
> Signed-off-by: Takashi Iwai <tiwai@suse.de>
> ---
> sound/usb/quirks.c | 23 +++++++++--------------
> 1 file changed, 9 insertions(+), 14 deletions(-)
>
> diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> index b8cb57aeec77..9aeb05f61f78 100644
> --- a/sound/usb/quirks.c
> +++ b/sound/usb/quirks.c
> @@ -1128,29 +1128,24 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
> /* devices which do not support reading the sample rate. */
> switch (chip->usb_id) {
> case USB_ID(0x041E, 0x4080): /* Creative Live Cam VF0610 */
> - case USB_ID(0x045E, 0x075D): /* MS Lifecam Cinema */
> - case USB_ID(0x045E, 0x076D): /* MS Lifecam HD-5000 */
> - case USB_ID(0x045E, 0x076E): /* MS Lifecam HD-5001 */
> - case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
> - case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
> - case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
> - case USB_ID(0x047F, 0x02F7): /* Plantronics BT-600 */
> - case USB_ID(0x047F, 0x0415): /* Plantronics BT-300 */
> - case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
> - case USB_ID(0x047F, 0xC022): /* Plantronics C310 */
> - case USB_ID(0x047F, 0xC036): /* Plantronics C520-M */
> case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
> case USB_ID(0x0556, 0x0014): /* Phoenix Audio TMX320VC */
> case USB_ID(0x05A3, 0x9420): /* ELP HD USB Camera */
> case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
> case USB_ID(0x1395, 0x740a): /* Sennheiser DECT */
> case USB_ID(0x1901, 0x0191): /* GE B850V3 CP2114 audio interface */
> - case USB_ID(0x1de7, 0x0013): /* Phoenix Audio MT202exe */
> - case USB_ID(0x1de7, 0x0014): /* Phoenix Audio TMX320 */
> - case USB_ID(0x1de7, 0x0114): /* Phoenix Audio MT202pcs */
> case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
> return true;
> }
> +
> + /* devices of these vendors don't support reading rate, either */
> + switch (USB_ID_VENDOR(chip->usb_id)) {
> + case 0x045E: /* MS Lifecam */
> + case 0x047F: /* Plantronics */
> + case 0x1de7: /* Phoenix Audio */
> + return true;
> + }
> +
> return false;
> }
>
This is definitely a better approach. Thanks!
Acked-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> --
> 2.14.2
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610
2017-10-06 4:27 ` Kai-Heng Feng
@ 2017-10-06 8:42 ` Takashi Iwai
0 siblings, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2017-10-06 8:42 UTC (permalink / raw)
To: Kai-Heng Feng; +Cc: perex, alsa-devel, LKML
On Fri, 06 Oct 2017 06:27:59 +0200,
Kai-Heng Feng wrote:
>
> Hi,
>
> On Fri, Oct 6, 2017 at 2:22 AM, Takashi Iwai <tiwai@suse.de> wrote:
> > On Thu, 05 Oct 2017 20:04:06 +0200,
> > Kai-Heng Feng wrote:
> >>
> >> Like other Plantronics devices, P610 does not support sample
> >> rate reading. Apply sample rate quirk to it.
> >>
> >> BugLink: https://bugs.launchpad.net/bugs/1719853
> >>
> >> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> >
> > Hrm, maybe we should ignore all Plantronics devices?
> > Also MS Lifecam and Phoenix devices are such candidates.
> >
> > So something like below.
> >
> > Takashi
> >
> > -- 8< --
> > From: Takashi Iwai <tiwai@suse.de>
> > Subject: [PATCH] ALSA: usb-audio: Apply vendor ID matching for sample rate
> > quirk
> >
> > So far, lots of Plantronics, MS and Phoenix Audio devices need the
> > quirk not to read sample rate back, and the list just grows.
> > In this patch, instead of adding each device, apply the quirk by
> > matching with these vendors.
> >
> > Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > ---
> > sound/usb/quirks.c | 23 +++++++++--------------
> > 1 file changed, 9 insertions(+), 14 deletions(-)
> >
> > diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
> > index b8cb57aeec77..9aeb05f61f78 100644
> > --- a/sound/usb/quirks.c
> > +++ b/sound/usb/quirks.c
> > @@ -1128,29 +1128,24 @@ bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip)
> > /* devices which do not support reading the sample rate. */
> > switch (chip->usb_id) {
> > case USB_ID(0x041E, 0x4080): /* Creative Live Cam VF0610 */
> > - case USB_ID(0x045E, 0x075D): /* MS Lifecam Cinema */
> > - case USB_ID(0x045E, 0x076D): /* MS Lifecam HD-5000 */
> > - case USB_ID(0x045E, 0x076E): /* MS Lifecam HD-5001 */
> > - case USB_ID(0x045E, 0x076F): /* MS Lifecam HD-6000 */
> > - case USB_ID(0x045E, 0x0772): /* MS Lifecam Studio */
> > - case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */
> > - case USB_ID(0x047F, 0x02F7): /* Plantronics BT-600 */
> > - case USB_ID(0x047F, 0x0415): /* Plantronics BT-300 */
> > - case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */
> > - case USB_ID(0x047F, 0xC022): /* Plantronics C310 */
> > - case USB_ID(0x047F, 0xC036): /* Plantronics C520-M */
> > case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */
> > case USB_ID(0x0556, 0x0014): /* Phoenix Audio TMX320VC */
> > case USB_ID(0x05A3, 0x9420): /* ELP HD USB Camera */
> > case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */
> > case USB_ID(0x1395, 0x740a): /* Sennheiser DECT */
> > case USB_ID(0x1901, 0x0191): /* GE B850V3 CP2114 audio interface */
> > - case USB_ID(0x1de7, 0x0013): /* Phoenix Audio MT202exe */
> > - case USB_ID(0x1de7, 0x0014): /* Phoenix Audio TMX320 */
> > - case USB_ID(0x1de7, 0x0114): /* Phoenix Audio MT202pcs */
> > case USB_ID(0x21B4, 0x0081): /* AudioQuest DragonFly */
> > return true;
> > }
> > +
> > + /* devices of these vendors don't support reading rate, either */
> > + switch (USB_ID_VENDOR(chip->usb_id)) {
> > + case 0x045E: /* MS Lifecam */
> > + case 0x047F: /* Plantronics */
> > + case 0x1de7: /* Phoenix Audio */
> > + return true;
> > + }
> > +
> > return false;
> > }
> >
>
> This is definitely a better approach. Thanks!
>
> Acked-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Thanks. I'll queue it for-next branch.
Meanwhile, the fix you submitted is still needed for 4.14, so I merged
it to for-linus branch, too.
Takashi
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-10-06 8:42 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-05 18:04 [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610 Kai-Heng Feng
2017-10-05 18:22 ` Takashi Iwai
2017-10-06 4:27 ` Kai-Heng Feng
2017-10-06 8:42 ` Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®