Only the "streamer" process has to initialise videobufs. Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> --- 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
