Commit e64a5219 authored by roel kluin's avatar roel kluin Committed by Greg Kroah-Hartman

USB: clean up redundant tests on unsigned

temp, bytes and param->{length,sglen,vary} are unsigned so
these tests do nothing.
Signed-off-by: default avatarRoel Kluin <roel.kluin@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 3b438e30
...@@ -455,9 +455,7 @@ static void qh_lines ( ...@@ -455,9 +455,7 @@ static void qh_lines (
(scratch >> 16) & 0x7fff, (scratch >> 16) & 0x7fff,
scratch, scratch,
td->urb); td->urb);
if (temp < 0) if (size < temp)
temp = 0;
else if (size < temp)
temp = size; temp = size;
size -= temp; size -= temp;
next += temp; next += temp;
...@@ -466,9 +464,7 @@ static void qh_lines ( ...@@ -466,9 +464,7 @@ static void qh_lines (
} }
temp = snprintf (next, size, "\n"); temp = snprintf (next, size, "\n");
if (temp < 0) if (size < temp)
temp = 0;
else if (size < temp)
temp = size; temp = size;
size -= temp; size -= temp;
next += temp; next += temp;
......
...@@ -192,8 +192,6 @@ static struct urb *simple_alloc_urb ( ...@@ -192,8 +192,6 @@ static struct urb *simple_alloc_urb (
{ {
struct urb *urb; struct urb *urb;
if (bytes < 0)
return NULL;
urb = usb_alloc_urb (0, GFP_KERNEL); urb = usb_alloc_urb (0, GFP_KERNEL);
if (!urb) if (!urb)
return urb; return urb;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment