[PATCH] gl-renderer: Add support for DRM_FORMAT_YUV444 buffers

[Quentin Glidic]

On 25/07/2016 16:41, Emmanuel Gil Peyrot wrote:
> On Mon, Jul 25, 2016 at 12:15:41PM +0200, Matthias Treydte wrote:
>> This uses the existing infrastructure for dealing with planar YUV buffers and only adds the
>> relevant yuv_format_descriptor to the table.
>> ---
>>  libweston/gl-renderer.c | 21 +++++++++++++++++++++
>>  1 file changed, 21 insertions(+)
>>
>> diff --git a/libweston/gl-renderer.c b/libweston/gl-renderer.c
>> index be6b11e..cdb7d14 100644
>> --- a/libweston/gl-renderer.c
>> +++ b/libweston/gl-renderer.c
>> @@ -1596,6 +1596,27 @@ struct yuv_format_descriptor yuv_formats[] = {
>>  			.format = DRM_FORMAT_R8,
>>  			.plane_index = 2
>>  		}}
>> +	}, {
>> +		.format = DRM_FORMAT_YUV444,
>> +		.input_planes = 3,
>> +		.output_planes = 3,
>> +		.texture_type = EGL_TEXTURE_Y_U_V_WL,
>> +		{{
>> +			.width_divisor = 1,
>> +			.height_divisor = 1,
>> +			.format = DRM_FORMAT_R8,
>> +			.plane_index = 0
>> +		}, {
>> +			.width_divisor = 1,
>> +			.height_divisor = 1,
>> +			.format = DRM_FORMAT_R8,
>> +			.plane_index = 1
>> +		}, {
>> +			.width_divisor = 1,
>> +			.height_divisor = 1,
>> +			.format = DRM_FORMAT_R8,
>> +			.plane_index = 2
>> +		}}
>>  	}
>>  };
>>
>> --
>> 2.9.0
>
> Hi, thanks for this patch, I originally only implemented very few
> formats just to demonstrate the capabilities of linux_dmabuf but it’s
> always useful to have other formats supported.
>
> Which program did you test it with?  Have you tested it with
> weston-simple-dmabuf-v4l alongside the vivid kernel module in 4:4:4
> mode?
>
> Anyway, the code is
> Reviewed-by: Emmanuel Gil Peyrot <emmanuel.peyrot at collabora.com>

Thanks, pushed:
eaf5841..aca3ffb  master -> master

Cheers,

-- 

Quentin “Sardem FF7” Glidic