diff options
author | Seung-Woo Kim <sw0312.kim@samsung.com> | 2013-11-29 02:50:29 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2013-12-09 08:42:16 -0500 |
commit | 77c0782e9729cee9d1d67cd1343ed7c3f947a36a (patch) | |
tree | ae8b39bda57a5fc3f27aab79739658c8f3aa4d29 | |
parent | f956035ce70b8da9928aac1424485c3526fccb11 (diff) |
[media] videobuf2: Add log for size checking error in __qbuf_dmabuf
__qbuf_dmabuf checks whether size of provided dmabuf is large
enough, and it returns error without any log. So this patch adds
error log in the case.
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r-- | drivers/media/v4l2-core/videobuf2-core.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index e949a0e64be9..144caf12fb6c 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c | |||
@@ -1116,6 +1116,8 @@ static int __qbuf_dmabuf(struct vb2_buffer *vb, const struct v4l2_buffer *b) | |||
1116 | 1116 | ||
1117 | if (planes[plane].length < planes[plane].data_offset + | 1117 | if (planes[plane].length < planes[plane].data_offset + |
1118 | q->plane_sizes[plane]) { | 1118 | q->plane_sizes[plane]) { |
1119 | dprintk(1, "qbuf: invalid dmabuf length for plane %d\n", | ||
1120 | plane); | ||
1119 | ret = -EINVAL; | 1121 | ret = -EINVAL; |
1120 | goto err; | 1122 | goto err; |
1121 | } | 1123 | } |