diff options
author | Jon Mason <jdmason@kudzu.us> | 2012-07-09 10:09:32 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-07-11 02:24:57 -0400 |
commit | eb716c54b1c71ad28ab20461bff831bd481066c4 (patch) | |
tree | 3798343256ba4e216570bb0e1f4b829047aff2f4 /drivers/net | |
parent | c768b681f4d786721ad56e800358ddab2abb9af1 (diff) |
sunbmac: remove unnecessary setting of skb->dev
skb->dev is being unnecessarily set during ring init and skb alloc in rx. It is
already being set to the proper value when eth_type_trans is called on packet
receive, and the skb->dev is not referenced anywhere else in the code.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/sun/sunbmac.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/ethernet/sun/sunbmac.c b/drivers/net/ethernet/sun/sunbmac.c index 2a83fc57edba..967fe8cb476e 100644 --- a/drivers/net/ethernet/sun/sunbmac.c +++ b/drivers/net/ethernet/sun/sunbmac.c | |||
@@ -233,7 +233,6 @@ static void bigmac_init_rings(struct bigmac *bp, int from_irq) | |||
233 | continue; | 233 | continue; |
234 | 234 | ||
235 | bp->rx_skbs[i] = skb; | 235 | bp->rx_skbs[i] = skb; |
236 | skb->dev = dev; | ||
237 | 236 | ||
238 | /* Because we reserve afterwards. */ | 237 | /* Because we reserve afterwards. */ |
239 | skb_put(skb, ETH_FRAME_LEN); | 238 | skb_put(skb, ETH_FRAME_LEN); |
@@ -838,7 +837,6 @@ static void bigmac_rx(struct bigmac *bp) | |||
838 | RX_BUF_ALLOC_SIZE - 34, | 837 | RX_BUF_ALLOC_SIZE - 34, |
839 | DMA_FROM_DEVICE); | 838 | DMA_FROM_DEVICE); |
840 | bp->rx_skbs[elem] = new_skb; | 839 | bp->rx_skbs[elem] = new_skb; |
841 | new_skb->dev = bp->dev; | ||
842 | skb_put(new_skb, ETH_FRAME_LEN); | 840 | skb_put(new_skb, ETH_FRAME_LEN); |
843 | skb_reserve(new_skb, 34); | 841 | skb_reserve(new_skb, 34); |
844 | this->rx_addr = | 842 | this->rx_addr = |