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 <max@xxxxxxxxxxxxx> --- 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 devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel