[igt-dev] [PATCH i-g-t, v2] tests/kms_hdmi_inject: Fix audio test by loading snd driver

[Shankar, Uma]

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of Nemesa Garg
> Sent: Wednesday, July 26, 2023 7:10 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Garg, Nemesa <nemesa.garg at intel.com>
> Subject: [igt-dev] [PATCH i-g-t, v2] tests/kms_hdmi_inject: Fix audio test by loading
> snd driver
> 
> As audio driver is blacklisted so load the snd_hda_intel module before the
> inject_audio test run.
> 
> v2: Address comments(Kamil)
> 
> Signed-off-by: Nemesa Garg <nemesa.garg at intel.com>
> ---
>  tests/kms_hdmi_inject.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/kms_hdmi_inject.c b/tests/kms_hdmi_inject.c index
> 9aa5c59fc..93a1e53d3 100644
> --- a/tests/kms_hdmi_inject.c
> +++ b/tests/kms_hdmi_inject.c
> @@ -29,6 +29,7 @@
>  #include "igt.h"
>  #include "igt_edid.h"
>  #include "igt_eld.h"
> +#include "igt_kmod.h"
>  #include "xe/xe_query.h"
> 
>  #define HDISPLAY_4K	3840
> @@ -206,6 +207,9 @@ igt_main
>  		igt_require(connector);
> 
>  		kmstest_unset_all_crtcs(drm_fd, res);
> +
> +		if (!igt_kmod_is_loaded("snd_hda_intel"))
> +			igt_kmod_load("snd_hda_intel", NULL);

I think the snd driver can be loaded as part of CI test prep along with i915
and any other dependent module, before the execution begins. This way
individual tests will not need to load at per test level.

@Sharma, Swati2 Can you also check once from validation and CI perspective.

Regards,
Uma Shankar
>  	}
> 
>  	igt_describe("Make sure that 4K modes exposed by DRM match the "
> --
> 2.25.1