diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2016-02-05 11:52:00 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2016-02-10 09:27:09 -0200 |
| commit | 4f388a9210e2a2c1cd83a5580b74aa13876dbf14 (patch) | |
| tree | 4c46620aa4a0feeb3aa3b934d557fcf1493a2adc | |
| parent | ac75fe5d8fe4a0bf063be18fb29684405279e79e (diff) | |
| download | linux-4f388a9210e2a2c1cd83a5580b74aa13876dbf14.tar.bz2 | |
[media] media: davinci_vpfe: fix missing unlock on error in vpfe_prepare_pipeline()
Add the missing unlock before return from function
vpfe_prepare_pipeline() in the error handling case.
video->lock is lock/unlock in function vpfe_open(),
and no need to unlock it here, so remove unlock
video->lock.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
| -rw-r--r-- | drivers/staging/media/davinci_vpfe/vpfe_video.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c b/drivers/staging/media/davinci_vpfe/vpfe_video.c index 3ec7e65a3ffa..db49af90217e 100644 --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c @@ -147,7 +147,7 @@ static int vpfe_prepare_pipeline(struct vpfe_video_device *video) mutex_lock(&mdev->graph_mutex); ret = media_entity_graph_walk_init(&graph, entity->graph_obj.mdev); if (ret) { - mutex_unlock(&video->lock); + mutex_unlock(&mdev->graph_mutex); return -ENOMEM; } media_entity_graph_walk_start(&graph, entity); |