From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751921AbdJFImN (ORCPT ); Fri, 6 Oct 2017 04:42:13 -0400 Received: from mx2.suse.de ([195.135.220.15]:33941 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751430AbdJFImN (ORCPT ); Fri, 6 Oct 2017 04:42:13 -0400 Date: Fri, 06 Oct 2017 10:42:10 +0200 Message-ID: From: Takashi Iwai To: Kai-Heng Feng Cc: perex@perex.cz, alsa-devel@alsa-project.org, LKML Subject: Re: [PATCH] ALSA: usb-audio: Add sample rate quirk for Plantronics P610 In-Reply-To: References: <20171005180406.748-1-kai.heng.feng@canonical.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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 > > > > 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 > > 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 > > --- > > 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 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