Re: [PATCH i-g-t v2 2/2] tests/kms_async_flips: Use lowest mode for crc subtest

["Naladala, Ramanaidu" <Ramanaidu.naladala@intel.com>]

Hi Santhosh,

On 3/17/2026 11:00 AM, Santhosh Reddy Guddati wrote:
> Currently The CRC subtest uses mode[0] which maps to preffered high
> refresh mode. On panels with multiple modes, pick the lower mode to
> speed up crc test.
Here your using a mode where pixel clock is low. Update in git subject 
and commit message.
>
> V2: Rename and move the function to lib to re-use. (Ramanaidu)
>
> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com>
> ---
>   tests/kms_async_flips.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
> index 4a2930968..e8a3da07c 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -819,8 +819,9 @@ static void test_crc(data_t *data)
>   
>   	igt_display_commit2(&data->display, data->display.is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
>   
> -	/* make things faster by using a smallish mode */
> -	mode = &data->output->config.connector->modes[0];
> +	/* make things faster by using a lower bandwidth mode */
> +	mode = igt_output_get_lowclk_mode(data->output);
> +	igt_output_override_mode(data->output, mode);
>   	width = mode->hdisplay;
>   	height = mode->vdisplay;
>   
Observed regression with your changes. Check once.