Commit 0282bc6a authored by Christophe JAILLET's avatar Christophe JAILLET Committed by David S. Miller
Browse files

net: ag71xx: Slightly simplify 'ag71xx_rx_packets()' 



There is no need to use 'list_for_each_entry_safe' here, as nothing is
removed from the list in the 'for' loop.
Use 'list_for_each_entry' instead, it is slightly less verbose.

Signed-off-by: default avatarChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 514e1150

drivers/net/ethernet/atheros/ag71xx.c

+2 −2
Original line number Diff line number Diff line
@@ -1658,9 +1658,9 @@ static int ag71xx_rx_packets(struct ag71xx *ag, int limit) 
	struct net_device *ndev = ag->ndev;

	int ring_mask, ring_size, done = 0;

	unsigned int pktlen_mask, offset;

	struct sk_buff *next, *skb;

	struct ag71xx_ring *ring;

	struct list_head rx_list;

	struct sk_buff *skb;



	ring = &ag->rx_ring;

	pktlen_mask = ag->dcfg->desc_pktlen_mask;
@@ -1725,7 +1725,7 @@ static int ag71xx_rx_packets(struct ag71xx *ag, int limit) 


	ag71xx_ring_rx_refill(ag);



	list_for_each_entry_safe(skb, next, &rx_list, list)

	list_for_each_entry(skb, &rx_list, list)

		skb->protocol = eth_type_trans(skb, ndev);

	netif_receive_skb_list(&rx_list);