2018-07-23 08:04:14

by Yue Wang

[permalink] [raw]
Subject: [PATCH] alsa/usb-audio: Topping DX7s quirk for DSD interface

This patch adds the support for native DSD for Topping DX7s DAC. The
product description is on
http://en.tpdz.net/products_detail/productId=32.html and it's
available on https://www.amazon.com/Balanced-Headphone-Amplifier-Topping-2ES9038Q2M/dp/B07B4VFS21

The DAC uses XMOS chip but uses a special vendor id and product id.
The patch will enable the SNDRV_PCM_FMTBIT_DSD_U32_BE bit for the DAC
hence enable native DSD playback up to DSD512 format.

Signed-off-by: Yue Wang <[email protected]>
---
sound/usb/quirks.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index 02b6cc02767f..c51e2dee3075 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -1374,6 +1374,7 @@ u64 snd_usb_interface_dsd_format_quirks(struct snd_usb_audio *chip,
break;

case USB_ID(0x0d8c, 0x0316): /* Hegel HD12 DSD */
+ case USB_ID(0x152a, 0x8750): /* Topping DX7s */
case USB_ID(0x16b0, 0x06b2): /* NuPrime DAC-10 */
case USB_ID(0x16d0, 0x0733): /* Furutech ADL Stratos */
case USB_ID(0x16d0, 0x09db): /* NuPrime Audio DAC-9 */
--
2.18.0



2018-07-23 08:34:53

by Jussi Laako

[permalink] [raw]
Subject: Re: [PATCH] alsa/usb-audio: Topping DX7s quirk for DSD interface

Hi,

> + case USB_ID(0x152a, 0x8750): /* Topping DX7s */
0x152a is Thesycon's VID and if it's based on XMOS I'd recommend to try
adding the 0x152a VID to the generic DSD capability auto-detect list
instead of this old device specific list.

I consider it very likely that it would work that way and at the same
time also cover all other devices with the same firmware and Thesycon's VID.


Best regards,

- Jussi

2018-07-23 08:54:57

by Yue Wang

[permalink] [raw]
Subject: Re: [PATCH] alsa/usb-audio: Topping DX7s quirk for DSD interface

Thanks for the comment Jussi. Let's drop this patch, and I will send a
new patch with this generic approach shortly. The new patch will have
a different commit title and message.
On Mon, Jul 23, 2018 at 1:26 AM Jussi Laako <[email protected]> wrote:
>
> Hi,
>
> > + case USB_ID(0x152a, 0x8750): /* Topping DX7s */
> 0x152a is Thesycon's VID and if it's based on XMOS I'd recommend to try
> adding the 0x152a VID to the generic DSD capability auto-detect list
> instead of this old device specific list.
>
> I consider it very likely that it would work that way and at the same
> time also cover all other devices with the same firmware and Thesycon's VID.
>
>
> Best regards,
>
> - Jussi