Re: [PATCH i-g-t 3/5] tests/nouveau_crc: Use igt_first_crtc() instead of igt_crtc_for_pipe()

[Jani Nikula <jani.nikula@intel.com>]

On Thu, 05 Mar 2026, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> The specific CRTC used for the ctx-flip-threshold-reset-after-capture
> subtest supposedly doesn't matter, so find one using the more hardware
> agnostic igt_first_crtc() rather than igt_crtc_for_pipe().

In all likelyhood they're the same thing anyway.

Reviewed-by: Jani Nikula <jani.nikula@intel.com>

>
> Cc: Lyude Paul <lyude@redhat.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  tests/nouveau_crc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/nouveau_crc.c b/tests/nouveau_crc.c
> index 59fa8a2daeb8..b71aac45bb79 100644
> --- a/tests/nouveau_crc.c
> +++ b/tests/nouveau_crc.c
> @@ -401,8 +401,8 @@ int igt_main()
>  	igt_describe("Make sure that the CRC notifier context flip threshold "
>  		     "is reset to its default value after a single capture.");
>  	igt_subtest("ctx-flip-threshold-reset-after-capture") {
> -		data.crtc = igt_crtc_for_pipe(&data.display, PIPE_A);
> -		igt_require(data.crtc->valid);
> +		data.crtc = igt_first_crtc(&data.display);
> +		igt_require(data.crtc);
>  
>  		run_test(&data, test_ctx_flip_threshold_reset_after_capture);
>  	}

-- 
Jani Nikula, Intel