staging: bcm2835-codec: Queue flushed buffers instead of completing
authorDave Stevenson <dave.stevenson@raspberrypi.com>
Thu, 16 Sep 2021 15:46:58 +0000 (16:46 +0100)
committerPhil Elwell <8911409+pelwell@users.noreply.github.com>
Thu, 23 Sep 2021 13:53:43 +0000 (14:53 +0100)
When a buffer is returned on a port that is disabled, return it
to the videobuf2 QUEUED state instead of DONE which returns it
to the client.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/staging/vc04_services/bcm2835-codec/bcm2835-v4l2-codec.c

index 6c6e0ba..03c9ccb 100644 (file)
@@ -926,7 +926,9 @@ static void ip_buffer_cb(struct vchiq_mmal_instance *instance,
 
        v4l2_dbg(3, debug, &ctx->dev->v4l2_dev, "%s: no error. Return buffer %p\n",
                 __func__, &buf->m2m.vb.vb2_buf);
-       vb2_buffer_done(&buf->m2m.vb.vb2_buf, VB2_BUF_STATE_DONE);
+       vb2_buffer_done(&buf->m2m.vb.vb2_buf,
+                       port->enabled ? VB2_BUF_STATE_DONE :
+                                       VB2_BUF_STATE_QUEUED);
 
        ctx->num_ip_buffers++;
        v4l2_dbg(2, debug, &ctx->dev->v4l2_dev, "%s: done %d input buffers\n",