From: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org, Pete Eberlein <pete@sensoray.com>,
Hans Verkuil <hans.verkuil@cisco.com>
Subject: Re: [REVIEW PATCH 01/11] s2255: convert to the control framework.
Date: Wed, 27 Feb 2013 23:10:49 +0100 [thread overview]
Message-ID: <512E8469.7000404@gmail.com> (raw)
In-Reply-To: <201302271005.17713.hverkuil@xs4all.nl>
On 02/27/2013 10:05 AM, Hans Verkuil wrote:
> On Tue 26 February 2013 23:21:19 Sylwester Nawrocki wrote:
>> On 02/26/2013 06:35 PM, Hans Verkuil wrote:
[...]
> Private controls always had overlapping IDs. During one of the mini-summits
> last year we decided to change that so they all had their own ID. The meye
> driver is one of the first to have a proper range defined, eventually all
> other drivers that have private controls will be added there. That includes
> those you found with grep.
>
> So give me time and it will all be fixed :-)
Ah, that's the whole plan then! It indeed makes sense. I just thought
only new
drivers would use fixed not overlapping control IDs. But having the existing
ones fixed as well sounds a way better.
Regards,
Sylwester
prev parent reply other threads:[~2013-02-27 22:10 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-26 17:35 s2255: v4l2 compliance fixes Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 01/11] s2255: convert to the control framework Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 02/11] s2255: add V4L2_CID_JPEG_COMPRESSION_QUALITY Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 03/11] s2255: add support for control events and prio handling Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 04/11] s2255: add device_caps support to querycap Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 05/11] s2255: fixes in the way standards are handled Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 06/11] s2255: zero priv and set colorspace Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 07/11] s2255: fix field handling Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 08/11] s2255: don't zero struct v4l2_streamparm Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 09/11] s2255: Add ENUM_FRAMESIZES support Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 10/11] s2255: choose YUYV as the default format, not YUV422P Hans Verkuil
2013-02-26 17:35 ` [REVIEW PATCH 11/11] s2255: fix big-endian support Hans Verkuil
2013-02-26 22:21 ` [REVIEW PATCH 01/11] s2255: convert to the control framework Sylwester Nawrocki
2013-02-27 9:05 ` Hans Verkuil
2013-02-27 22:10 ` Sylwester Nawrocki [this message]
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=512E8469.7000404@gmail.com \
--to=sylvester.nawrocki@gmail.com \
--cc=hans.verkuil@cisco.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
--cc=pete@sensoray.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).