Re: [PATCH v2] [RFC] selftests: alsa - add PCM test

[Takashi Iwai <tiwai@suse.de>]

On Tue, 08 Nov 2022 12:59:14 +0100,
Jaroslav Kysela wrote:
> 
> This initial code does a simple sample transfer tests. By default,
> all PCM devices are detected and tested with short and long
> buffering parameters for 4 seconds. If the sample transfer timing
> is not in a +-100ms boundary, the test fails. Only the interleaved
> buffering scheme is supported in this version.
> 
> The configuration may be modified with the configuration files.
> A specific hardware configuration is detected and activated
> using the sysfs regex matching. This allows to use the DMI string
> (/sys/class/dmi/id/* tree) or any other system parameters
> exposed in sysfs for the matching for the CI automation.
> 
> The configuration file may also specify the PCM device list to detect
> the missing PCM devices.
> 
> v1..v2:
>   - added missing alsa-local.h header file
> 
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Pierre-Louis Bossart <pierre-louis.bossart@intel.com>
> Cc: Liam Girdwood <liam.r.girdwood@intel.com>
> Cc: Jesse Barnes <jsbarnes@google.com>
> Cc: Jimmy Cheng-Yi Chiang <cychiang@google.com>
> Cc: Curtis Malainey <cujomalainey@google.com>
> Cc: Brian Norris <briannorris@chromium.org>
> Signed-off-by: Jaroslav Kysela <perex@perex.cz>

Sorry for the late reply.

The patch looks good and works fine on the smoke test, so I applied it
now.  There are a few issues Mark already commented on, and we can fix
or improve them in incremental manner later.


thanks,

Takashi