diff options
author | Michael Chan <mchan@broadcom.com> | 2008-01-21 17:06:41 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 15:10:14 -0800 |
commit | 1097f5e92107ca3950fabf5e1d724faa80c91e7f (patch) | |
tree | 7088f4b9ab0d0481d43e2fc3d6a998d44c006dc2 /drivers/net | |
parent | 2dd201d7b735f252df1a915a3f2e6a71910a3d87 (diff) | |
download | op-kernel-dev-1097f5e92107ca3950fabf5e1d724faa80c91e7f.zip op-kernel-dev-1097f5e92107ca3950fabf5e1d724faa80c91e7f.tar.gz |
[BNX2]: Fix minor bug in bnx2_has_work().
It is more correct to get the status block from the bnx2_napi struct
instead of the bnx2 struct. It happens that they are the same in this
case because we are using the first MSIX vector.
Signed-off-by: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/bnx2.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/bnx2.c b/drivers/net/bnx2.c index 4473461..18ed806 100644 --- a/drivers/net/bnx2.c +++ b/drivers/net/bnx2.c @@ -2903,8 +2903,7 @@ bnx2_tx_msix(int irq, void *dev_instance) static inline int bnx2_has_work(struct bnx2_napi *bnapi) { - struct bnx2 *bp = bnapi->bp; - struct status_block *sblk = bp->status_blk; + struct status_block *sblk = bnapi->status_blk; if ((bnx2_get_hw_rx_cons(bnapi) != bnapi->rx_cons) || (bnx2_get_hw_tx_cons(bnapi) != bnapi->hw_tx_cons)) |