[alsa-devel] [PATCH] ALSA: core: Add 384kHz Support

Takashi Iwai tiwai at suse.de
Wed Jan 27 15:36:08 CET 2016


On Wed, 27 Jan 2016 15:11:10 +0100,
Brian Austin wrote:
> 
> On Wed, 27 Jan 2016, Takashi Iwai wrote:
> 
> > On Wed, 27 Jan 2016 14:53:45 +0100,
> > <brian.austin at cirrus.com> wrote:
> > > 
> > > From: Brian Austin <brian.austin at cirrus.com>
> > > 
> > > Adding support for 384kHz audio sample rates
> > 
> > The most important information is missing: why this is needed.
> So I can specify that I want 384kHz. 
> > 
> > (Note that the support of 384kHz itself can be done without this
> >  addition :) 
> > 
> So we stay at SNDRV_PCM_RATE_192000 always for sample rate defines?
> And everything else is just covered with SNDRV_PCM_RATE_KNOT? 

Depends.  If a new sample rate is (or will be) demanded by many
drivers, it's worth to add it, of course, as it would simplify the
code.

That's why I mentioned "why" is most important information; you need
to convince others about the necessity of this change, after all.
Then you see that "because I-wanna-it" doesn't sound convincing
enough, right?


Takashi


More information about the Alsa-devel mailing list