Re: [PATCH v3 12/12] tests/kms_colorop: Increase VKMS bracket to 3 up

Alex Hung <[email protected]> Sat, 25 Jul 2026 01:24:33 -0600
Newsgroups org.freedesktop.lists.igt-dev
Message-ID <[email protected]>
Reviewed-by: Alex Hung <[email protected]>

On 6/23/26 11:57, Harry Wentland wrote:
> The delta is a bit wider for tests of the CSC with other colorops.
> 
> Increase it to 3, which is still pretty tight.
> 
> Signed-off-by: Harry Wentland <[email protected]>
> ---
>   tests/kms_colorop.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/kms_colorop.c b/tests/kms_colorop.c
> index a2784243c0f5..393c66de51ed 100644
> --- a/tests/kms_colorop.c
> +++ b/tests/kms_colorop.c
> @@ -234,7 +234,7 @@ static bool compare_with_bracket(igt_fb_t *in, igt_fb_t *out)
>   {
>   	/* Each driver is expected to have its own bracket, i.e., by trial and error */
>   	if (is_vkms_device(in->fd))
> -		return igt_cmp_fb_pixels(in, out, 1, 1);
> +		return igt_cmp_fb_pixels(in, out, 3, 1);
>   
>   	if (is_amdgpu_device(in->fd))
>   		return igt_cmp_fb_pixels(in, out, 13, 13);