This was printing the wrong value on the warning. While at it lets expand this warning to provide a little more useful information for debugging such as the band and hardware rate index when possible and clarify what the warning is actually printing. Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx> --- I hit this warning and found it would be more useful with this. I can't reproduce this right now, but I believe what I did was suspend at home and resume at the office. Anyway this should help clear things up. drivers/net/wireless/ath/ath5k/base.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c index 5056410..a4e69c5 100644 --- a/drivers/net/wireless/ath/ath5k/base.c +++ b/drivers/net/wireless/ath/ath5k/base.c @@ -1136,11 +1136,15 @@ ath5k_hw_to_driver_rix(struct ath5k_softc *sc, int hw_rix) /* return base rate on errors */ if (WARN(hw_rix < 0 || hw_rix >= AR5K_MAX_RATES, - "hw_rix out of bounds: %x\n", hw_rix)) + "hardware rate index out of bounds: %x" + "(band: %s)\n", hw_rix, + sc->curband->band ? "5 GHz" : "2.4 GHz")) return 0; rix = sc->rate_idx[sc->curband->band][hw_rix]; - if (WARN(rix < 0, "invalid hw_rix: %x\n", hw_rix)) + if (WARN(rix < 0, "invalid driver rate index: %x " + "(hw_rix: %x band: %s)\n", rix, hw_rix, + sc->curband->band ? "5 GHz" : "2.4 GHz")) rix = 0; return rix; -- 1.6.3.3 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
