diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-07-17 01:08:29 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-07-17 16:18:00 -0400 |
commit | ae97fec3701a559929c3529e35417fab133a4d39 (patch) | |
tree | 3eddac64147a077f5f312e24ddc801b074ac2285 /drivers/usb | |
parent | cad1b9da74f14c5f15b63ffc93c53debe09b3781 (diff) | |
download | op-kernel-dev-ae97fec3701a559929c3529e35417fab133a4d39.zip op-kernel-dev-ae97fec3701a559929c3529e35417fab133a4d39.tar.gz |
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: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/misc/auerswald.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/misc/auerswald.c b/drivers/usb/misc/auerswald.c index 1fd5fc2..3e22b2f 100644 --- a/drivers/usb/misc/auerswald.c +++ b/drivers/usb/misc/auerswald.c @@ -630,7 +630,7 @@ static int auerchain_start_wait_urb (pauerchain_t acp, struct urb *urb, int time } else status = urb->status; - if (actual_length) + if (status >= 0) *actual_length = urb->actual_length; return status; |