summaryrefslogtreecommitdiffstats
path: root/net/mac802154
diff options
context:
space:
mode:
authorstephen hemminger <stephen@networkplumber.org>2014-02-12 20:51:22 -0800
committerDavid S. Miller <davem@davemloft.net>2014-02-13 18:33:38 -0500
commit2045ceaed4d54e6e698874d008be727ee5b2a01c (patch)
treea481bc211dd20db90275aceab3721c7a8fb147d9 /net/mac802154
parent45f7435968363816f8fc4c6abef692808534140d (diff)
downloadop-kernel-dev-2045ceaed4d54e6e698874d008be727ee5b2a01c.zip
op-kernel-dev-2045ceaed4d54e6e698874d008be727ee5b2a01c.tar.gz
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.c4
-rw-r--r--net/mac802154/rx.c1
2 files changed, 0 insertions, 5 deletions
diff --git a/net/mac802154/mib.c b/net/mac802154/mib.c
index 8ded97c..f48f40c1 100644
--- a/net/mac802154/mib.c
+++ b/net/mac802154/mib.c
@@ -62,8 +62,6 @@ static void hw_addr_notify(struct work_struct *work)
pr_debug("failed changed mask %lx\n", nw->changed);
kfree(nw);
-
- return;
}
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)
work->dev = dev;
work->changed = changed;
queue_work(priv->hw->dev_workqueue, &work->work);
-
- return;
}
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 38548ec..03855b0 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)
mac802154_wpans_rx(priv, skb);
out:
dev_kfree_skb(skb);
- return;
}
static void mac802154_rx_worker(struct work_struct *work)
OpenPOWER on IntegriCloud