diff options
| author | Antti Palosaari <crope@iki.fi> | 2014-02-26 18:48:23 -0500 |
|---|---|---|
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-03-13 09:26:53 -0400 |
| commit | d287e4e31ed6374f8899c131075461c3c8cbd8a9 (patch) | |
| tree | d2352b472471fd7a5983567f643766946777c569 | |
| parent | 2c57213fe043d0ed9c8f4709161ac89316e6250a (diff) | |
[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 <crope@iki.fi>
Cc: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
| -rw-r--r-- | drivers/staging/media/msi3101/sdr-msi3101.c | 2 |
1 files changed, 1 insertions, 1 deletions
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, | |||
| 831 | dev_dbg(&s->udev->dev, "%s: *nbuffers=%d\n", __func__, *nbuffers); | 831 | dev_dbg(&s->udev->dev, "%s: *nbuffers=%d\n", __func__, *nbuffers); |
| 832 | 832 | ||
| 833 | /* Absolute min and max number of buffers available for mmap() */ | 833 | /* Absolute min and max number of buffers available for mmap() */ |
| 834 | *nbuffers = 32; | 834 | *nbuffers = clamp_t(unsigned int, *nbuffers, 8, 32); |
| 835 | *nplanes = 1; | 835 | *nplanes = 1; |
| 836 | /* | 836 | /* |
| 837 | * 3, wMaxPacketSize 3x 1024 bytes | 837 | * 3, wMaxPacketSize 3x 1024 bytes |