Re: [PATCH v1 6/6] kselftest/alsa: Add more coverage of sample rates and channel counts

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 30 Nov 2022 14:42:49 +0100,
Mark Brown wrote:
> 
> On Wed, Nov 30, 2022 at 12:06:08AM +0000, Mark Brown wrote:
> 
> > +	{ "8k.1.big",    "S16_LE",   8000, 2,     2000,   16000 },
> > +	{ "8k.2.big",    "S16_LE",   8000, 2,     4000,   32000 },
> > +	{ "44k1.2.big",  "S16_LE",  44100, 2,    22050,  192000 },
> > +	{ "48k.2.small", "S16_LE",  48000, 2,      512,    4096 },
> > +	{ "48k.2.big",   "S16_LE",  48000, 2,    24000,  192000 },
> > +	{ "48k.6.big",   "S16_LE",  48000, 6,    48000,  576000 },
> > +	{ "96k.2.big",   "S16_LE",  96000, 2,    48000,  384000 },
> 
> It looks like the period/buffer size numbers need some tuning other than
> for 44.1kHz and 48kHz 6 channel, if I do any more fiddling with this
> series I'll split the problematic ones out into a separate commit if I
> don't get things sorted (but we could just leave things as-is and fix
> incrementally too I guess).

I'm fine in either way.  I wait for Jaroslav's review for now.


thanks,

Takashi



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux