2009/6/5 Gábor Stefanik <netrolller.3d@xxxxxxxxx>: > On Fri, Jun 5, 2009 at 7:41 AM, Luis R. Rodriguez<lrodriguez@xxxxxxxxxxx> wrote: >> index ba06e78..abad86b 100644 >> --- a/drivers/net/wireless/ath/ath9k/rc.c >> +++ b/drivers/net/wireless/ath/ath9k/rc.c >> @@ -741,10 +741,24 @@ static u8 ath_rc_ratefind_ht(struct ath_softc *sc, >> if (rate > (ath_rc_priv->rate_table_size - 1)) >> rate = ath_rc_priv->rate_table_size - 1; >> >> - ASSERT((rate_table->info[rate].valid && >> - (ath_rc_priv->ht_cap & WLAN_RC_DS_FLAG)) || >> - (rate_table->info[rate].valid_single_stream && >> - !(ath_rc_priv->ht_cap & WLAN_RC_DS_FLAG))); >> + if (rate_table->info[rate].valid && >> + (ath_rc_priv->ht_cap & WLAN_RC_DS_FLAG)) >> + return rate; >> + >> + if (rate_table->info[rate].valid_single_stream && >> + !(ath_rc_priv->ht_cap & WLAN_RC_DS_FLAG)); >> + return rate; >> + >> + /* >> + * This should not happen, but we know it does for now... This >> + * needs a proper fix but we're still not sure how this is caused. >> + * Its not *critical* though so lets just warn when debug is enabled >> + * for configuration changes. >> + */ >> + if (sc->debug.debug_mask & ATH_DBG_RATE) >> + WARN_ON(1); > > WARN_ON(sc->debug.debug_mask & ATH_DBG_RATE) Will do, thanks. Luis -- 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
