diff options
| author | Michael Chan <mchan@broadcom.com> | 2008-01-22 04:06:41 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-29 02:10:14 +0300 |
| commit | 1097f5e92107ca3950fabf5e1d724faa80c91e7f (patch) | |
| tree | 7088f4b9ab0d0481d43e2fc3d6a998d44c006dc2 /drivers | |
| parent | 2dd201d7b735f252df1a915a3f2e6a71910a3d87 (diff) | |
| download | linux-1097f5e92107ca3950fabf5e1d724faa80c91e7f.tar.xz | |
[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')
| -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 44734613ab37..18ed8068dc2d 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)) |