| From a149403360c557cd39c94a5ed6b69fa87fd8bc48 Mon Sep 17 00:00:00 2001 |
| From: Hans Verkuil <hans.verkuil@cisco.com> |
| Date: Thu, 6 Feb 2014 05:46:11 -0300 |
| Subject: [media] vb2: rename queued_count to owned_by_drv_count |
| |
| 'queued_count' is a bit vague since it is not clear to which queue it |
| refers to: the vb2 internal list of buffers or the driver-owned list |
| of buffers. |
| |
| Rename to make it explicit. |
| |
| Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> |
| Acked-by: Pawel Osciak <pawel@osciak.com> |
| Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> |
| Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com> |
| (cherry picked from commit 6ea3b980f058d9dbc79ba88c652d581fa2d00792) |
| Signed-off-by: Simon Horman <horms+renesas@verge.net.au> |
| |
| Conflicts: |
| drivers/media/v4l2-core/videobuf2-core.c |
| --- |
| drivers/media/v4l2-core/videobuf2-core.c | 10 +++++----- |
| include/media/videobuf2-core.h | 4 ++-- |
| 2 files changed, 7 insertions(+), 7 deletions(-) |
| |
| --- a/drivers/media/v4l2-core/videobuf2-core.c |
| +++ b/drivers/media/v4l2-core/videobuf2-core.c |
| @@ -951,7 +951,7 @@ void vb2_buffer_done(struct vb2_buffer * |
| spin_lock_irqsave(&q->done_lock, flags); |
| vb->state = state; |
| list_add_tail(&vb->done_entry, &q->done_list); |
| - atomic_dec(&q->queued_count); |
| + atomic_dec(&q->owned_by_drv_count); |
| spin_unlock_irqrestore(&q->done_lock, flags); |
| |
| /* Inform any processes that may be waiting for buffers */ |
| @@ -1240,7 +1240,7 @@ static void __enqueue_in_driver(struct v |
| unsigned int plane; |
| |
| vb->state = VB2_BUF_STATE_ACTIVE; |
| - atomic_inc(&q->queued_count); |
| + atomic_inc(&q->owned_by_drv_count); |
| |
| /* sync buffers */ |
| for (plane = 0; plane < vb->num_planes; ++plane) |
| @@ -1394,7 +1394,7 @@ static int vb2_start_streaming(struct vb |
| int ret; |
| |
| /* Tell the driver to start streaming */ |
| - ret = call_qop(q, start_streaming, q, atomic_read(&q->queued_count)); |
| + ret = call_qop(q, start_streaming, q, atomic_read(&q->owned_by_drv_count)); |
| |
| /* |
| * If there are not enough buffers queued to start streaming, then |
| @@ -1618,7 +1618,7 @@ int vb2_wait_for_all_buffers(struct vb2_ |
| } |
| |
| if (!q->retry_start_streaming) |
| - wait_event(q->done_wq, !atomic_read(&q->queued_count)); |
| + wait_event(q->done_wq, !atomic_read(&q->owned_by_drv_count)); |
| return 0; |
| } |
| EXPORT_SYMBOL_GPL(vb2_wait_for_all_buffers); |
| @@ -1754,7 +1754,7 @@ static void __vb2_queue_cancel(struct vb |
| * has not already dequeued before initiating cancel. |
| */ |
| INIT_LIST_HEAD(&q->done_list); |
| - atomic_set(&q->queued_count, 0); |
| + atomic_set(&q->owned_by_drv_count, 0); |
| wake_up_all(&q->done_wq); |
| |
| /* |
| --- a/include/media/videobuf2-core.h |
| +++ b/include/media/videobuf2-core.h |
| @@ -320,7 +320,7 @@ struct v4l2_fh; |
| * @bufs: videobuf buffer structures |
| * @num_buffers: number of allocated/used buffers |
| * @queued_list: list of buffers currently queued from userspace |
| - * @queued_count: number of buffers owned by the driver |
| + * @owned_by_drv_count: number of buffers owned by the driver |
| * @done_list: list of buffers ready to be dequeued to userspace |
| * @done_lock: lock to protect done_list list |
| * @done_wq: waitqueue for processes waiting for buffers ready to be dequeued |
| @@ -355,7 +355,7 @@ struct vb2_queue { |
| |
| struct list_head queued_list; |
| |
| - atomic_t queued_count; |
| + atomic_t owned_by_drv_count; |
| struct list_head done_list; |
| spinlock_t done_lock; |
| wait_queue_head_t done_wq; |