Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4989ccb2 authored by roel kluin's avatar roel kluin Committed by David S. Miller
Browse files

au1000_eth: Duplicate test of RX_OVERLEN bit in update_rx_stats() 



in update_rx_stats() the RX_OVERLEN bit is set twice, replace it by RX_RUNT.
in au1000_rx() the RX_MISSED_FRAME bit was tested a few lines earlier already

Signed-off-by: default avatarRoel Kluin <roel.kluin@gmail.com>
Acked-by: default avatarManuel Lauss <manuel.lauss@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fa9859ef

drivers/net/au1000_eth.c

+1 −3
Original line number Original line Diff line number Diff line
@@ -721,7 +721,7 @@ static inline void update_rx_stats(struct net_device *dev, u32 status) 
		ps->rx_errors++;

		ps->rx_errors++;

		if (status & RX_MISSED_FRAME)

		if (status & RX_MISSED_FRAME)

			ps->rx_missed_errors++;

			ps->rx_missed_errors++;

		if (status & (RX_OVERLEN | RX_OVERLEN | RX_LEN_ERROR))

		if (status & (RX_OVERLEN | RX_RUNT | RX_LEN_ERROR))

			ps->rx_length_errors++;

			ps->rx_length_errors++;

		if (status & RX_CRC_ERROR)

		if (status & RX_CRC_ERROR)

			ps->rx_crc_errors++;

			ps->rx_crc_errors++;
@@ -794,8 +794,6 @@ static int au1000_rx(struct net_device *dev) 
					printk("rx len error\n");

					printk("rx len error\n");

				if (status & RX_U_CNTRL_FRAME)

				if (status & RX_U_CNTRL_FRAME)

					printk("rx u control frame\n");

					printk("rx u control frame\n");

				if (status & RX_MISSED_FRAME)

					printk("rx miss\n");

			}

			}

		}

		}

		prxd->buff_stat = (u32)(pDB->dma_addr | RX_DMA_ENABLE);

		prxd->buff_stat = (u32)(pDB->dma_addr | RX_DMA_ENABLE);