[media] msi3101: clamp mmap buffers to reasonable level
authorAntti Palosaari <crope@iki.fi>
Wed, 26 Feb 2014 23:48:23 +0000 (20:48 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 13 Mar 2014 13:26:53 +0000 (10:26 -0300)
That value is coming from the user and we need only ensure it is
reasonable. That was pointed by Hans when reviewing rtl2832_sdr driver.

Signed-off-by: Antti Palosaari <crope@iki.fi>
Cc: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/staging/media/msi3101/sdr-msi3101.c

index 2135940f2d584f461dc9f10ab681f4571ec83aa8..93e6ebadafbe932cca3e65ce025e8e5dbc36d691 100644 (file)
@@ -831,7 +831,7 @@ static int msi3101_queue_setup(struct vb2_queue *vq,
        dev_dbg(&s->udev->dev, "%s: *nbuffers=%d\n", __func__, *nbuffers);
 
        /* Absolute min and max number of buffers available for mmap() */
-       *nbuffers = 32;
+       *nbuffers = clamp_t(unsigned int, *nbuffers, 8, 32);
        *nplanes = 1;
        /*
         *   3, wMaxPacketSize 3x 1024 bytes