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. johannes -- 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
