diff options
author | Joe Perches <joe@perches.com> | 2010-05-14 03:19:28 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-05-14 03:19:28 -0400 |
commit | a4b770972b8f819e408d7cc3ae9637e15bff62f6 (patch) | |
tree | 62e3c413b084c106d750cf6cc0701709cde30100 /drivers/net/wireless/libertas | |
parent | 21ce849ba5cc178740c6532ba3dded852296ad91 (diff) |
drivers/net: Remove unnecessary returns from void function()s
This patch removes from drivers/net/ all the unnecessary
return; statements that precede the last closing brace of
void functions.
It does not remove the returns that are immediately
preceded by a label as gcc doesn't like that.
It also does not remove null void functions with return.
Done via:
$ grep -rP --include=*.[ch] -l "return;\n}" net/ | \
xargs perl -i -e 'local $/ ; while (<>) { s/\n[ \t\n]+return;\n}/\n}/g; print; }'
with some cleanups by hand.
Compile tested x86 allmodconfig only.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/libertas')
-rw-r--r-- | drivers/net/wireless/libertas/debugfs.c | 3 | ||||
-rw-r--r-- | drivers/net/wireless/libertas/if_usb.c | 4 | ||||
-rw-r--r-- | drivers/net/wireless/libertas/rx.c | 1 |
3 files changed, 0 insertions, 8 deletions
diff --git a/drivers/net/wireless/libertas/debugfs.c b/drivers/net/wireless/libertas/debugfs.c index 6f5b843c1f4..de2caac11dd 100644 --- a/drivers/net/wireless/libertas/debugfs.c +++ b/drivers/net/wireless/libertas/debugfs.c | |||
@@ -757,15 +757,12 @@ void lbs_debugfs_init(void) | |||
757 | { | 757 | { |
758 | if (!lbs_dir) | 758 | if (!lbs_dir) |
759 | lbs_dir = debugfs_create_dir("lbs_wireless", NULL); | 759 | lbs_dir = debugfs_create_dir("lbs_wireless", NULL); |
760 | |||
761 | return; | ||
762 | } | 760 | } |
763 | 761 | ||
764 | void lbs_debugfs_remove(void) | 762 | void lbs_debugfs_remove(void) |
765 | { | 763 | { |
766 | if (lbs_dir) | 764 | if (lbs_dir) |
767 | debugfs_remove(lbs_dir); | 765 | debugfs_remove(lbs_dir); |
768 | return; | ||
769 | } | 766 | } |
770 | 767 | ||
771 | void lbs_debugfs_init_one(struct lbs_private *priv, struct net_device *dev) | 768 | void lbs_debugfs_init_one(struct lbs_private *priv, struct net_device *dev) |
diff --git a/drivers/net/wireless/libertas/if_usb.c b/drivers/net/wireless/libertas/if_usb.c index fcea5741ba6..f41594c7ac1 100644 --- a/drivers/net/wireless/libertas/if_usb.c +++ b/drivers/net/wireless/libertas/if_usb.c | |||
@@ -133,8 +133,6 @@ static void if_usb_write_bulk_callback(struct urb *urb) | |||
133 | /* print the failure status number for debug */ | 133 | /* print the failure status number for debug */ |
134 | lbs_pr_info("URB in failure status: %d\n", urb->status); | 134 | lbs_pr_info("URB in failure status: %d\n", urb->status); |
135 | } | 135 | } |
136 | |||
137 | return; | ||
138 | } | 136 | } |
139 | 137 | ||
140 | /** | 138 | /** |
@@ -651,8 +649,6 @@ static void if_usb_receive_fwload(struct urb *urb) | |||
651 | if_usb_submit_rx_urb_fwload(cardp); | 649 | if_usb_submit_rx_urb_fwload(cardp); |
652 | 650 | ||
653 | kfree(syncfwheader); | 651 | kfree(syncfwheader); |
654 | |||
655 | return; | ||
656 | } | 652 | } |
657 | 653 | ||
658 | #define MRVDRV_MIN_PKT_LEN 30 | 654 | #define MRVDRV_MIN_PKT_LEN 30 |
diff --git a/drivers/net/wireless/libertas/rx.c b/drivers/net/wireless/libertas/rx.c index e2b8d886b09..a115bfa9513 100644 --- a/drivers/net/wireless/libertas/rx.c +++ b/drivers/net/wireless/libertas/rx.c | |||
@@ -90,7 +90,6 @@ static void lbs_save_rawSNRNF(struct lbs_private *priv, struct rxpd *p_rx_pd) | |||
90 | priv->nextSNRNF++; | 90 | priv->nextSNRNF++; |
91 | if (priv->nextSNRNF >= DEFAULT_DATA_AVG_FACTOR) | 91 | if (priv->nextSNRNF >= DEFAULT_DATA_AVG_FACTOR) |
92 | priv->nextSNRNF = 0; | 92 | priv->nextSNRNF = 0; |
93 | return; | ||
94 | } | 93 | } |
95 | 94 | ||
96 | /** | 95 | /** |