
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@cirrus.com wrote:
From: Brian Austin brian.austin@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