Hi Niklas, Thank you for the patch. On Tuesday 14 Mar 2017 19:59:50 Niklas Söderlund wrote: > It's possible to grab frames using only one buffer, this should never > have been set to anything else then 1. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/platform/rcar-vin/rcar-dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/rcar-vin/rcar-dma.c > b/drivers/media/platform/rcar-vin/rcar-dma.c index > 9ccd5ff55e192514..c37f7a2993fb5565 100644 > --- a/drivers/media/platform/rcar-vin/rcar-dma.c > +++ b/drivers/media/platform/rcar-vin/rcar-dma.c > @@ -1183,7 +1183,7 @@ int rvin_dma_probe(struct rvin_dev *vin, int irq) > q->ops = &rvin_qops; > q->mem_ops = &vb2_dma_contig_memops; > q->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_MONOTONIC; > - q->min_buffers_needed = 2; > + q->min_buffers_needed = 1; > q->dev = vin->dev; > > ret = vb2_queue_init(q); -- Regards, Laurent Pinchart
