Re: [PATCH i-g-t v4 09/12] tests/intel/kms_fbcon_fbt: find and store the crtc index

["Reddy Guddati, Santhosh" <santhosh.reddy.guddati@intel.com>]

On 02-06-2026 02:14, Vinod Govindapillai wrote:
> As we need to find the fbc status from crtc debugfs entry, find
> and keep the crtc index from the crtc_id and use it wherever
> required.
> 
> Signed-off-by: Vinod Govindapillai <vinod.govindapillai@intel.com>

Reviewed-by: Santhosh Reddy Guddati <santhosh.reddy.guddati@intel.com>

> ---
>   tests/intel/kms_fbcon_fbt.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/intel/kms_fbcon_fbt.c b/tests/intel/kms_fbcon_fbt.c
> index 5863c9e68..96136f8ff 100644
> --- a/tests/intel/kms_fbcon_fbt.c
> +++ b/tests/intel/kms_fbcon_fbt.c
> @@ -68,7 +68,7 @@ IGT_TEST_DESCRIPTION("Test the relationship between fbcon and the frontbuffer "
>   #define MAX_CONNECTORS 32
>   
>   struct drm_info {
> -	int fd, debugfs_fd, crtc_id, devid;
> +	int fd, debugfs_fd, crtc_id, crtc_index, devid;
>   	struct igt_fb fb;
>   	drmModeResPtr res;
>   	drmModeConnectorPtr connectors[MAX_CONNECTORS];
> @@ -149,10 +149,9 @@ static bool fbc_check_cursor_blinking(struct drm_info *drm)
>   	igt_pipe_crc_t *pipe_crc;
>   	igt_crc_t crc[2];
>   	bool ret;
> -	int i, crtc_index;
> +	int i;
>   
> -	crtc_index = kmstest_get_crtc_index_from_id(drm->fd, drm->crtc_id);
> -	pipe_crc = igt_pipe_crc_new(drm->fd, crtc_index, IGT_PIPE_CRC_SOURCE_AUTO);
> +	pipe_crc = igt_pipe_crc_new(drm->fd, drm->crtc_index, IGT_PIPE_CRC_SOURCE_AUTO);
>   
>   	igt_pipe_crc_start(pipe_crc);
>   	igt_pipe_crc_drain(pipe_crc);
> @@ -222,6 +221,7 @@ static void set_mode_for_one_screen(struct drm_info *drm,
>   		      "No connector available\n");
>   
>   	drm->crtc_id = kmstest_find_crtc_for_connector(drm->fd, drm->res, c, 0);
> +	drm->crtc_index = kmstest_get_crtc_index_from_id(drm->fd, drm->crtc_id);
>   
>   	buffer_id = igt_create_fb(drm->fd, mode->hdisplay, mode->vdisplay,
>   				  DRM_FORMAT_XRGB8888,