diff options
author | Simon Horman <horms@verge.net.au> | 2010-02-25 08:50:19 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-26 07:18:43 -0500 |
commit | c43491d73ea04277c92a4aa1a7e63ccab4577756 (patch) | |
tree | e47db5b26b4514fc8048f2da94aba158787e3ebb /drivers/net/greth.c | |
parent | e5e26d75f490d7d41f25a4b39ed6db1713beb417 (diff) |
greth: fall through to common return statement on error
There doesn't seem to be any reason to explicitly return
NETDEV_TX_OK as err is set to NETDEV_TX_OK in all cases that
reach this point.
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/greth.c')
-rw-r--r-- | drivers/net/greth.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/greth.c b/drivers/net/greth.c index d75b46141603..2b9c1cbc9ec1 100644 --- a/drivers/net/greth.c +++ b/drivers/net/greth.c | |||
@@ -560,8 +560,6 @@ map_error: | |||
560 | if (net_ratelimit()) | 560 | if (net_ratelimit()) |
561 | dev_warn(greth->dev, "Could not create TX DMA mapping\n"); | 561 | dev_warn(greth->dev, "Could not create TX DMA mapping\n"); |
562 | dev_kfree_skb(skb); | 562 | dev_kfree_skb(skb); |
563 | return NETDEV_TX_OK; | ||
564 | |||
565 | out: | 563 | out: |
566 | return err; | 564 | return err; |
567 | } | 565 | } |