diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-10-27 15:30:47 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-10-28 05:38:59 -0400 |
commit | 50c8012c429f4a2fe867178e3f2e6a616a20e620 (patch) | |
tree | 1b074849ed2cd2cb16e6f98c66a719582ca93789 | |
parent | 534d032601b1e274da0f0297c1312e79e56344a9 (diff) |
[media] zr364xx: urb actual_length is unsigned
drivers/media/usb/zr364xx/zr364xx.c:1010:2: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits]
Cc: Antoine Jacquet <royale@zerezo.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/usb/zr364xx/zr364xx.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/usb/zr364xx/zr364xx.c b/drivers/media/usb/zr364xx/zr364xx.c index 9afab35878b4..39edd4442932 100644 --- a/drivers/media/usb/zr364xx/zr364xx.c +++ b/drivers/media/usb/zr364xx/zr364xx.c | |||
@@ -1007,8 +1007,7 @@ static void read_pipe_completion(struct urb *purb) | |||
1007 | return; | 1007 | return; |
1008 | } | 1008 | } |
1009 | 1009 | ||
1010 | if (purb->actual_length < 0 || | 1010 | if (purb->actual_length > pipe_info->transfer_size) { |
1011 | purb->actual_length > pipe_info->transfer_size) { | ||
1012 | dev_err(&cam->udev->dev, "wrong number of bytes\n"); | 1011 | dev_err(&cam->udev->dev, "wrong number of bytes\n"); |
1013 | return; | 1012 | return; |
1014 | } | 1013 | } |