diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2014-03-04 07:27:13 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-03-11 06:56:38 -0300 |
| commit | 0647064293d745720fc62e2edc7734fa8af06adf (patch) | |
| tree | b8e1092039c7a3a10ef3102b8bf1ee4fc028f75e /drivers/media/v4l2-core | |
| parent | b5b4541eef8eac83f5c0d166d8e494f7c9fff202 (diff) | |
| download | op-kernel-dev-0647064293d745720fc62e2edc7734fa8af06adf.zip op-kernel-dev-0647064293d745720fc62e2edc7734fa8af06adf.tar.gz | |
[media] vb2: change result code of buf_finish to void
The buf_finish op should always work, so change the return type to void.
Update the few drivers that use it.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Pawel Osciak <pawel@osciak.com>
Reviewed-by: Pawel Osciak <pawel@osciak.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Diffstat (limited to 'drivers/media/v4l2-core')
| -rw-r--r-- | drivers/media/v4l2-core/videobuf2-core.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 917b1cb..2be3cfe 100644 --- a/drivers/media/v4l2-core/videobuf2-core.c +++ b/drivers/media/v4l2-core/videobuf2-core.c @@ -1834,11 +1834,7 @@ static int vb2_internal_dqbuf(struct vb2_queue *q, struct v4l2_buffer *b, bool n if (ret < 0) return ret; - ret = call_vb_qop(vb, buf_finish, vb); - if (ret) { - dprintk(1, "dqbuf: buffer finish failed\n"); - return ret; - } + call_vb_qop(vb, buf_finish, vb); switch (vb->state) { case VB2_BUF_STATE_DONE: |
