Commit ae97fec3 authored by Jeff Garzik's avatar Jeff Garzik

drivers/usb/misc/auerswald: fix status check, remove redundant check

1) We should only set 'actual_length' output variable if usb length is
known to be good.

2) No need to check actual_length for NULL.  The only caller always
passes non-NULL value.
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
parent cad1b9da
...@@ -630,7 +630,7 @@ static int auerchain_start_wait_urb (pauerchain_t acp, struct urb *urb, int time ...@@ -630,7 +630,7 @@ static int auerchain_start_wait_urb (pauerchain_t acp, struct urb *urb, int time
} else } else
status = urb->status; status = urb->status;
if (actual_length) if (status >= 0)
*actual_length = urb->actual_length; *actual_length = urb->actual_length;
return status; return status;
......
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