[alsa-devel] [PATCH 0/1] Add big-endian DSD sample formats and fix XMOS DSD sample format

Takashi Iwai tiwai at suse.de
Fri Nov 21 13:15:28 CET 2014


At Fri, 21 Nov 2014 14:08:23 +0200 (EET),
Jussi Laako wrote:
> 
> On Fri, 21 Nov 2014, Takashi Iwai wrote:
> 
> > At Fri, 21 Nov 2014 12:00:26 +0200,
> > Jussi Laako wrote:
> > > 
> > > Resend, previous had non matching subject for cover letter vs patch.
> > > 
> > > This patch fixes XMOS DSD sample format to DSD_U32_BE and also adds
> > > DSD_U16_BE and DSD_U32_BE sample formats.
> > > 
> > > Jussi Laako (1):
> > >   Add big-endian DSD sample formats and fix XMOS DSD sample format
> > 
> > The changes there look good (supposing you tested with the real
> > device).  Since DSD_U32 was added in 3.18-rc1, I think we can put this
> > as a fix for 3.18.
> > 
> > Could you submit the patch for alsa-lib as well?
> 
> I sent two patches, one is for kernel and another one is for alsa-lib.

Ah, I missed it.  Then please give a proper prefix to identify that.

Also, the patch without any proper description looks wrong.  What you
wrote in patch 0 should be put into patch 1.  Also better to mention
which devices were actually tested.

Last but not least, we don't need any cover letter for a single
patch.  Just describe the patch itself well.

Could you again resend with proper descriptions?


thanks,

Takashi


> I have tested the patches on top of Ubuntu Trusty using iFi iDSD Nano and 
> everything works.
> 
> 
> Best regards,
> 
> 	- Jussi
> 


More information about the Alsa-devel mailing list