[media] rcar-vin: select capture mode based on free buffers
authorNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Wed, 24 May 2017 00:15:34 +0000 (21:15 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 6 Jun 2017 17:43:06 +0000 (14:43 -0300)
Instead of selecting single or continuous capture mode based on how many
buffers userspace intends to give us select capture mode based on number
of free buffers we can allocate to hardware when the stream is started.

This change is a prerequisite to enable the driver to switch from
continuous to single capture mode (or the other way around) when the
driver is stalled by userspace not feeding it buffers as fast as it
consumes it.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/rcar-vin/rcar-dma.c

index 11e9799..47d78f6 100644 (file)
@@ -405,7 +405,21 @@ static void rvin_capture_off(struct rvin_dev *vin)
 
 static int rvin_capture_start(struct rvin_dev *vin)
 {
-       int ret;
+       struct rvin_buffer *buf, *node;
+       int bufs, ret;
+
+       /* Count number of free buffers */
+       bufs = 0;
+       list_for_each_entry_safe(buf, node, &vin->buf_list, list)
+               bufs++;
+
+       /* Continuous capture requires more buffers then there are HW slots */
+       vin->continuous = bufs > HW_BUFFER_NUM;
+
+       if (!rvin_fill_hw(vin)) {
+               vin_err(vin, "HW not ready to start, not enough buffers available\n");
+               return -EINVAL;
+       }
 
        rvin_crop_scale_comp(vin);
 
@@ -1062,22 +1076,7 @@ static int rvin_start_streaming(struct vb2_queue *vq, unsigned int count)
        vin->state = RUNNING;
        vin->sequence = 0;
 
-       /* Continuous capture requires more buffers then there are HW slots */
-       vin->continuous = count > HW_BUFFER_NUM;
-
-       /*
-        * This should never happen but if we don't have enough
-        * buffers for HW bail out
-        */
-       if (!rvin_fill_hw(vin)) {
-               vin_err(vin, "HW not ready to start, not enough buffers available\n");
-               ret = -EINVAL;
-               goto out;
-       }
-
        ret = rvin_capture_start(vin);
-out:
-       /* Return all buffers if something went wrong */
        if (ret) {
                return_all_buffers(vin, VB2_BUF_STATE_QUEUED);
                v4l2_subdev_call(sd, video, s_stream, 0);