Only the "streamer" process has to initialise videobufs.

Signed-off-by: Guennadi Liakhovetski <g.liakhovet...@gmx.de>
---

Janusz, this and the following patch can be useful, if we decide to 
implement dynamic switching between videobuf implementations. Only 
compile-tested...

 drivers/media/video/soc_camera.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/video/soc_camera.c b/drivers/media/video/soc_camera.c
index a499cac..d90386c 100644
--- a/drivers/media/video/soc_camera.c
+++ b/drivers/media/video/soc_camera.c
@@ -158,6 +158,8 @@ static int soc_camera_reqbufs(struct file *file, void *priv,
 
        WARN_ON(priv != file->private_data);
 
+       ici->ops->init_videobuf(&icf->vb_vidq, icd);
+
        ret = videobuf_reqbufs(&icf->vb_vidq, p);
        if (ret < 0)
                return ret;
@@ -409,8 +411,6 @@ static int soc_camera_open(struct file *file)
        file->private_data = icf;
        dev_dbg(&icd->dev, "camera device open\n");
 
-       ici->ops->init_videobuf(&icf->vb_vidq, icd);
-
        mutex_unlock(&icd->video_lock);
 
        return 0;
-- 
1.7.2

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to