On Sun, Sep 25, 2011 at 11:58:47AM +0300, Eliad Peller wrote:

On Fri, Sep 23, 2011 at 6:12 PM, Rajkumar Manoharan
[off-list ref] wrote:

quoted

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 <redacted>
Signed-off-by: Rajkumar Manoharan <redacted>
---

[...]

quoted

+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