Re: [PATCH 1/2] ALSA: add DSD formats

[Daniel Mack <zonque@gmail.com>]

On 06.04.2013 19:53, Jussi Laako wrote:
> On 04/06/2013 02:44 PM, Daniel Mack wrote:
>> +#define SNDRV_PCM_FMTBIT_DSD_DOP_U8	_SNDRV_PCM_FMTBIT(DSD_DOP_U8)
>> +#define SNDRV_PCM_FMTBIT_DSD_DOP_U16	_SNDRV_PCM_FMTBIT(DSD_DOP_U16)
> 
> No, this is not good, DoP always has the 8-bit marker in the MSB and has 
> to be either 24-bit or longer word. One byte marker and two bytes of 
> payload, padded with one byte of zeros if payload is 32-bit dword.
> 

Well, the idea was to denote the fact that transport goes in multiple of
8 or 16 bits, which can then be interpreted by the calculation of the
actual sample rate. But I'm fine with anything else. What name would you
pick, assuming that you're fine with the general approach?


Thanks,
Daniel