Re: [igt-dev] [PATCH i-g-t v2] Add an api to support the CTA range color square video test pattern as explained in section 3.2.5.3 of the DP CTS specification. This pattern is required for supporting the CTA range for RGB formats.

[Abhinav Kumar <quic_abhinavk@quicinc.com>]

Hi Petri

On 2/21/2022 2:13 AM, Petri Latvala wrote:
> On Wed, Feb 16, 2022 at 03:03:42PM -0800, Abhinav Kumar wrote:
>> From: Maitreyee Rao <maitreye@codeaurora.org>
>>
>> Also rename the existing api igt_fill_cts_framebuffer to
>> igt_fill_cts_color_ramp_framebuffer to higlight the
>> pattern type.
>>
>> changes in v2:
>> 	- removed redundant pointers
>> 	- fixed overall formatting issues
>> 	- Fixed author name
>>
>> Signed-off-by: Maitreyee Rao <maitreye@codeaurora.org>
>> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
> 
> 
> ../lib/igt_fb.c: In function ‘igt_fill_cts_color_square_framebuffer’:
> ../lib/igt_fb.c:1515:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
>   1515 |  uint32_t colors[8][3] = {
>        |  ^~~~~~~~
> ../lib/igt_fb.c:1555:19: warning: initialization of ‘uint8_t *’ {aka ‘unsigned char *’} from incompatible pointer type ‘uint32_t *’ {aka ‘unsigned int *’} [-Wincompatible-pointer-types]
>   1555 |   uint8_t *temp = pixmap;
>        |                   ^~~~~~
> 
> 
> 

I have addressed these and pushed 
https://patchwork.freedesktop.org/patch/476306/.

Somehow the chrome compiler didnt catch these.

Thanks

Abhinav