Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

From: Hans Verkuil
Date: Sat Jul 16 2016 - 12:14:48 EST


On 07/16/2016 05:57 PM, Ricardo Ribalda Delgado wrote:
> Hi Hans
>
> On Sat, Jul 16, 2016 at 5:28 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote:
>
>>
>>> +
>>> +enum v4l2_rgb_encoding {
>>> + V4L2_RGB_ENC_FULL = 32,
>>> + V4L2_HSV_ENC_16_235 = 33,
>>> +};
>>
>> No.
>
> I was trying to fit also Laurent special 16-235 RGB format. I will
> remove it on future versions.
>
>
> Can I make this change as 2 new patches on my vivid-hsv patchset?
>
> 1) Add hsv_encoding
> 2) Add support for vivid hsv_encoding

Yes. I prefer them last in the series, since I am not quite 100% certain yet.

Regards,

Hans