Hi Michael, Thanks for the patch. On Wednesday 27 June 2012 17:06:57 Michael Jones wrote: > Signed-off-by: Michael Jones <michael.jones@xxxxxxxxxxxxxxxx> > --- > drivers/media/video/omap3isp/ispqueue.c | 9 ++------- > 1 files changed, 2 insertions(+), 7 deletions(-) > > This comment looks like it was a copy-paste from the description of qbuf. You're right. I've applied your patch to my tree, with a slightly modified comment. > diff --git a/drivers/media/video/omap3isp/ispqueue.c > b/drivers/media/video/omap3isp/ispqueue.c index 5fda5d0..23915ce 100644 > --- a/drivers/media/video/omap3isp/ispqueue.c > +++ b/drivers/media/video/omap3isp/ispqueue.c > @@ -908,13 +908,8 @@ done: > * > * This function is intended to be used as a VIDIOC_DQBUF ioctl handler. > * > - * The v4l2_buffer structure passed from userspace is first sanity tested. > If > - * sane, the buffer is then processed and added to the main queue and, if > the > - * queue is streaming, to the IRQ queue. > - * > - * Before being enqueued, USERPTR buffers are checked for address changes. > If > - * the buffer has a different userspace address, the old memory area is > unlocked > - * and the new memory area is locked. > + * if nonblocking=1, returns -EAGAIN if no buffer is available. > + * if nonblocking=0, waits on IRQ queue until a buffer becomes available. > */ > int omap3isp_video_queue_dqbuf(struct isp_video_queue *queue, > struct v4l2_buffer *vbuf, int nonblocking) -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html