Hi, On 22 August 2011 23:50, Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote: > In preparation for fixing the rate-mapping situation, place a driver-agnostic > version in rtlwifi. This one contains the updated rate incormation. > > Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > Cc: Chaoming Li <chaoming_li@xxxxxxxxxxxxxx> > --- > drivers/net/wireless/rtlwifi/base.c | 161 +++++++++++++++++++++++++++++++++++ > drivers/net/wireless/rtlwifi/base.h | 2 + > drivers/net/wireless/rtlwifi/wifi.h | 35 ++++++++ > 3 files changed, 198 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/wireless/rtlwifi/base.c b/drivers/net/wireless/rtlwifi/base.c > index 0b598db..098fc55 100644 > --- a/drivers/net/wireless/rtlwifi/base.c > +++ b/drivers/net/wireless/rtlwifi/base.c > @@ -664,6 +664,167 @@ static u8 _rtl_get_highest_n_rate(struct ieee80211_hw *hw) > return hw_rate; > } > > +/* mac80211's rate_idx is like this: > + * > + * 2.4G band:rx_status->band == IEEE80211_BAND_2GHZ > + * > + * B/G rate: > + * (rx_status->flag & RX_FLAG_HT) = 0, > + * DESC92_RATE1M-->DESC92_RATE54M ==> idx is 0-->11, > + * > + * N rate: > + * (rx_status->flag & RX_FLAG_HT) = 1, > + * DESC92_RATEMCS0-->DESC92_RATEMCS15 ==> idx is 0-->15 > + * > + * 5G band:rx_status->band == IEEE80211_BAND_5GHZ > + * A rate: > + * (rx_status->flag & RX_FLAG_HT) = 0, > + * DESC92_RATE6M-->DESC92_RATE54M ==> idx is 0-->7, > + * > + * N rate: > + * (rx_status->flag & RX_FLAG_HT) = 1, > + * DESC92_RATEMCS0-->DESC92_RATEMCS15 ==> idx is 0-->15 > + */ > +int rtlwifi_rate_mapping(struct ieee80211_hw *hw, > + bool isht, u8 desc_rate, bool first_ampdu) first_ampdu is never used in this function, is this intentional? I see the (later) removed functions change the default based on this, but without much knowledge of 802.11 I can't tell whether this actually makes sense, etc. ;-). </review> Jonas -- 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
