Re: [PATCH i-g-t] tests/kms_bw: use igt_display_n_crtcs()

[Karthik B S <karthik.b.s@intel.com>]

On 4/20/2026 1:39 PM, Jani Nikula wrote:
> Now that igt_display_n_crtcs() returns the actual number of CRTCs again,
> use it instead of iterating and counting separately.
>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Karthik B S <karthik.b.s@intel.com>
> ---
>   tests/kms_bw.c | 8 ++------
>   1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/tests/kms_bw.c b/tests/kms_bw.c
> index 4df5c2cee885..00677b4ae765 100644
> --- a/tests/kms_bw.c
> +++ b/tests/kms_bw.c
> @@ -220,16 +220,12 @@ static void run_test_linear_tiling(data_t *data, int n_crtcs, const drmModeModeI
>   	igt_output_t *output;
>   	struct igt_fb buffer[IGT_MAX_PIPES];
>   	igt_crc_t zero, captured[IGT_MAX_PIPES];
> -	int i = 0, num_pipes = 0;
> +	int i = 0;
>   	igt_crtc_t *crtc;
>   	int ret;
>   	bool has_supported_mode = false;
>   
> -	/* Cannot use igt_display_n_crtcs() due to fused pipes on i915 where they do
> -	 * not give the numver of valid crtcs and always return IGT_MAX_PIPES */
> -	for_each_crtc(display, crtc) num_pipes++;
> -
> -	igt_skip_on_f(n_crtcs > num_pipes,
> +	igt_skip_on_f(n_crtcs > igt_display_n_crtcs(display),
>                         "ASIC does not have %d pipes\n", n_crtcs);
>   
>   	test_init(data, physical);