On Mon, 2008-03-03 at 15:08 +0800, Zhu Yi wrote: > On Fri, 2008-02-29 at 17:25 +0100, Johannes Berg wrote: > > now that John merged more stuff you should only need the hostapd > > patch and the allow-ap-vlan-modes patch from my kernel series :) > > OK. This time is much better. Now I can at least ping my 4965 AP with > below fix. > > The removal of mac80211 IEEE80211_CHAN_W_SCAN flag causes channel flag > incompatibility between hostapd and driver. Hostapd expects > HOSTAPD_CHAN_W_SCAN (0x1) to be set for the configured channel while 0x1 > means IEEE80211_CHAN_DISABLED in the new mac80211. We should make > ieee80211_channel_flags and HOSTAPD_CHAN_W_xxx compatible. What do you > think? I now just workaround the problem with: > > diff --git a/hostapd/hw_features.c b/hostapd/hw_features.c > index c58e1e4..8a7b013 100644 > --- a/hostapd/hw_features.c > +++ b/hostapd/hw_features.c > @@ -294,8 +294,7 @@ static int select_hw_mode1(struct hostapd_iface > *iface) > for (j = 0; j < iface->current_mode->num_channels; j++) { > struct hostapd_channel_data *chan = > &iface->current_mode->channels[j]; > - if ((chan->flag & HOSTAPD_CHAN_W_SCAN) && > - (chan->chan == iface->conf->channel)) { > + if (chan->chan == iface->conf->channel) { > ok = 1; > break; > } That's rather odd. ieee80211_channel_flags and HOSTAPD_CHAN_W_xx need not be compatible because in driver_nl80211.c, phy_info_handler(), they're translated into each other. What exactly is your setup? johannes
Attachment:
signature.asc
Description: This is a digitally signed message part