On Thu, Aug 07, 2008 at 04:46:28PM +0200, Johannes Berg wrote: > On Thu, 2008-08-07 at 17:27 +0300, Jouni Malinen wrote: > > Well, I don't actually need it per-BSS for hardware that I'm most likely > > to work with ;-), but yeah, this is a per-BSS parameter, so in that > > sense your patch is doing the correct thing. > > Yeah, ok, I guess I don't really care, do whichever way you want. It'd > be a bit strange to set it with cfg80211 per BSS and have it show up on > different BSSes, but that's about it, and a little strangeness hasn't > hurt anyone ;) It looks like it is easiest to make this a per-BSS parameter now and use bss_info_changed() for it. However, I'm planning on leaving the other parts of your patch out from the initial version. In other words, I will not remove IEEE80211_CONF_SHORT_SLOT_TIME now (I just added a TODO note to get it removed once drivers have been converted; mac80211 was never actually setting this, anyway). This avoids the need to modify all drivers with this change. In addition, I'm leaving out the client MLME stuff for now, but the notification part will be available for it once the MLME code is added. -- Jouni Malinen PGP id EFC895FA -- 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
