On Fri, 2009-10-23 at 16:19 +0200, Johannes Berg wrote: > On Thu, 2009-10-22 at 15:31 +0200, Holger Schurig wrote: > > > +#ifdef CONFIG_LIBERTAS_WEXT > > dev->ethtool_ops = &lbs_ethtool_ops; > > dev->wireless_handlers = &lbs_handler_def; > > #endif > > So here you make the wireless_handlers optional. > > I still don't understand why you can't just replace only some of the > handlers, i.e. instead of doing this, simply do > > - (iw_handler) lbs_set_freq, /* SIOCSIWFREQ */ > + (iw_handler) cfg80211_wext_siwfreq, > > etc. > > That's why those exist and are exported. And that way you don't need the > Kconfig at all, and can remove most of the wext code. I *think* because Holger is trying to keep WEXT around for a bit, because cfg80211 won't be mature enough yet (IMHO, I could be wrong) by 2.6.33, and because it doesn't support the OLPC-style mesh stuff and the libertas monitor interface code. There's a few loose-ends that need to be tied up before a cfg80211 conversion would completely replace the functionality of the WEXT code. Dan -- 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
