On Fri, 2013-08-09 at 16:35 +0200, Simon Wunderlich wrote: > --- a/net/mac80211/mlme.c > +++ b/net/mac80211/mlme.c > @@ -981,54 +981,35 @@ static void ieee80211_chswitch_timer(unsigned long data) > ieee80211_queue_work(&sdata->local->hw, &sdata->u.mgd.chswitch_work); > } > > -static void > -ieee80211_sta_process_chanswitch(struct ieee80211_sub_if_data *sdata, > - u64 timestamp, struct ieee802_11_elems *elems, > - bool beacon) > +int ieee80211_parse_ch_switch_ie(struct ieee80211_sub_if_data *sdata, I think you should also move this function out of mlme.c, maybe the (almost empty) spectmgmt.c file? > + struct ieee802_11_elems *elems, bool beacon, > + enum ieee80211_band current_band, > + u32 sta_flags, Passing sta_flags seems a bit odd - I understand the motivation but I think it would be worthwhile to document (maybe in the kernel-doc in the header file) which flags are used. 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
