From mboxrd@z Thu Jan 1 00:00:00 1970 From: Clemens Ladisch Subject: Re: [PATCH] add native DSD support for XMOS based DACs. Date: Fri, 05 Sep 2014 12:29:06 +0200 Message-ID: <54099072.3040202@ladisch.de> References: <1409906971-3365-1-git-send-email-gtmkramer@xs4all.nl> <54098410.6050103@ladisch.de> <1409910794.2959.1.camel@xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from dehamd003.servertools24.de (dehamd003.servertools24.de [31.47.254.18]) by alsa0.perex.cz (Postfix) with ESMTP id 82FC4261A06 for ; Fri, 5 Sep 2014 12:29:08 +0200 (CEST) In-Reply-To: <1409910794.2959.1.camel@xs4all.nl> 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: Jurgen Kramer Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org Jurgen Kramer wrote: > On Fri, 2014-09-05 at 11:36 +0200, Clemens Ladisch wrote: >> Jurgen Kramer wrote: >>> Add quirk for XMOS based DACs for native DSD playback support using the new DSD_U32_LE sample >>> format. Works for all DAC vendors which use XMOS as idVendor. E.g. iFi Audio >>> with their micro iDSD and nano iDSD. >>> >>> +++ b/sound/usb/quirks.c >>> + /* XMOS based DACs, e.g. iFi Audio micro/nano iDSD */ >>> + if (le16_to_cpu(chip->dev->descriptor.idVendor) == 0x20b1) { >>> + switch (fp->altsetting) { >>> + case 2: >>> + return SNDRV_PCM_FMTBIT_DSD_U32_LE; >> >> Can you confirm that all XMOS devices ever built will have DSD on that >> alternate setting? > > No, I can not. Most newer implementations will use alt setting 2 for > native DSD support. But it's possible to have other XMOS-based devices that use multiple alternate settings for PCM. > My first version also used idProduct to only target iFi devices. If > this is required, I'll rework my patch. I guess there isn't any property of the descriptors to distinguish DoP from PCM? How does the generic Windows driver know which alternate settings are DoP? Regards, Clemens