Remove redundant return statements at the end of void functions. This
addresses the checkpatch.pl warnings:

WARNING: void function return statements are not generally useful
#533: FILE: stub_rx.c:533:
+       return;
+}
#1125: FILE: vhci_hcd.c:1125:
+       return;
+}
#116: FILE: vhci_rx.c:116:
+       return;
+}

Signed-off-by: Max Vozeler <m...@hinterhof.net>
---
 drivers/staging/usbip/stub_rx.c  | 1 -
 drivers/staging/usbip/vhci_hcd.c | 1 -
 drivers/staging/usbip/vhci_rx.c  | 2 --
 3 files changed, 4 deletions(-)

diff --git a/drivers/staging/usbip/stub_rx.c b/drivers/staging/usbip/stub_rx.c
index eac1325..5156588 100644
--- a/drivers/staging/usbip/stub_rx.c
+++ b/drivers/staging/usbip/stub_rx.c
@@ -530,7 +530,6 @@ static void stub_recv_cmd_submit(struct stub_device *sdev,
        }
 
        usbip_dbg_stub_rx("Leave\n");
-       return;
 }
 
 /* recv a pdu */
diff --git a/drivers/staging/usbip/vhci_hcd.c b/drivers/staging/usbip/vhci_hcd.c
index e21c1b4..c02374b 100644
--- a/drivers/staging/usbip/vhci_hcd.c
+++ b/drivers/staging/usbip/vhci_hcd.c
@@ -1121,7 +1121,6 @@ static struct platform_driver vhci_driver = {
  */
 static void the_pdev_release(struct device *dev)
 {
-       return;
 }
 
 static struct platform_device the_pdev = {
diff --git a/drivers/staging/usbip/vhci_rx.c b/drivers/staging/usbip/vhci_rx.c
index d07fcb5..00e4a54 100644
i--- a/drivers/staging/usbip/vhci_rx.c
+++ b/drivers/staging/usbip/vhci_rx.c
@@ -111,8 +111,6 @@ static void vhci_recv_ret_submit(struct vhci_device *vdev,
        usb_hcd_giveback_urb(vhci_to_hcd(the_controller), urb, urb->status);
 
        usbip_dbg_vhci_rx("Leave\n");
-
-       return;
 }
 
 static struct vhci_unlink *dequeue_pending_unlink(struct vhci_device *vdev,
-- 
2.0.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to