[igt-dev] [PATCH 2/2 i-g-t] tests/kms_universal_plane: enable gen9_test_pipe for all devices

[Petri Latvala]

On Fri, May 06, 2022 at 03:22:53PM -0600, Alex Hung wrote:
> This subtest runs on non-i915 devices, and let's make it generic by
> removing igt_require_intel and renaming it.
> 
> Signed-off-by: Alex Hung <alex.hung at amd.com>
> ---
>  tests/kms_universal_plane.c | 11 +++++------
>  1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
> index 3a3667e6..65f898fe 100644
> --- a/tests/kms_universal_plane.c
> +++ b/tests/kms_universal_plane.c
> @@ -701,15 +701,14 @@ gen9_test_fini(gen9_test_t *test, igt_output_t *output)
>   * windowing)
>   */
>  static void
> -gen9_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
> +pageflip_win_test_pipe(data_t *data, enum pipe pipe, igt_output_t *output)
>  {
>  	gen9_test_t test = { .data = data };
>  	igt_plane_t *primary;
>  
>  	int ret = 0;
> -
> -	igt_require_intel(data->drm_fd);
> -	igt_skip_on(data->display_ver < 9);
> +	if (is_i915_device(data->drm_fd))
> +		igt_skip_on(data->display_ver < 9);
>  	igt_require_pipe(&data->display, pipe);
>  
>  	igt_output_set_pipe(output, pipe);
> @@ -789,10 +788,10 @@ run_tests_for_pipe(data_t *data, enum pipe pipe)
>  			cursor_leak_test_pipe(data, pipe, output);
>  
>  	igt_describe("Check if pageflip succeeds in windowed setting");
> -	igt_subtest_f("universal-plane-gen9-features-pipe-%s",
> +	igt_subtest_f("universal-plane-pageflip_windowed-pipe-%s",

Don't mix dashes and underscores, make it pageflip-windowed.


-- 
Petri Latvala


>  		      kmstest_pipe_name(pipe))
>  		for_each_valid_output_on_pipe(&data->display, pipe, output)
> -			gen9_test_pipe(data, pipe, output);
> +			pageflip_win_test_pipe(data, pipe, output);
>  }
>  
>  static data_t data;
> -- 
> 2.25.1
>