On Sun, Sep 25, 2011 at 11:58:47AM +0300, Eliad Peller wrote:
> On Fri, Sep 23, 2011 at 6:12 PM, Rajkumar Manoharan
> <rmanohar@xxxxxxxxxxxxxxxx> wrote:
> > Whenever the scan request or tx_mgmt is requesting not to
> > use CCK rate for managemet frames through
> > NL80211_ATTR_TX_NO_CCK_RATE attribute, then mac80211 should
> > select appropriate least non-CCK rate. This could help to
> > send P2P probes and P2P action frames at non 11b rates
> > without diabling 11b rates globally.
> >
> > Cc: Jouni Malinen <jouni@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx>
> > ---
> [...]
>
> > +static inline s8
> > +rate_lowest_non_cck_index(struct ieee80211_supported_band *sband,
> > + struct ieee80211_sta *sta)
> > +{
> > + int i;
> > +
> > + for (i = 0; i < sband->n_bitrates; i++) {
> > + struct ieee80211_rate *srate = &sband->bitrates[i];
> > + if ((srate->bitrate == 10) || (srate->bitrate == 20) ||
> > + (srate->bitrate == 55) || (srate->bitrate == 110))
> > + continue;
> > +
> > + if ((i != sband->n_bitrates) &&
> > + rate_supported(sta, sband->band, i))
> > + return i;
>
> the first condition seems redundant (according to the loop condition,
> i < sband->n_bitrates)
Yeah. i missed it ;)
--
Rajkumar
--
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
