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

Takashi Iwai tiwai at suse.de
Wed Nov 30 14:52:33 CET 2022


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


More information about the Alsa-devel mailing list