Re: [PATCH 0/5] Pushdown bkl from v4l ioctls

2010-05-01 Thread Frederic Weisbecker
On Sat, May 01, 2010 at 11:55:37AM +0200, Hans Verkuil wrote: > On Thursday 29 April 2010 09:10:42 Laurent Pinchart wrote: > > Hi Hans, > > > > On Thursday 29 April 2010 08:44:29 Hans Verkuil wrote: > > > On Thursday 29 April 2010 05:42:39 Frederi

[PATCH 2/5] v4l: Use video_ioctl2_unlocked from drivers that don't want the bkl

2010-04-28 Thread Frederic Weisbecker
There are three drivers that use video_ioctl2() as an unlocked_ioctl, let them use the new video_ioctl2_unlocked. Signed-off-by: Frederic Weisbecker --- drivers/media/video/davinci/vpfe_capture.c |2 +- drivers/media/video/gspca/gspca.c |2 +- drivers/media/video/hdpvr/hdpvr

[PATCH 4/5] v4l: Pushdown bkl to drivers that implement their own ioctl

2010-04-28 Thread Frederic Weisbecker
These are the last remaining v4l drivers that implement the ioctl callback. Signed-off-by: Frederic Weisbecker --- drivers/media/video/bw-qcam.c| 11 - drivers/media/video/c-qcam.c | 11 - drivers/media/video/cpia.c | 11

[PATCH 3/5] v4l: Change users of video_ioctl2 to use unlocked_ioctl

2010-04-28 Thread Frederic Weisbecker
Now that video_ioctl2() got the bkl pushed down, update its users to use .unlocked_ioctl instead of ioctl. Signed-off-by: Frederic Weisbecker --- drivers/media/common/saa7146_fops.c |2 +- drivers/media/radio/dsbr100.c|2 +- drivers/media/radio/radio

[PATCH 1/5] v4l: Pushdown bkl into video_ioctl2

2010-04-28 Thread Frederic Weisbecker
video_ioctl2 is a generic ioctl helper used by a lot of drivers. Most of them put it as their bkl'ed .ioctl callback, then let's pushdown the bkl inside but also provide an unlocked version for those that use it as an unlocked ioctl. Signed-off-by: Frederic Weisbecker --- drivers/m

[PATCH 0/5] Pushdown bkl from v4l ioctls

2010-04-28 Thread Frederic Weisbecker
struct v4l2_file_operations::ioctl for good. It passed make allyesconfig on sparc. Please tell me what you think. Thanks. Frederic Weisbecker (5): v4l: Pushdown bkl into video_ioctl2 v4l: Use video_ioctl2_unlocked from drivers that don't want the bkl v4l: Change use