Holger Schurig wrote: > On Tuesday 28 July 2009 18:27:54 Dave wrote: >> You can also eliminate orinoco_ioctl_setfreq from wext.c with >> this change. > > You mean this ?!? Yes, except: > @@ -1481,7 +1432,6 @@ > static const iw_handler orinoco_handler[] = { > STD_IW_HANDLER(SIOCSIWCOMMIT, orinoco_ioctl_commit), > STD_IW_HANDLER(SIOCGIWNAME, cfg80211_wext_giwname), > - STD_IW_HANDLER(SIOCSIWFREQ, orinoco_ioctl_setfreq), + STD_IW_HANDLER(SIOCSIWFREQ, cfg80211_wext_setfreq ish), > STD_IW_HANDLER(SIOCGIWFREQ, orinoco_ioctl_getfreq), > STD_IW_HANDLER(SIOCSIWMODE, cfg80211_wext_siwmode), > STD_IW_HANDLER(SIOCGIWMODE, cfg80211_wext_giwmode), > > I tried, but with then an "iwconfig eth1 channel 1" gave me an > error. I did not investigate this further. And as Johannes pointed out, we need to have the non-cfg80211 version so adhoc connections can still be made via wext. So the original patch is fine as is. Ack from me. Dave. -- 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