Re: [PATCH i-g-t 3/4] tests/kms_color_pipeline: Disable pipe color properties in test setup

["Sharma, Swati2" <swati2.sharma@intel.com>]

Hi Chaitanya,

Patch LGTM

Reviewed-by: Swati Sharma <swati2.sharma@intel.com>

On 02-02-2026 11:34 am, Chaitanya Kumar Borah wrote:
> Move disabling of pipe-level color properties (CTM, degamma, gamma)
> from the individual plane color tests into the common test setup.
>
> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
> ---
>   tests/kms_color_pipeline.c | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/tests/kms_color_pipeline.c b/tests/kms_color_pipeline.c
> index dd2eed637..e15f4d79a 100644
> --- a/tests/kms_color_pipeline.c
> +++ b/tests/kms_color_pipeline.c
> @@ -65,6 +65,11 @@ static void test_setup(data_t *data, enum pipe p)
>   	data->pipe_crc = igt_crtc_crc_new(crtc,
>   					  IGT_PIPE_CRC_SOURCE_AUTO);
>   
> +	/* Disable Pipe color props. */
> +	disable_ctm(crtc);
> +	disable_degamma(crtc);
> +	disable_gamma(crtc);
> +
>   	igt_display_commit_atomic(&data->display, DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>   }
>   
> @@ -107,10 +112,6 @@ static void _test_plane_colorops(data_t *data, enum pipe pipe,
>   				 &fb));
>   	igt_plane_set_fb(plane, &fb);
>   
> -	/* Disable Pipe color props. */
> -	disable_ctm(plane->crtc);
> -	disable_degamma(plane->crtc);
> -	disable_gamma(plane->crtc);
>   	igt_display_commit_atomic(&data->display, 0, NULL);
>   
>   	/* Reference (software-equivalent) CRC */