On Mon, 28 Aug 2006 13:57:28 -0700, mabbas wrote:
> diff --git a/net/d80211/ieee80211_sta.c b/net/d80211/ieee80211_sta.c
> index a933d92..374193e 100644
> --- a/net/d80211/ieee80211_sta.c
> +++ b/net/d80211/ieee80211_sta.c
> @@ -1543,8 +1543,6 @@ #endif
>       bss->channel = channel;
>       bss->freq = local->conf.freq;
>       if (channel != local->conf.channel &&
> -         (local->conf.phymode == MODE_IEEE80211G ||
> -          local->conf.phymode == MODE_IEEE80211B) &&
>           channel >= 1 && channel <= 14) {

The patch doesn't look correct. As I already wrote, no hackish guessing of
the phymode from channel numbers, please.

Are there any problems with just using rx_status->phymode instead of
local->conf.phymode (other than "drivers don't set that field" as that can
be easily fixed)?

Thanks,

 Jiri

-- 
Jiri Benc
SUSE Labs
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to