[PATCH i-g-t v1] tests/kms_async_flips: Remove unused format modifier checks

Karthik B S karthik.b.s at intel.com
Wed Aug 20 08:40:54 UTC 2025


Hi Santhosh,

On 8/20/2025 11:56 AM, Santhosh Reddy Guddati wrote:
> with IN_FORMATS_ASYNC, hardcoded modifiers are no longer needed
> for invalid async flip subtest.
>
> Fixes: 4972020ea0f3 ("tests/kms_async_flips: Replace hard coded modifiers with async modifiers")
> closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/3768
> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>

LGTM.

Reviewed-by: Karthik B S <karthik.b.s at intel.com>

> ---
>   tests/kms_async_flips.c | 5 -----
>   1 file changed, 5 deletions(-)
>
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
> index 1ac5e6db0..733abf417 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -1186,11 +1186,6 @@ igt_main
>   		data.atomic_path = true;
>   		/* TODO: support more vendors */
>   		igt_require(is_intel_device(data.drm_fd));
> -		igt_require(igt_display_has_format_mod(&data.display, DRM_FORMAT_XRGB8888,
> -						       I915_FORMAT_MOD_X_TILED));
> -		igt_require(igt_display_has_format_mod(&data.display, DRM_FORMAT_XRGB8888,
> -						       I915_FORMAT_MOD_Y_TILED));
> -
>   		run_test(&data, test_invalid);
>   	}
>   


More information about the igt-dev mailing list