diff options
| author | Florian Fainelli <f.fainelli@gmail.com> | 2014-06-05 13:22:16 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-06-05 18:36:54 -0400 |
| commit | ad51c6106119adf8565adb179989a06ab4b865bb (patch) | |
| tree | 8593ff0466835b38026f1e90aa1e9334eb681dd8 | |
| parent | d8498088cf466483093c9a29121e3833e7923287 (diff) | |
net: systemport: correctly check for RX_STATUS_OVFLOW flag
We were missing an and comparison with status to check whether
RX_STATUS_OVFLOW is asserted or not in the per-packet status word, fix
that.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/broadcom/bcmsysport.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bcmsysport.c b/drivers/net/ethernet/broadcom/bcmsysport.c index 4dfc93fe9744..fc84aba0cde0 100644 --- a/drivers/net/ethernet/broadcom/bcmsysport.c +++ b/drivers/net/ethernet/broadcom/bcmsysport.c | |||
| @@ -514,7 +514,7 @@ static unsigned int bcm_sysport_desc_rx(struct bcm_sysport_priv *priv, | |||
| 514 | 514 | ||
| 515 | if (unlikely(status & (RX_STATUS_ERR | RX_STATUS_OVFLOW))) { | 515 | if (unlikely(status & (RX_STATUS_ERR | RX_STATUS_OVFLOW))) { |
| 516 | netif_err(priv, rx_err, ndev, "error packet\n"); | 516 | netif_err(priv, rx_err, ndev, "error packet\n"); |
| 517 | if (RX_STATUS_OVFLOW) | 517 | if (status & RX_STATUS_OVFLOW) |
| 518 | ndev->stats.rx_over_errors++; | 518 | ndev->stats.rx_over_errors++; |
| 519 | ndev->stats.rx_dropped++; | 519 | ndev->stats.rx_dropped++; |
| 520 | ndev->stats.rx_errors++; | 520 | ndev->stats.rx_errors++; |
