Commit 996ce3af authored by Niklas Söderlund's avatar Niklas Söderlund Committed by Mauro Carvalho Chehab

[media] rcar-vin: select capture mode based on free buffers

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: default avatarNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 9e921447
...@@ -405,7 +405,21 @@ static void rvin_capture_off(struct rvin_dev *vin) ...@@ -405,7 +405,21 @@ static void rvin_capture_off(struct rvin_dev *vin)
static int rvin_capture_start(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); rvin_crop_scale_comp(vin);
...@@ -1062,22 +1076,7 @@ static int rvin_start_streaming(struct vb2_queue *vq, unsigned int count) ...@@ -1062,22 +1076,7 @@ static int rvin_start_streaming(struct vb2_queue *vq, unsigned int count)
vin->state = RUNNING; vin->state = RUNNING;
vin->sequence = 0; 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); ret = rvin_capture_start(vin);
out:
/* Return all buffers if something went wrong */
if (ret) { if (ret) {
return_all_buffers(vin, VB2_BUF_STATE_QUEUED); return_all_buffers(vin, VB2_BUF_STATE_QUEUED);
v4l2_subdev_call(sd, video, s_stream, 0); v4l2_subdev_call(sd, video, s_stream, 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment