From: Frederic Weisbecker <fweisbec@gmail.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Mauro Carvalho Chehab <mchehab@infradead.org>,
LKML <linux-kernel@vger.kernel.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Arnd Bergmann <arnd@arndb.de>, John Kacur <jkacur@redhat.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
Jan Blunck <jblunck@gmail.com>,
Thomas Gleixner <tglx@linutronix.de>,
Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: [PATCH 0/4 v2] V4l bkl ioctl pushdown
Date: Wed, 19 May 2010 12:55:21 +0200 [thread overview]
Message-ID: <20100519105519.GA1213@nowhere> (raw)
In-Reply-To: <201005180850.27813.hverkuil@xs4all.nl>
On Tue, May 18, 2010 at 08:50:27AM +0200, Hans Verkuil wrote:
> On Monday 17 May 2010 08:50:48 Frederic Weisbecker wrote:
> > Hi,
> >
> > This is the second attempt to pushdown the bkl in v4l fops ioctl
> > handlers.
> >
> > Amongst the changes:
> >
> > - keep video_ioctl2() unlocked but bring a new video_ioctl2_bkl()
> > that drivers can use and attribute to their .unlocked_ioctl instead.
> >
> > - two new (or it I missed them last time) drivers have been included
> > to the pushdown.
> >
> > This is against a fresh v4l/master.
> >
> > Thanks.
>
> Other than the single comment about the vivi driver in patch 3/4:
>
> Acked-by: Hans Verkuil <hverkuil@xs4all.nl>
>
> It looks good to me and it would be nice if this could be merged for 2.6.35.
>
> Regards,
>
> Hans
Thanks I'm fixing the issue with the vivi driver and will resend the set
with your acks.
prev parent reply other threads:[~2010-05-19 10:55 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-17 6:50 [PATCH 0/4 v2] V4l bkl ioctl pushdown Frederic Weisbecker
2010-05-17 6:50 ` [PATCH 1/4] v4l: Introduce video_ioctl2_bkl Frederic Weisbecker
2010-05-17 6:50 ` [PATCH 2/4] v4l: Convert video_ioctl2 to video_ioctl2_bkl for users that need the bkl Frederic Weisbecker
2010-05-17 6:50 ` [PATCH 3/4] v4l: Pushdown bkl to drivers that implement their own ioctl Frederic Weisbecker
2010-05-18 6:47 ` Hans Verkuil
2010-05-17 6:50 ` [PATCH 4/4] v4l: Remove struct v4l2_file_operations::ioctl Frederic Weisbecker
2010-05-18 6:50 ` [PATCH 0/4 v2] V4l bkl ioctl pushdown Hans Verkuil
2010-05-19 10:55 ` Frederic Weisbecker [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=20100519105519.GA1213@nowhere \
--to=fweisbec@gmail.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=arnd@arndb.de \
--cc=hverkuil@xs4all.nl \
--cc=jblunck@gmail.com \
--cc=jkacur@redhat.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mchehab@infradead.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.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.