[igt-dev] [PATCH igt] tests/kms_bw: Skip unsupported modes

[Jessica Zhang]

On 7/9/2022 9:23 AM, Rob Clark wrote:
> From: Rob Clark <robdclark at chromium.org>
> 
> Signed-off-by: Rob Clark <robdclark at chromium.org>

Will merge after CI passes:

Tested-by: Jessica Zhang <quic_jesszhan at quicinc.com> # Trogdor (sc7180)
Reviewed-by: Jessica Zhang <quic_jesszhan at quicinc.com>

Thanks,

Jessica Zhang

> ---
>   tests/kms_bw.c | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/tests/kms_bw.c b/tests/kms_bw.c
> index a4553cde..7bb1332e 100644
> --- a/tests/kms_bw.c
> +++ b/tests/kms_bw.c
> @@ -146,6 +146,7 @@ static void run_test_linear_tiling(data_t *data, int pipe, const drmModeModeInfo
>   	igt_crc_t zero, captured[IGT_MAX_PIPES];
>   	int i = 0, num_pipes = 0;
>   	enum pipe p;
> +	int ret;
>   
>   	/* Cannot use igt_display_get_n_pipes() due to fused pipes on i915 where they do
>   	 * not give the numver of valid crtcs and always return IGT_MAX_PIPES */
> @@ -175,6 +176,12 @@ static void run_test_linear_tiling(data_t *data, int pipe, const drmModeModeInfo
>   		igt_plane_set_fb(data->primary[i], &buffer[i]);
>   	}
>   
> +	ret = igt_display_try_commit_atomic(display,
> +					    DRM_MODE_ATOMIC_ALLOW_MODESET |
> +					    DRM_MODE_ATOMIC_TEST_ONLY,
> +					    NULL);
> +	igt_skip_on_f(ret != 0, "Unsupported mode\n");
> +
>   	igt_display_commit_atomic(display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>   
>   	for (i = 0; i <= pipe; i++) {
> -- 
> 2.36.1
>