[PATCH] kselftests/alsa: pcm - move more configuration to configuration files
Mark Brown
broonie at kernel.org
Thu Dec 1 23:55:37 CET 2022
On Thu, Dec 01, 2022 at 09:42:52PM +0100, Jaroslav Kysela wrote:
> As for the generic tool, I would like to have an option to control all mechanisms:
> 1) only default config
> 2) only hw specific config
> 3) default + hw specific configs (merged)
> A new field in the pcm configuration may be added for this to build the
> per-device-stream configuration. If we merge those two configs, I think that
> only a field which will skip the default config is sufficient to cover all
> possibilities. The test names are a good identification, what was executed.
That's not quite what I implemented (see below...) but I think it's
not too far off.
> As Takashi already applied your set, I'll try to merge my code with yours
> and keep the added functionality.
I've just prepared a branch which I'm running through my tests now which
just does a revert, applies your patch and then does most of what I
said. I'll send it later assuming everything looks OK enough on my test
farm (probably in ~1-2 hours). All this working together is great but
one of us needs to move to a different timezone. :P I think we're
*roughly* on the same page here, just some edges to iron out.
For the config merging/selection thing I think for the merge we could
either add a section in the per-board config which overrides defaults or
just go with what I did where we add the test list, as well as adding
command line options to select only the per-board or only the
per-system. Does any of that line up with what you were thinking?
The following changes since commit 7d721baea138696d5a6746fb5bce0a510a91bd65:
kselftest/alsa: Add more coverage of sample rates and channel counts (2022-12-01 20:02:14 +0100)
are available in the Git repository at:
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/ci.git alsa-pcm-test-hacks
for you to fetch changes up to 9e96c58e581313bd639fd51f37c8e831d7b4a05c:
kselftest/alsa: pcm - Add more coverage by default (2022-12-01 22:46:30 +0000)
----------------------------------------------------------------
Jaroslav Kysela (1):
kselftest/alsa: pcm - move more configuration to configuration files
Mark Brown (6):
kselftest/alsa: pcm - Drop recent coverage improvement changes
kselftest/alsa: pcm - Always run the default set of tests
kselftest/alsa: pcm - skip tests when we fail to set params
kselftest/alsa: pcm - Support optional description for tests
kselftest/alsa: pcm - Provide descriptions for the default tests
kselftest/alsa: pcm - Add more coverage by default
tools/testing/selftests/alsa/Makefile | 2 +-
tools/testing/selftests/alsa/alsa-local.h | 3 +
tools/testing/selftests/alsa/conf.c | 26 ++-
.../alsa/conf.d/Lenovo_ThinkPad_P1_Gen2.conf | 43 +++--
tools/testing/selftests/alsa/pcm-test.c | 205 ++++++++++++++-------
tools/testing/selftests/alsa/pcm-test.conf | 63 +++++++
6 files changed, 250 insertions(+), 92 deletions(-)
create mode 100644 tools/testing/selftests/alsa/pcm-test.conf
-------------- 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/20221201/da32bec2/attachment.sig>
More information about the Alsa-devel
mailing list