On Friday, February 01, 2008 8:29 AM, Abbas, Mohamed wrote: > From: Tomas Winkler [mailto:tomasw@xxxxxxxxx] > Sent: Friday, February 01, 2008 3:44 AM > On Feb 1, 2008 7:46 AM, Reinette Chatre > <reinette.chatre@xxxxxxxxx> wrote: >> From: Mohamed Abbas <mohamed.abbas@xxxxxxxxx> >> >> rate_idx is derived from plcp information >> >> Signed-off-by: Mohamed Abbas <mohamed.abbas@xxxxxxxxx> >> Signed-off-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> --- >> drivers/net/wireless/iwlwifi/iwl-4965.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> [..snip..] > NACK > This should be > rate_idx = > iwl4965_rate_index_from_plcp(wl4965_hw_get_rate(rx_start->rate_ > n_flags)); > > iwl4965_hw_get_rate - this one extract plcp from rate_n_flags - also > takes care of endinaity Tomas > > Actually iwl4965_rate_index_from_plcp expect rate and flag > data to get the rate index. > Mohamed John, Sorry about the confusing emails - please go ahead and apply this patch. Thank you Reinette - 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
