[igt-dev] [PATCH i-g-t v4 3/4] tests/kms_color_chamelium: add subtests to validate color

[Petri Latvala]

On Thu, Jan 23, 2020 at 01:30:41PM +0530, Kunal Joshi wrote:
> To validate color subtests using chamelium, subtests modified
> to do frame dump comparison instead of crc comparison.
> Tests require chamelium and will validate color features
> at pipe level.
> 
> (v2)
>         Comparing framedump with framebuffer reference instead of
>         comparing two framedump.
> (v3)
>         Moved common functions with kms_color to lib/igt_color
> (v4)
>         Added changes to skip the tests if there is no connected port with
>         chamelium.
>         Rearranged position of valid_output.
> 
> Signed-off-by: Kunal Joshi <kunal1.joshi at intel.com>
> Signed-off-by: Swati Sharma <swati2.sharma at intel.com>
> Suggested-by: Uma Shankar <uma.shankar at intel.com>
> ---
>  tests/kms_color_chamelium.c | 759 ++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 759 insertions(+)
>  create mode 100644 tests/kms_color_chamelium.c
> 
> diff --git a/tests/kms_color_chamelium.c b/tests/kms_color_chamelium.c
> new file mode 100644
> index 0000000..b6b6731
> --- /dev/null
> +++ b/tests/kms_color_chamelium.c
> @@ -0,0 +1,759 @@
> +/*
> + * Copyright © 2020 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include "kms_color.h"
> +
> +IGT_TEST_DESCRIPTION("Test Color Features at Pipe level");

" using chamelium to verify instead of CRC"


Otherwise looks like what it claims, a "chamelium port" of
kms_color. Whoever feels like it gets to do even more refactoring to
get the two tests to share more code, a job for later.

With the description augmented,

Reviewed-by: Petri Latvala <petri.latvala at intel.com>