Received: by 10.192.165.148 with SMTP id m20csp2397745imm; Thu, 26 Apr 2018 10:16:22 -0700 (PDT) X-Google-Smtp-Source: AIpwx49iM0G0Y7a5ssbb1dUAj3fwaef36J6FES7dMm1wPQhJQbGkn+ToQ2sH5bmqiEpHrM2xJH6y X-Received: by 10.99.95.14 with SMTP id t14mr27011860pgb.94.1524762982478; Thu, 26 Apr 2018 10:16:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1524762982; cv=none; d=google.com; s=arc-20160816; b=yjYR+CaErvz6hvAKMB6+feCWAnHfY86CK0cCBbIwY7gaClzr2ZmBkTPmHZNbngKuXu XCxsftJmXZ6JrFMg4XjyUtZBFU7ST8bnABPculFjVVYm0H8tWokHkNf5y4VM5laS8Jqi 5gqqHepd4U/JxVFPdkSqH3FL+GExyZyfwgYCEed9Yfm/dojLpOtUp9Bw7H2y/t3fMM1J COlJ93Tz+oyBHlMUB+B4iUJjG59cEN5ZypOyJkGtik7OE9TLT4GDP+zqw/w0K9XKYZTP aj797pXM6Tc1g5/Fiart3YDt7Qc5D8GXrr+ktXoSyBLu3lIIexf/ty7nZbcq4rFyyjP8 ff/Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=rDTwcEmJVaIoDhfD8arfg0vpQVZAObnTTaM6GWyUx4k=; b=Q4VbnaySu/6V6u3TDZO3WadLaGE8S7Z6ZoqBzjnVvFgVYYXNbxrinRG3yjaqG2eVVf vtpigEEq+wnyqEhHyPYIwelh7EgQHOJc0n+kdYoeOYS1INf6bBcmKrRw7+he1JCF8reH cAz4LFP9bbZ8xzh9P1H3/RBrooNADFNdR5u4d20y0BmilptBxeLNnWqDx3cOTzYXlRQe R4O82K/ndSxWVbUak3cG6fDZBM8VT7Q9EoU/slVT1pNHHT6zqF4BZ3Pr/+7fYV3lS5Zf 5x4CqkuhaVpbah094OODAeJDYVAyMrEiueC6UEHBzOMWlW/XYN5R6BWIPtTCcYJEIrB5 Zgpw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=codethink.co.uk Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c12-v6si6083685plr.467.2018.04.26.10.16.07; Thu, 26 Apr 2018 10:16:22 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=codethink.co.uk Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756831AbeDZRNY (ORCPT + 99 others); Thu, 26 Apr 2018 13:13:24 -0400 Received: from imap1.codethink.co.uk ([176.9.8.82]:35128 "EHLO imap1.codethink.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754419AbeDZRNX (ORCPT ); Thu, 26 Apr 2018 13:13:23 -0400 Received: from 167-98-27-229.cust-167.exponential-e.net ([167.98.27.229] helo=[10.24.4.248]) by imap1.codethink.co.uk with esmtpsa (Exim 4.84_2 #1 (Debian)) id 1fBkSO-0001b5-LB; Thu, 26 Apr 2018 18:13:20 +0100 Subject: Re: [alsa-devel] [PATCH v2 0/4] ALSA: usb: UAC3 new features. To: Ruslan Bilovol , Takashi Iwai Cc: Greg Kroah-Hartman , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org References: <20180420170327.31569-1-jorge.sanjuan@codethink.co.uk> <20180424172445.31928-1-jorge.sanjuan@codethink.co.uk> From: Jorge Message-ID: <74748a72-dfba-666f-aaf7-b2976cda9b87@codethink.co.uk> Date: Thu, 26 Apr 2018 18:13:20 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 26/04/18 10:26, Ruslan Bilovol wrote: > On Tue, Apr 24, 2018 at 9:02 PM, Takashi Iwai wrote: >> On Tue, 24 Apr 2018 19:24:41 +0200, >> Jorge Sanjuan wrote: >>> >>> v2 fixes: >>> - If/else statements braces style fixes. >>> - Add wrapping function to mixer unit code. >>> - Make connectors control kctl struct const. >>> - Little endian to cpu conversion in several places. >>> - Sing off and add Fixes tag to fixup commit. >>> - Remove flex-array for a struct that is used statically. >>> >>> Now that the UAC3 patch [1] has made it to linux-next I have some extra >>> features to make a UAC3 device fully work in Linux. Including Jack >>> insertion control that I have put on top of this other patch [2] for >>> UAC2. Also adding support for the UAC3 Mixer Unit which is most likely >>> to appear in most headset type devices. > > Thanks for adding these improvements! > >> >> These patches look reasonable, I'm OK to merge. But I'll wait for >> Ruslan's comments (or at best with test results). > > I reviewed first 3 patches and will review jack detection patch later, > and I'm going to test this patchset in a next few days. > >> >>> UAC3 devices also require to have a Basic Audio Device (BADD) in a separate >>> config for which both Ruslan Bilovol and myself have submited different >>> approaches[3][4] but I don't know what the final merge will be. Once there >>> is official support for BADD, we'll need to test it with an actual UAC3 >>> device to confirm it all wokrs. >> >> Could you guys try to get agreement which approach should we take? >> >> I have no big preference. Currently Ruslan's patch series look >> easier, just because its addition is a bit smaller, though. > > The BADD devices are quite simple, so direct initialization internal ALSA > structures looks easy and straightforward, comparing to generation of > missing descriptors. > I'm currently improving the patch series so it will look even more > smaller and easier, let's see how it goes Hi Ruslan, I agree that the BADD devices may not require that much logic using all the descriptors. Besides, what makes your approach more interesting to me is the fact that there is no need to bypass the cluster descriptor every single time if the UAC3 device operates in BADD mode. I have not yet tested your patch with the UAC3 device that I have. I was wondering whether that BADD mixer code will work with and AS interface with several alt settings with different endpoints wMaxPacketSize. That is something I am working on/testing for this device. I'd have to take a closer look to the patch to provide some useful input on that. Thanks, Jorge > > Thanks, > Ruslan > >> >> >> Thanks! >> >> Takashi >> >>> All this features are tested with an actual UAC3 device that is still in >>> development. For this patch series, only the legacy config (#1. UAC1/UAC2) >>> and the UAC3 config have been tested. The BADD config is only tested using >>> and updated verison of [4]. >>> >>> [1]: https://patchwork.kernel.org/patch/10298179/ >>> [2]: https://patchwork.kernel.org/patch/10305847/ >>> [3]: https://patchwork.kernel.org/patch/10340851/ >>> [4]: https://www.spinics.net/lists/alsa-devel/msg71617.html >>> >>> Based on linux-next tag: next-20180420 >>> >>> Jorge Sanjuan (3): >>> ALSA: usb-audio: UAC3. Add support for mixer unit. >>> ALSA: usb-audio: Use Class Specific EP for UAC3 devices. >>> ALSA: usb-audio: UAC3 Add support for connector insertion. >>> >>> Michael Drake (1): >>> ALSA: usb-audio: ADC3: Fix channel mapping conversion for ADC3. >>> >>> include/linux/usb/audio-v2.h | 7 ++ >>> include/linux/usb/audio-v3.h | 14 +++ >>> include/uapi/linux/usb/audio.h | 13 ++- >>> sound/usb/mixer.c | 195 +++++++++++++++++++++++++++++++++++++---- >>> sound/usb/stream.c | 11 ++- >>> 5 files changed, 217 insertions(+), 23 deletions(-) >>> >>> -- >>> 2.11.0 >>> >>> >> _______________________________________________ >> Alsa-devel mailing list >> Alsa-devel@alsa-project.org >> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel >