From: Hans Verkuil <hverkuil@xs4all.nl>
To: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Antti Palosaari <crope@iki.fi>,
Guennadi Liakhovetski <guennadi.liakhovetski@intel.com>,
Helen Mae Koike Fornazier <helen.koike@collabora.co.uk>,
Philipp Zabel <p.zabel@pengutronix.de>,
Shuah Khan <shuahkh@osg.samsung.com>,
linux-media <linux-media@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 2/6] [media] Documentation: Add HSV format
Date: Sat, 16 Jul 2016 18:14:37 +0200 [thread overview]
Message-ID: <61de90a6-812d-08d5-919c-669ff33cc836@xs4all.nl> (raw)
In-Reply-To: <CAPybu_3=j99ThtMGbecNRUjTTNOM1f=MoReGgYV05JsjbRauAg@mail.gmail.com>
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@xs4all.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
next prev parent reply other threads:[~2016-07-16 16:14 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-15 16:13 [PATCH v2 0/6] Add HSV format Ricardo Ribalda Delgado
2016-07-15 16:13 ` [PATCH v2 1/6] [media] videodev2.h Add HSV formats Ricardo Ribalda Delgado
2016-07-15 16:13 ` [PATCH v2 2/6] [media] Documentation: Add HSV format Ricardo Ribalda Delgado
2016-07-15 18:11 ` Laurent Pinchart
2016-07-16 8:19 ` Hans Verkuil
2016-07-16 12:38 ` Laurent Pinchart
2016-07-16 13:59 ` Hans Verkuil
2016-07-16 14:12 ` Laurent Pinchart
2016-07-16 14:32 ` Ricardo Ribalda Delgado
2016-07-16 15:28 ` Hans Verkuil
2016-07-16 15:57 ` Ricardo Ribalda Delgado
2016-07-16 16:14 ` Hans Verkuil [this message]
2016-07-16 15:19 ` Hans Verkuil
2016-07-16 8:22 ` Ricardo Ribalda Delgado
2016-07-15 16:13 ` [PATCH v2 3/6] [media] Documentation: Add Ricardo Ribalda Ricardo Ribalda Delgado
2016-07-15 16:13 ` [PATCH v2 4/6] [media] vivid: code refactor for color representation Ricardo Ribalda Delgado
2016-07-15 17:58 ` Hans Verkuil
2016-07-15 18:12 ` Hans Verkuil
2016-07-15 18:28 ` Hans Verkuil
2016-07-15 16:13 ` [PATCH v2 5/6] [media] vivid: Add support for HSV formats Ricardo Ribalda Delgado
2016-07-15 18:29 ` Hans Verkuil
2016-07-15 16:13 ` [PATCH v2 6/6] [media] vivid: Rename variable Ricardo Ribalda Delgado
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=61de90a6-812d-08d5-919c-669ff33cc836@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=crope@iki.fi \
--cc=guennadi.liakhovetski@intel.com \
--cc=helen.koike@collabora.co.uk \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@osg.samsung.com \
--cc=p.zabel@pengutronix.de \
--cc=ricardo.ribalda@gmail.com \
--cc=sakari.ailus@linux.intel.com \
--cc=shuahkh@osg.samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).