diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-08-21 09:36:36 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2006-09-26 11:30:32 -0400 |
commit | d084aa706761060940110279794e6a4e83785858 (patch) | |
tree | 8669af3857f5ef9e70f3261e3a8c0af9651518fc /drivers | |
parent | 3153d0d05d2b4e0466818e4f2319882010928d59 (diff) |
V4L/DVB (4510): Fix signedness error in drivers/media/video/vivi.c
when checking the -Wextra signedness warnings issued by gcc 4.1
I came across this one:
drivers/media/video/vivi.c:1001: warning: comparison of unsigned expression < 0 is always false
Since videobuf_reqbufs() returns negative values on errors the current
code does no real error checking since gcc removes the comparison.
This patch fixes this issue by making ret a normal, signed integer.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/media/video/vivi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/media/video/vivi.c b/drivers/media/video/vivi.c index 841884af0cc0..06b44e1dda1d 100644 --- a/drivers/media/video/vivi.c +++ b/drivers/media/video/vivi.c | |||
@@ -992,7 +992,8 @@ static int vidiocgmbuf (struct file *file, void *priv, struct video_mbuf *mbuf) | |||
992 | struct vivi_fh *fh=priv; | 992 | struct vivi_fh *fh=priv; |
993 | struct videobuf_queue *q=&fh->vb_vidq; | 993 | struct videobuf_queue *q=&fh->vb_vidq; |
994 | struct v4l2_requestbuffers req; | 994 | struct v4l2_requestbuffers req; |
995 | unsigned int i, ret; | 995 | unsigned int i; |
996 | int ret; | ||
996 | 997 | ||
997 | req.type = q->type; | 998 | req.type = q->type; |
998 | req.count = 8; | 999 | req.count = 8; |