[PATCH] [media] omap3isp: fix dqbuf description comment

[PATCH] [media] omap3isp: fix dqbuf description comment

[Michael Jones]

Signed-off-by: Michael Jones <michael.jones@matrix-vision.de>
---
 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.

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)
-- 
1.7.4.1


MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler
Registergericht: Amtsgericht Stuttgart, HRB 271090
Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier

Re: [PATCH] [media] omap3isp: fix dqbuf description comment

[Laurent Pinchart]

Hi Michael,

Thanks for the patch.

On Wednesday 27 June 2012 17:06:57 Michael Jones wrote:
> Signed-off-by: Michael Jones <michael.jones@matrix-vision.de>
> ---
>  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