From: Antti Palosaari Date: Wed, 26 Feb 2014 23:48:23 +0000 (-0300) Subject: [media] msi3101: clamp mmap buffers to reasonable level X-Git-Tag: firefly_0821_release~176^2~3573^2~363 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=d287e4e31ed6374f8899c131075461c3c8cbd8a9;p=firefly-linux-kernel-4.4.55.git [media] msi3101: clamp mmap buffers to reasonable level 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 Cc: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/staging/media/msi3101/sdr-msi3101.c b/drivers/staging/media/msi3101/sdr-msi3101.c index 2135940f2d58..93e6ebadafbe 100644 --- a/drivers/staging/media/msi3101/sdr-msi3101.c +++ b/drivers/staging/media/msi3101/sdr-msi3101.c @@ -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