Re: [PATCH 2/2] mac80211: Send the management frame at requested rate

[Rajkumar Manoharan <rmanohar@qca.qualcomm.com>]

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@qca.qualcomm.com> 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@qca.qualcomm.com>
> > Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> > ---
> [...]
> 
> > +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