| Message ID | 20230712-asoc-topology-kunit-enable-v1-1-b9f2da9dca23@kernel.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ASoC: Improve coverage in default KUnit runs | expand |
On Thu, Jul 13, 2023 at 12:19:13PM +0800, David Gow wrote: > While I love the idea of this, it breaks the default UML --alltests > build, as all of ALSA is behind an "if !UML". Oh, UML finally works again - that's good. > ALSA folks, how horrifying a prospect is removing the "if !UML" > everywhere? If it's not trivial, how do we feel about adding > "sound/soc/.kunitconfig" containing these tests? I suspect this is due to UML not providing some key APIs like DMA, if it's building now I guess those dependencies might have been fixed. Do all the drivers build properly if you enable UML, otherwise all the randconfig people will get upset, it could've been to save all the drivers from having to deal with UML features issues? Another option would be for the KUnit runner to just ignore kconfig options getting disabled.
diff --git a/tools/testing/kunit/configs/all_tests.config b/tools/testing/kunit/configs/all_tests.config index 0393940c706a..13d15bc693fb 100644 --- a/tools/testing/kunit/configs/all_tests.config +++ b/tools/testing/kunit/configs/all_tests.config @@ -35,3 +35,7 @@ CONFIG_DAMON_DBGFS=y CONFIG_SECURITY=y CONFIG_SECURITY_APPARMOR=y + +CONFIG_SOUND=y +CONFIG_SND=y +CONFIG_SND_SOC=y
There are KUnit tests for some of the ASoC utility functions which are not enabled in the KUnit all_tests.config, do so. Signed-off-by: Mark Brown <broonie@kernel.org> --- tools/testing/kunit/configs/all_tests.config | 4 ++++ 1 file changed, 4 insertions(+)