From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752368AbbDDMIZ (ORCPT ); Sat, 4 Apr 2015 08:08:25 -0400 Received: from cantor2.suse.de ([195.135.220.15]:57498 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751981AbbDDMIW (ORCPT ); Sat, 4 Apr 2015 08:08:22 -0400 Date: Sat, 04 Apr 2015 14:08:20 +0200 Message-ID: From: Takashi Iwai To: Eric Wong Cc: Joe Turner , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ALSA: usb-audio: don't try to get Benchmark DAC1 sample rate In-Reply-To: <20150331073405.GA5912@dcvr.yhbt.net> References: <20150331073405.GA5912@dcvr.yhbt.net> 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/24.4 (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 At Tue, 31 Mar 2015 07:34:05 +0000, Eric Wong wrote: > > Adding this quirk allows us to avoid the noisy > "cannot get freq at ep 0x1" message in dmesg output every time > playback starts. > > This ought to affect other Benchmark DAC1 variations using the same > "Microchip Technology, Inc." chip as well, but I have only tested > with the "Pre" variant. > > Signed-off-by: Eric Wong > Cc: Joe Turner > Cc: Takashi Iwai Applied, thanks. Takashi > --- > I also had another generic patch prepared before I noticed Joe's > quirk addition for the MS Lifecam HD-5000. > > sound/usb/quirks.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c > index 753a47d..9a28365 100644 > --- a/sound/usb/quirks.c > +++ b/sound/usb/quirks.c > @@ -1113,8 +1113,13 @@ void snd_usb_set_format_quirk(struct snd_usb_substream *subs, > > bool snd_usb_get_sample_rate_quirk(struct snd_usb_audio *chip) > { > - /* MS Lifecam HD-5000 doesn't support reading the sample rate. */ > - return chip->usb_id == USB_ID(0x045E, 0x076D); > + /* devices which do not support reading the sample rate. */ > + switch (chip->usb_id) { > + case USB_ID(0x045E, 0x076D): /* MS Lifecam HD-5000 */ > + case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */ > + return true; > + } > + return false; > } > > /* Marantz/Denon USB DACs need a vendor cmd to switch > -- > EW >