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.
Ok, I will fix that. Checkpatch didn't catch that so missed it.
> PS: Your clock/timezone is wrong by approximately 10 hours.
Oh..I have sent this from my virtual-box where time is not properly set :-).
--
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
