On Fri, Feb 27, 2015 at 2:50 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > +Arik. > > It'd be nice (for me anyway) if you didn't send this in a series of > other patches I don't care about - I only ever saw this due to > patchwork. > > On Wed, 2015-02-25 at 08:55 +0100, Marek Puzyniak wrote: >> Currently when TDLS station in driver goes from assoc >> to authorized state it can not use rate control parameters >> because rate control is not initialized yet. Some drivers >> require parameters already initialized by rate control when >> entering authorized state. It can be done by initializing >> rate control after station transition to authorized state >> but before notifyiIEEE80211_STA_ASSOCng driver about that. > > 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). 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
