From: Hans de Goede <hdegoede@redhat.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org
Subject: Re: [RFCv2 PATCH 0/8] Add V4L2_CTRL_FLAG_VOLATILE and change volatile autocluster handling.
Date: Sat, 27 Aug 2011 12:35:47 +0200 [thread overview]
Message-ID: <4E58C883.60406@redhat.com> (raw)
In-Reply-To: <1314360013-9876-1-git-send-email-hverkuil@xs4all.nl>
Looks good ACK series.
Acked-by: Hans de Goede <hdegoede@redhat.com>
On 08/26/2011 02:00 PM, Hans Verkuil wrote:
> This is the second patch for this. The first is here:
>
> http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/36650
>
> This second version changes the pwc code as suggested by Hans de Goede and it
> adds documentation. The v4l2-ctrls.c code has also been improved to avoid
> unnecessary calls to update_from_auto_cluster(). Thanks to Hans de Goede for
> pointing me in the right direction.
>
> If there are no additional comments, then I will make a pull request early next
> week.
>
> This will also be the basis for converting soc-camera to the control framework.
>
> Regards,
>
> Hans
>
prev parent reply other threads:[~2011-08-27 10:33 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-26 12:00 [RFCv2 PATCH 0/8] Add V4L2_CTRL_FLAG_VOLATILE and change volatile autocluster handling Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 1/8] videodev2.h: add V4L2_CTRL_FLAG_VOLATILE Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 2/8] v4l2-ctrls: replace is_volatile with V4L2_CTRL_FLAG_VOLATILE Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 3/8] v4l2-ctrls: implement new volatile autocluster scheme Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 4/8] v4l2-controls.txt: update auto cluster documentation Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 5/8] pwc: switch to the new auto-cluster volatile handling Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 6/8] vivi: add support for VIDIOC_LOG_STATUS Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 7/8] pwc: " Hans Verkuil
2011-08-26 12:00 ` [RFCv2 PATCH 8/8] saa7115: use the new auto cluster support Hans Verkuil
2011-08-27 10:35 ` Hans de Goede [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=4E58C883.60406@redhat.com \
--to=hdegoede@redhat.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
/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.