Got it, Tested using VID and fp->dsd_raw method. Works as well, patch updated below. Sign off line attached. --- a/sound/usb/quirks.c 2019-10-23 05:38:05.664262661 +0000 +++ b/sound/usb/quirks.c 2019-10-23 05:41:15.907336660 +0000 @@ -1452,6 +1452,7 @@ case 0x2ab6: /* T+A devices */ case 0x3842: /* EVGA */ case 0xc502: /* HiBy devices */ + case 0x292b: /* Gustard/Ess based devices */ if (fp->dsd_raw) return SNDRV_PCM_FMTBIT_DSD_U32_BE; break; By making a contribution to this project, I certify that: 1. The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or 2. The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or 3. The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. 4. I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved. Signed-off-by: Justin Song<flyingecar@xxxxxxxxx> On Tue, Oct 22, 2019 at 10:04 PM Takashi Iwai <tiwai@xxxxxxx> wrote: > On Wed, 23 Oct 2019 06:05:08 +0200, > Fly Pig wrote: > > > > Hi There > > > > This patch add native DSD support for Gustard U16/X26 USB Interface. > Tested on > > my unit, working properly. Hence submit the patch. > > > > Cheers. > > > > Thank you very much. > > > > Pig > > The upstream code changed the DSD detection recently, so your patch > doesn't apply. In short, check whether you get fp->dsd_raw set. If > yes, we can apply the vendor check instead of vendor:device pair > check. > > Please resubmit the patch with the correction, and don't forget to add > your sign-off line (see Documentation/process/submitting-patches.rst). > It's a legal requirement for taking the patch to upstream. > > > thanks, > > Takashi > > > > > --- a/sound/usb/quirks.c 2019-09-22 21:28:09.757897000 +0000 > > +++ b/sound/usb/quirks.c 2019-09-22 21:33:36.892274800 +0000 > > @@ -1369,6 +1369,7 @@ > > case USB_ID(0x22d9, 0x0461): /* OPPO UDP-205 */ > > case USB_ID(0x2522, 0x0012): /* LH Labs VI DAC Infinity */ > > case USB_ID(0x2772, 0x0230): /* Pro-Ject Pre Box S2 Digital */ > > + case USB_ID(0x292b, 0xc4b3): /* Gustard U16 */ > > if (fp->altsetting == 2) > > return SNDRV_PCM_FMTBIT_DSD_U32_BE; > > break; > > > > > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
