[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <61de90a6-812d-08d5-919c-669ff33cc836@xs4all.nl>
Date: Sat, 16 Jul 2016 18:14:37 +0200
From: Hans Verkuil <hverkuil@...all.nl>
To: Ricardo Ribalda Delgado <ricardo.ribalda@...il.com>
Cc: Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Mauro Carvalho Chehab <mchehab@....samsung.com>,
Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Antti Palosaari <crope@....fi>,
Guennadi Liakhovetski <guennadi.liakhovetski@...el.com>,
Helen Mae Koike Fornazier <helen.koike@...labora.co.uk>,
Philipp Zabel <p.zabel@...gutronix.de>,
Shuah Khan <shuahkh@....samsung.com>,
linux-media <linux-media@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 2/6] [media] Documentation: Add HSV format
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@...all.nl> 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
Powered by blists - more mailing lists