From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Mack Subject: Re: [PATCH] ALSA: snd-usb: sample rate detection workaround for Axe-Fx II Date: Sun, 08 Sep 2013 19:52:40 +0200 Message-ID: <522CB968.7080307@gmail.com> References: <1378658119-31056-1-git-send-email-zonque@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-bk0-f45.google.com (mail-bk0-f45.google.com [209.85.214.45]) by alsa0.perex.cz (Postfix) with ESMTP id 52B5F26262D for ; Sun, 8 Sep 2013 19:52:42 +0200 (CEST) Received: by mail-bk0-f45.google.com with SMTP id mx11so1986784bkb.32 for ; Sun, 08 Sep 2013 10:52:42 -0700 (PDT) In-Reply-To: <1378658119-31056-1-git-send-email-zonque@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Daniel Mack Cc: tiwai@suse.de, alsa-devel@alsa-project.org, clemens@ladisch.de, joachim.gahl@web.de List-Id: alsa-devel@alsa-project.org On 08.09.2013 18:35, Daniel Mack wrote: > Fractal Audio Systems Axe-Fx II and maybe other devices fail to return > their sample rate range descriptor when only asked for the first two > bytes. > > Add a workaround and assume exactly one sample rate in this case, and > continue with the full descriptor fetch. Please ignore this patch for now. In an off-list discussion, it turned out that this device needs a manual firmware download each time it is connected, and the vendor is willing to fix that bug on the firmware side. Hence, that issue can be fixed for all users at once, and so we won't need that nasty workaround eventually. Thanks, Daniel > > Signed-off-by: Daniel Mack > Reported-and-tested-by: Joachim Gahl > --- > sound/usb/format.c | 21 +++++++++++++-------- > 1 file changed, 13 insertions(+), 8 deletions(-) > > diff --git a/sound/usb/format.c b/sound/usb/format.c > index 3525231..d9de43d 100644 > --- a/sound/usb/format.c > +++ b/sound/usb/format.c > @@ -297,13 +297,17 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip, > snd_usb_ctrl_intf(chip) | (clock << 8), > tmp, sizeof(tmp)); > > - if (ret < 0) { > - snd_printk(KERN_ERR "%s(): unable to retrieve number of sample rates (clock %d)\n", > - __func__, clock); > - goto err; > - } > + /* > + * Some devices, including Fractal Audio Systems Axe-Fx II, fail to > + * return only a fraction of this descriptor. If that happens, assume > + * we have only one sample rate and continue. In case of any other > + * error, we will bail on the full descriptor fetch below. > + */ > + if (ret < 0) > + nr_triplets = 1; > + else > + nr_triplets = combine_word(tmp); > > - nr_triplets = (tmp[1] << 8) | tmp[0]; > data_size = 2 + 12 * nr_triplets; > data = kzalloc(data_size, GFP_KERNEL); > if (!data) { > @@ -319,8 +323,9 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip, > data, data_size); > > if (ret < 0) { > - snd_printk(KERN_ERR "%s(): unable to retrieve sample rate range (clock %d)\n", > - __func__, clock); > + snd_printk(KERN_ERR "%s(): unable to retrieve sample rate range " > + "(clock %d, nr_triplets %d)\n", > + __func__, clock, nr_triplets); > ret = -EINVAL; > goto err_free; > } >