Re: [PATCH 1/8] d80211/bcm43xx: Modify struct ieee80211_rx_status for wireless statistics.

2006-08-09 Thread Jouni Malinen
On Wed, Aug 09, 2006 at 10:21:16AM -0500, Larry Finger wrote: > Dan Williams wrote: > >> int ssi; > >>+ int maxssi; > >Spacing? Looks like you're using real tabs here, but the current code > >uses 8 spaces instead. Not sure if that's a problem. > > I am using tabs, which I think is t

Re: [PATCH 1/8] d80211/bcm43xx: Modify struct ieee80211_rx_status for wireless statistics.

2006-08-09 Thread Larry Finger
Dan Williams wrote: On Tue, 2006-08-08 at 18:58 -0500, Larry Finger wrote: diff --git a/include/net/d80211.h b/include/net/d80211.h index ba5cb4c..8479b58 100644 --- a/include/net/d80211.h +++ b/include/net/d80211.h @@ -205,6 +205,9 @@ struct ieee80211_rx_status { int channel;

Re: [PATCH 1/8] d80211/bcm43xx: Modify struct ieee80211_rx_status for wireless statistics.

2006-08-09 Thread Dan Williams
On Tue, 2006-08-08 at 18:58 -0500, Larry Finger wrote: > Part 1 of 8 to add wireless statistics to the bcm43xx-d80211 system. This > patch adds new variables > to struct ieee80211_rx_status to contain the latest values for signal, noise, > and the maximum value > of the received ssi. These could

[PATCH 1/8] d80211/bcm43xx: Modify struct ieee80211_rx_status for wireless statistics.

2006-08-08 Thread Larry Finger
Part 1 of 8 to add wireless statistics to the bcm43xx-d80211 system. This patch adds new variables to struct ieee80211_rx_status to contain the latest values for signal, noise, and the maximum value of the received ssi. These could be u8's, but I left them as integers in case other drivers need