Staging: usb-ip: vhci_hdc:Fix the returned error value
authorShan Wei <shanwei@cn.fujitsu.com>
Fri, 24 Jul 2009 08:57:35 +0000 (16:57 +0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 15 Sep 2009 19:02:05 +0000 (12:02 -0700)
Compared with other drivers, the "ret" should be nagative and
returned. But in vhci_hdc, it always return 0;

I dont't use the driver, and I'm not familiar with the code.
Hope the patch is helpful.

Signed-off-by: Shan Wei <shanwei@cn.fujitsu.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/usbip/vhci_hcd.c

index 114db8b26028f86dab2da8d356b437de7bf6fd5c..6e91fc2bd850a2bf9c3e2f11f40fb935b93df873 100644 (file)
@@ -594,7 +594,7 @@ static int vhci_urb_enqueue(struct usb_hcd *hcd, struct urb *urb,
 
                if (type != PIPE_CONTROL || !ctrlreq) {
                        dev_err(dev, "invalid request to devnum 0\n");
-                       ret = EINVAL;
+                       ret = -EINVAL;
                        goto no_need_xmit;
                }
 
@@ -653,7 +653,7 @@ no_need_unlink:
 
        usb_hcd_giveback_urb(vhci_to_hcd(the_controller), urb, urb->status);
 
-       return 0;
+       return ret;
 }
 
 /*