Re: [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610

[Kai-Heng Feng]

Hi,

On Fri, Oct 6, 2017 at 2:22 AM, Takashi Iwai <tiwai@xxxxxxx> 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@xxxxxxxxxxxxx>
>
> 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@xxxxxxx>
> 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@xxxxxxx>
> ---
>  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@xxxxxxxxxxxxx>

> --
> 2.14.2
>