[PATCH v2] kselftest: alsa: Add simplistic test for ALSA mixer controls kselftest

Mark Brown broonie at kernel.org
Mon Dec 6 19:17:08 CET 2021


On Mon, Dec 06, 2021 at 11:01:25AM -0600, Pierre-Louis Bossart wrote:

> > It's not really idiomatic for kselftest to require any per system
> > configuration by default - half the thing is that you can just run it
> > and it should do as much as it can sensibly on the system.  You could
> > definitely add some command line options for development or manual usage
> > though.

> I was thinking of adding this test to our CI, it's a bit orthogonal to
> self-tests indeed. IIRC we check that we can modify all the PGA settings

I do think it's useful to run there, it'd give coverage for all the
CODEC drivers in your systems and of course for controls added for the
DSPs.

> for volume control but this test is a lot more generic.

It should be fine to run in your CI as it is - assuming there's no bugs
it finds in the USB cards anyway.  It tries to leave the card in the
state it found it in so providing we can read and write whatever the
current settings are it should leave things configured as they were.
I'm not against adding an option to run on specific cards, you'd have to
run it outside of the kselftest harness for that but that's more of a
thing for your CI anyway.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20211206/c5726e84/attachment.sig>


More information about the Alsa-devel mailing list