On Thu, Jan 30, 2014 at 11:48 PM, Krishna Chaitanya <chaitanya.mgit@xxxxxxxxx> wrote: > 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. Was checking my old submissions. This was marked as "Not Applicable" in patchwork. Felix/Johannes, Can you please explain to me the meaning of the status "Not Applicable"? Regards, Chaitanya T K. -- Thanks, Regards, Chaitanya T K. -- 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