[igt-dev] [PATCH] tests/kms_cursor_legacy: igt_require_pipe_crc for flip-vs-cursor{-busy}-crc

[Petri Latvala]

On Wed, Dec 08, 2021 at 11:34:40AM -0500, Mark Yacoub wrote:
> From: Mark Yacoub <markyacoub at google.com>
> 
> [Why]
> flip_vs_cursor_crc and flip_vs_cursor_busy_crc use CRC in debugfs which is
> not supported by some drivers.
> 
> [How]
> igt_require_pipe_crc in igt_fixture before the subtests start.
> 
> Test: igt at kms_cursor_legacy@flip-vs-cursor{-busy}-crc-{atomic,legacy} on
> ChromeOS Jacuzzi (MTK)
> 
> Signed-off-by: Mark Yacoub <markyacoub at chromium.org>

Reviewed-by: Petri Latvala <petri.latvala at intel.com>

> ---
>  tests/kms_cursor_legacy.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> index d50155e8..47b9b4e1 100644
> --- a/tests/kms_cursor_legacy.c
> +++ b/tests/kms_cursor_legacy.c
> @@ -1533,6 +1533,8 @@ igt_main
>  	/*Test description for cursor-crc tests*/
>  	igt_describe("Test will first does a page flip and then cursor update");
>  	igt_subtest_group {
> +		igt_fixture
> +			igt_require_pipe_crc(display.drm_fd);
>  
>  		igt_subtest("flip-vs-cursor-crc-legacy")
>  			flip_vs_cursor_crc(&display, false);
> @@ -1544,6 +1546,8 @@ igt_main
>  	/*Test description for busy-crc tests*/
>  	igt_describe("this test perform a busy bo update followed by a cursor update");
>  	igt_subtest_group {
> +		igt_fixture
> +			igt_require_pipe_crc(display.drm_fd);
>  
>  		igt_subtest("flip-vs-cursor-busy-crc-legacy")
>  			flip_vs_cursor_busy_crc(&display, false);
> -- 
> 2.34.1.400.ga245620fadb-goog
>