Johannes Berg wrote: > We already have a bss pointer here, would it be better to pass that in, > avoid the bss lookup, and do the bss lookup manually in the action frame > case, to avoid a double lookup here? > We don't have a bss pointer in ieee80211_rx_h_action(), where CSA action frames are handled too. > Didn't you want to put this into the workqueue? Calling ieee80211_hw_config() from the timer was problematic since drivers might use a mutex, this won't matter. But if it makes things clearer, I can add it to the workqueue here too. Sujith -- 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