[media] vivi: queue_setup improvements
authorHans Verkuil <hans.verkuil@cisco.com>
Mon, 10 Feb 2014 11:08:47 +0000 (08:08 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Mon, 24 Feb 2014 13:59:15 +0000 (10:59 -0300)
Drop the vid_limit module option: there is really no need to limit this.
No other driver does that. If you try to allocate more buffers then vb2
will automatically reduce the number of buffers anyway.

Also add sanity checks if the size in the fmt argument is going to be
used and drop the code that checks against *nbuffers == 0: this can
never happen (the vb2 framework ensures that) and the code was wrong
anyway since *nbuffers should have been set to the minimum number of
required buffers which is 1 for this driver.

Since vivi is often used as a template driver it is good to have this
driver be as compliant as possible. This broken code was for example
copied to the s2255 driver (which is being fixed as well).

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/platform/vivi.c

index 4114bb624b13e514f1856d2a0d607493795c9a5f..e9cd96ecf4d0b9b5d7c04aa2a79d7f6eb35bc2c3 100644 (file)
@@ -70,10 +70,6 @@ static unsigned debug;
 module_param(debug, uint, 0644);
 MODULE_PARM_DESC(debug, "activates debug info");
 
-static unsigned int vid_limit = 16;
-module_param(vid_limit, uint, 0644);
-MODULE_PARM_DESC(vid_limit, "capture memory limit in megabytes");
-
 /* Global font descriptor */
 static const u8 *font8x16;
 
@@ -816,19 +812,15 @@ static int queue_setup(struct vb2_queue *vq, const struct v4l2_format *fmt,
        struct vivi_dev *dev = vb2_get_drv_priv(vq);
        unsigned long size;
 
-       if (fmt)
+       size = dev->width * dev->height * dev->pixelsize;
+       if (fmt) {
+               if (fmt->fmt.pix.sizeimage < size)
+                       return -EINVAL;
                size = fmt->fmt.pix.sizeimage;
-       else
-               size = dev->width * dev->height * dev->pixelsize;
-
-       if (size == 0)
-               return -EINVAL;
-
-       if (0 == *nbuffers)
-               *nbuffers = 32;
-
-       while (size * *nbuffers > vid_limit * 1024 * 1024)
-               (*nbuffers)--;
+               /* check against insane over 8K resolution buffers */
+               if (size > 7680 * 4320 * dev->pixelsize)
+                       return -EINVAL;
+       }
 
        *nplanes = 1;