On Feb 1, 2008 7:40 PM, Chatre, Reinette <reinette.chatre@xxxxxxxxx> wrote: > 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 Sorry it was my mistake. > 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