diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-14 03:19:18 -0500 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2014-01-17 03:57:47 -0500 |
commit | 4ac8bd618b6c70fbc043c82f95d2d9b529c57109 (patch) | |
tree | 2884e16ab3e5a66e37a7bf2cfc048e6f3ea52767 | |
parent | c3235bfc0cba3d05d22c37e696b0c03682a5ddc6 (diff) |
video: mmp: delete a stray mutex_unlock()
We aren't holding the disp_lock so we shouldn't release it.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Haojian Zhuang <haojian.zhuang@gmail.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
-rw-r--r-- | drivers/video/mmp/core.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/video/mmp/core.c b/drivers/video/mmp/core.c index 84de2632857a..c8d426541612 100644 --- a/drivers/video/mmp/core.c +++ b/drivers/video/mmp/core.c | |||
@@ -173,7 +173,7 @@ struct mmp_path *mmp_register_path(struct mmp_path_info *info) | |||
173 | + sizeof(struct mmp_overlay) * info->overlay_num; | 173 | + sizeof(struct mmp_overlay) * info->overlay_num; |
174 | path = kzalloc(size, GFP_KERNEL); | 174 | path = kzalloc(size, GFP_KERNEL); |
175 | if (!path) | 175 | if (!path) |
176 | goto failed; | 176 | return NULL; |
177 | 177 | ||
178 | /* path set */ | 178 | /* path set */ |
179 | mutex_init(&path->access_ok); | 179 | mutex_init(&path->access_ok); |
@@ -219,11 +219,6 @@ struct mmp_path *mmp_register_path(struct mmp_path_info *info) | |||
219 | 219 | ||
220 | mutex_unlock(&disp_lock); | 220 | mutex_unlock(&disp_lock); |
221 | return path; | 221 | return path; |
222 | |||
223 | failed: | ||
224 | kfree(path); | ||
225 | mutex_unlock(&disp_lock); | ||
226 | return NULL; | ||
227 | } | 222 | } |
228 | EXPORT_SYMBOL_GPL(mmp_register_path); | 223 | EXPORT_SYMBOL_GPL(mmp_register_path); |
229 | 224 | ||