Re: [PATCH] ALSA: core: Add 384kHz Support

[Charles Keepax <ckeepax@opensource.wolfsonmicro.com>]

On Wed, Jan 27, 2016 at 05:55:40PM +0100, Lars-Peter Clausen wrote:
> On 01/27/2016 04:51 PM, Takashi Iwai wrote:
> > On Wed, 27 Jan 2016 16:02:35 +0100,
> > Brian Austin wrote:
> >>
> >> On Wed, 27 Jan 2016, Takashi Iwai wrote:
> >>
> >>>> 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
> >>>
> >> I really, really, really want it :)
> >>
> >> I understand. Going forward, from our perspective, 384 and 
> >> other high sample rates are going to be defaults for devices as the market 
> >> is moving that way. I just wanted to make it easier to use those instead 
> >> of doing all the contraint coding.
> > 
> > Yeah that's the reason I could *guess*, but it wasn't mentioned.
> > 
> >> Now my understanding on the KNOT define 
> >> may be wrong. I can add the other rates to this, but for now just wanted 
> >> to add one we use currently.
> >>
> >> Does that make sense? 
> > 
> > Just resubmit the patch with mo' better advertisement :)
> 
> For symmetry reasons maybe also include the matching 44.1kHz based rate.
> Unless you think that is not going to be a thing.

Everything I have seen using >192k sample rates has been using
the 48k based rate families. My guess would be that the matching
44.1k rate is unlikely to be a thing.

Thanks,
Charles