On Thu, Jan 23, 2014 at 2:57 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > On Wed, 2014-01-22 at 12:27 +0530, Chaitanya T K wrote: > > > + if (sta->smps_mode == IEEE80211_SMPS_DYNAMIC && > > + minstrel_mcs_groups[i].streams > 1) { > > + mi->tx_flags |= IEEE80211_TX_RC_USE_RTS_CTS; > > + } else if (sta->smps_mode == IEEE80211_SMPS_STATIC && > > + minstrel_mcs_groups[i].streams > 1) { > > Your indentation is off inside the if statements. I'll have to wait for > nbd to review the patch itself, I guess. > Felix, Any comments other than the indentation? If no, will send in the v2 with indentation fixed. -- 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