diff options
| author | roel kluin <roel.kluin@gmail.com> | 2009-10-06 11:54:18 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-10-07 12:50:10 +0200 |
| commit | 4989ccb25cc2a6d03d3503a2c852644f9a750948 (patch) | |
| tree | 3e4063313fd26bb6e1a1d5566ed9ecda888d4060 /drivers/net/au1000_eth.c | |
| parent | netxen: Fix Unlikely(x) > y (diff) | |
| download | linux-4989ccb25cc2a6d03d3503a2c852644f9a750948.tar.xz linux-4989ccb25cc2a6d03d3503a2c852644f9a750948.zip | |
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: Roel Kluin <roel.kluin@gmail.com>
Acked-by: Manuel Lauss <manuel.lauss@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/au1000_eth.c')
| -rw-r--r-- | drivers/net/au1000_eth.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/au1000_eth.c b/drivers/net/au1000_eth.c index fdf5937233fc..04f63c77071d 100644 --- a/drivers/net/au1000_eth.c +++ b/drivers/net/au1000_eth.c @@ -721,7 +721,7 @@ static inline void update_rx_stats(struct net_device *dev, u32 status) ps->rx_errors++; if (status & RX_MISSED_FRAME) 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++; if (status & RX_CRC_ERROR) ps->rx_crc_errors++; @@ -794,8 +794,6 @@ static int au1000_rx(struct net_device *dev) printk("rx len error\n"); if (status & RX_U_CNTRL_FRAME) 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); |