This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: v4l: sh_mobile_ceu: Return buffers on streamoff()
Author:  Jacopo Mondi <[email protected]>
Date:    Wed Nov 15 12:59:12 2017 -0500

videobuf2 core reports an error when not all buffers have been returned
to the framework:

drivers/media/v4l2-core/videobuf2-core.c:1651
WARN_ON(atomic_read(&q->owned_by_drv_count))

Fix this returning all buffers currently in capture queue.

Signed-off-by: Jacopo Mondi <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c 
b/drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c
index 36762ec954e7..9180a1d96acb 100644
--- a/drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c
+++ b/drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c
@@ -451,13 +451,18 @@ static void sh_mobile_ceu_stop_streaming(struct vb2_queue 
*q)
        struct soc_camera_host *ici = to_soc_camera_host(icd->parent);
        struct sh_mobile_ceu_dev *pcdev = ici->priv;
        struct list_head *buf_head, *tmp;
+       struct vb2_v4l2_buffer *vbuf;
 
        spin_lock_irq(&pcdev->lock);
 
        pcdev->active = NULL;
 
-       list_for_each_safe(buf_head, tmp, &pcdev->capture)
+       list_for_each_safe(buf_head, tmp, &pcdev->capture) {
+               vbuf = &list_entry(buf_head, struct sh_mobile_ceu_buffer,
+                                  queue)->vb;
+               vb2_buffer_done(&vbuf->vb2_buf, VB2_BUF_STATE_DONE);
                list_del_init(buf_head);
+       }
 
        spin_unlock_irq(&pcdev->lock);
 

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to