diff options
author | stephen hemminger <stephen@networkplumber.org> | 2014-02-12 23:51:22 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-02-13 18:33:38 -0500 |
commit | 2045ceaed4d54e6e698874d008be727ee5b2a01c (patch) | |
tree | a481bc211dd20db90275aceab3721c7a8fb147d9 /net/mac802154 | |
parent | 45f7435968363816f8fc4c6abef692808534140d (diff) |
net: remove unnecessary return's
One of my pet coding style peeves is the practice of
adding extra return; at the end of function.
Kill several instances of this in network code.
I suppose some coccinelle wizardy could do this automatically.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/mac802154')
-rw-r--r-- | net/mac802154/mib.c | 4 | ||||
-rw-r--r-- | net/mac802154/rx.c | 1 |
2 files changed, 0 insertions, 5 deletions
diff --git a/net/mac802154/mib.c b/net/mac802154/mib.c index 8ded97cf1c33..f48f40c1da1a 100644 --- a/net/mac802154/mib.c +++ b/net/mac802154/mib.c | |||
@@ -62,8 +62,6 @@ static void hw_addr_notify(struct work_struct *work) | |||
62 | pr_debug("failed changed mask %lx\n", nw->changed); | 62 | pr_debug("failed changed mask %lx\n", nw->changed); |
63 | 63 | ||
64 | kfree(nw); | 64 | kfree(nw); |
65 | |||
66 | return; | ||
67 | } | 65 | } |
68 | 66 | ||
69 | static void set_hw_addr_filt(struct net_device *dev, unsigned long changed) | 67 | static void set_hw_addr_filt(struct net_device *dev, unsigned long changed) |
@@ -79,8 +77,6 @@ static void set_hw_addr_filt(struct net_device *dev, unsigned long changed) | |||
79 | work->dev = dev; | 77 | work->dev = dev; |
80 | work->changed = changed; | 78 | work->changed = changed; |
81 | queue_work(priv->hw->dev_workqueue, &work->work); | 79 | queue_work(priv->hw->dev_workqueue, &work->work); |
82 | |||
83 | return; | ||
84 | } | 80 | } |
85 | 81 | ||
86 | void mac802154_dev_set_short_addr(struct net_device *dev, u16 val) | 82 | void mac802154_dev_set_short_addr(struct net_device *dev, u16 val) |
diff --git a/net/mac802154/rx.c b/net/mac802154/rx.c index 38548ec2098f..03855b0677cc 100644 --- a/net/mac802154/rx.c +++ b/net/mac802154/rx.c | |||
@@ -80,7 +80,6 @@ mac802154_subif_rx(struct ieee802154_dev *hw, struct sk_buff *skb, u8 lqi) | |||
80 | mac802154_wpans_rx(priv, skb); | 80 | mac802154_wpans_rx(priv, skb); |
81 | out: | 81 | out: |
82 | dev_kfree_skb(skb); | 82 | dev_kfree_skb(skb); |
83 | return; | ||
84 | } | 83 | } |
85 | 84 | ||
86 | static void mac802154_rx_worker(struct work_struct *work) | 85 | static void mac802154_rx_worker(struct work_struct *work) |