diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2014-03-04 11:27:13 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-03-11 10:56:38 +0100 |
| commit | 0647064293d745720fc62e2edc7734fa8af06adf (patch) | |
| tree | b8e1092039c7a3a10ef3102b8bf1ee4fc028f75e /drivers/media/parport/bw-qcam.c | |
| parent | [media] vb2: add debugging code to check for unbalanced ops (diff) | |
| download | linux-0647064293d745720fc62e2edc7734fa8af06adf.tar.xz linux-0647064293d745720fc62e2edc7734fa8af06adf.zip | |
[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/parport/bw-qcam.c')
| -rw-r--r-- | drivers/media/parport/bw-qcam.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/parport/bw-qcam.c b/drivers/media/parport/bw-qcam.c index a0a6ee6398fe..cf2db63da3b1 100644 --- a/drivers/media/parport/bw-qcam.c +++ b/drivers/media/parport/bw-qcam.c @@ -667,7 +667,7 @@ static void buffer_queue(struct vb2_buffer *vb) vb2_buffer_done(vb, VB2_BUF_STATE_DONE); } -static int buffer_finish(struct vb2_buffer *vb) +static void buffer_finish(struct vb2_buffer *vb) { struct qcam *qcam = vb2_get_drv_priv(vb->vb2_queue); void *vbuf = vb2_plane_vaddr(vb, 0); @@ -691,7 +691,6 @@ static int buffer_finish(struct vb2_buffer *vb) if (len != size) vb->state = VB2_BUF_STATE_ERROR; vb2_set_plane_payload(vb, 0, len); - return 0; } static struct vb2_ops qcam_video_qops = { |