[Mesa-dev] [PATCH 01/11] gallium: add P016 format

Christian König deathsimple at vodafone.de
Mon Mar 13 11:47:00 UTC 2017


Am 13.03.2017 um 11:33 schrieb Emil Velikov:
> Hi Christian,
>
> On 8 March 2017 at 12:29, Christian König <deathsimple at vodafone.de> wrote:
>> From: Christian König <christian.koenig at amd.com>
>>
>> Same layout as NV12, but 16bit per channel instead of 8.
>> --- a/src/gallium/include/pipe/p_format.h
>> +++ b/src/gallium/include/pipe/p_format.h
>> @@ -389,6 +389,8 @@ enum pipe_format {
>>      PIPE_FORMAT_ASTC_12x10_SRGB         = 305,
>>      PIPE_FORMAT_ASTC_12x12_SRGB         = 306,
>>
>> +   PIPE_FORMAT_P016                    = 307,
>> +
> This piece seems to be breaking svga [other?], with the gist as below.
> Can you take a quick look please ?

Sorry for the inconvenience. Patch just handling the format as invalid 
is on the mailing list.

Anything else broken by this?

Please review,
Christian.

>
> ../../../../../src/gallium/drivers/svga/svga_format.c: In function
> 'check_format_tables':
> ../../../../../src/gallium/drivers/svga/svga_format.c:1842:7: error:
> size of unnamed array is negative
>         STATIC_ASSERT(ARRAY_SIZE(format_conversion_table) == PIPE_FORMAT_COUNT);
>
> -Emil




More information about the mesa-dev mailing list