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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.