On Tue, Mar 3, 2015 at 12:06 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Tue, 2015-03-03 at 12:02 +0200, Arik Nemtsov wrote: >> On Tue, Mar 3, 2015 at 11:18 AM, Johannes Berg >> <johannes@xxxxxxxxxxxxxxxx> wrote: >> > On Sun, 2015-03-01 at 10:21 +0200, Arik Nemtsov wrote: >> > >> >> > Arik, you have a similar patch handling only NSS. Does this one look >> >> > fine to you, and would it solve the problem your other patch solved? >> >> >> >> Well currently iwlmvm requires the NSS to be set before >> >> IEEE80211_STA_ASSOC (earlier), so this doesn't help directly. >> >> I could change mvm a bit to make it work, but I don't really see a >> >> good reason for it :) >> >> >> >> The patch looks good. Shouldn't introduce new issues (at least for iwlwifi). >> > >> > Ok, thanks. Do you think it would be possible to move all of this before >> > ASSOC? >> >> We can probably move his rate_control_rate_init() to assoc, since we have this: > > And doing so would also address the nss problem, right? IIRC that's done > in the rate control init inline. Right. Arik -- 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
