Re: [PATCH i-g-t] tests/intel/kms_dirtyfb: Fix misplaced parenthesis in display version check

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

Hi Jeevan,

On 04-04-2025 10:45, Jeevan B wrote:
> Fixed a misplaced parenthesis in the FBC display version check to ensure
> correct tiling format selection.
> 
> Fixes: f0b66883398b4a07bb8fcaa3663a2140129d9a58
> Signed-off-by: Jeevan B <jeevan.b@intel.com>

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

> ---
>   tests/intel/kms_dirtyfb.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/kms_dirtyfb.c b/tests/intel/kms_dirtyfb.c
> index 8efa14447..a5707c5b1 100644
> --- a/tests/intel/kms_dirtyfb.c
> +++ b/tests/intel/kms_dirtyfb.c
> @@ -386,7 +386,7 @@ igt_main
>   
>   					/* FBC Disp_ver 8 and below supports only I915_FORMAT_MOD_X_TILED */
>   					if (data.feature == FEATURE_FBC &&
> -					    intel_display_ver(intel_get_drm_devid(data.drm_fd) <= 8))
> +					    intel_display_ver(intel_get_drm_devid(data.drm_fd)) <= 8)
>   						data.modifier = I915_FORMAT_MOD_X_TILED;
>   					else
>   						data.modifier = DRM_FORMAT_MOD_LINEAR;