On Fri, 2009-02-27 at 19:43 +0100, Alina Friedrichsen wrote: > This patch _not_ implements commit in wireless extensions, but solves some dependencies needed by the other patches. > > The maybe usage of this ioctl must discussed after the complete joining process (STA and *IBSS*) in nl80211 defined. > > I hope this is a compromise. > > Signed-off-by: Alina Friedrichsen <x-alina@xxxxxxx> > --- > diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h > index ecbc8e0..7f00c70 100644 > --- a/net/mac80211/ieee80211_i.h > +++ b/net/mac80211/ieee80211_i.h > @@ -920,6 +920,7 @@ static inline int ieee80211_bssid_match(const u8 *raddr, const u8 *addr) > > int ieee80211_hw_config(struct ieee80211_local *local, u32 changed); > int ieee80211_if_config(struct ieee80211_sub_if_data *sdata, u32 changed); > +void ieee80211_commit(struct ieee80211_sub_if_data *sdata); > void ieee80211_tx_set_protected(struct ieee80211_tx_data *tx); > void ieee80211_bss_info_change_notify(struct ieee80211_sub_if_data *sdata, > u32 changed); > diff --git a/net/mac80211/main.c b/net/mac80211/main.c > index f38db4d..1b2d598 100644 > --- a/net/mac80211/main.c > +++ b/net/mac80211/main.c > @@ -294,6 +294,20 @@ int ieee80211_hw_config(struct ieee80211_local *local, u32 changed) > return ret; > } > > +void ieee80211_commit(struct ieee80211_sub_if_data *sdata) > +{ > + switch (sdata->vif.type) { > + case NL80211_IFTYPE_ADHOC: > + ieee80211_ibss_commit(sdata); > + break; > + case NL80211_IFTYPE_STATION: > + ieee80211_sta_commit(sdata); > + break; > + default: > + break; > + } I don't really see a need for that, it's only called in what, two places? And it makes things interdependent. At _least_ put it into a more appropriate file like wext.c and open-code it in cfg.c johannes
Attachment:
signature.asc
Description: This is a digitally signed message part