diff options
author | Bob Copeland <me@bobcopeland.com> | 2009-04-27 22:18:10 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-05-06 15:14:50 -0400 |
commit | 2516baa63bc7d05a6573a0f584135484ed182a87 (patch) | |
tree | ff2c1efd231d9c5d241be1d6b1374d07791b8122 /drivers/net/wireless/ath/ath5k/base.c | |
parent | 35f00cfcc06bb85e0659f9847400518008d78145 (diff) |
ath5k: correct interrupt storm warning
Ben Greear points out that the "too many interrupts" message will
never print in the intended case since the interrupt counter
will be -1 after the loop. Change it to pre-decrement so it will
be 0 on the thousandth iteration.
Cc: Ben Greear <greearb@candelatech.com>
Signed-off-by: Bob Copeland <me@bobcopeland.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/ath/ath5k/base.c')
-rw-r--r-- | drivers/net/wireless/ath/ath5k/base.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index 410fc4cfc4c6..7b80cebffd41 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c | |||
@@ -2505,7 +2505,7 @@ ath5k_intr(int irq, void *dev_id) | |||
2505 | ath5k_hw_update_mib_counters(ah, &sc->ll_stats); | 2505 | ath5k_hw_update_mib_counters(ah, &sc->ll_stats); |
2506 | } | 2506 | } |
2507 | } | 2507 | } |
2508 | } while (ath5k_hw_is_intr_pending(ah) && counter-- > 0); | 2508 | } while (ath5k_hw_is_intr_pending(ah) && --counter > 0); |
2509 | 2509 | ||
2510 | if (unlikely(!counter)) | 2510 | if (unlikely(!counter)) |
2511 | ATH5K_WARN(sc, "too many interrupts, giving up for now\n"); | 2511 | ATH5K_WARN(sc, "too many interrupts, giving up for now\n"); |