On Tue, 22 Aug 2006 16:15:47 -0500, Larry Finger wrote: > No it cannot change between frames; however, the max value can be very > different for different > drivers using d80211. On the bcm43xx, it is 60; whereas 100 seems to be a > better value for the > rt2x00 chips. Adding it here seemed like a good way to handle this situation. > Do you suggest > something else?
As Johannes already said, it should be in ieee80211_hw. > >> [...] > >> --- a/net/d80211/ieee80211_i.h > >> +++ b/net/d80211/ieee80211_i.h > >> @@ -337,6 +337,9 @@ struct ieee80211_local { > >> struct net_device *apdev; /* wlan#ap - management frames (hostapd) */ > >> int open_count; > >> int monitors; > >> + int link_quality; > >> + int noise; > >> + struct iw_statistics wstats; > > > > Why are these three variables in ieee80211_local? They are not used > > anywhere. > > You are right about the first two; however, wstats is used in the new > routine ieee80211_get_wireless_stats. Oh, get_wireless_stats returns struct iw_statistics, so it needs to be allocated all the time. Grr, that's stupid... Okay, wstats really needs to be in ieee80211_local then. 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