Commit c43491d7 authored by Simon Horman's avatar Simon Horman Committed by David S. Miller

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: default avatarSimon Horman <horms@verge.net.au>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e5e26d75
...@@ -560,8 +560,6 @@ greth_start_xmit_gbit(struct sk_buff *skb, struct net_device *dev) ...@@ -560,8 +560,6 @@ greth_start_xmit_gbit(struct sk_buff *skb, struct net_device *dev)
if (net_ratelimit()) if (net_ratelimit())
dev_warn(greth->dev, "Could not create TX DMA mapping\n"); dev_warn(greth->dev, "Could not create TX DMA mapping\n");
dev_kfree_skb(skb); dev_kfree_skb(skb);
return NETDEV_TX_OK;
out: out:
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment