On 2012-12-03 7:18 AM, Saravana wrote: > On 12/02/2012 06:08 PM, Felix Fietkau wrote: >> On 2012-11-29 3:24 PM, Saravana wrote: >>> Add a debugfs file showing the rate at which >>> the last packet is received. >>> >>> Signed-off-by: Saravana <saravanad@xxxxxxxxxxx> >> What's the point of adding debugfs files for data which is already >> available via nl80211? >> > The debugfs is a repository of statistics. Many of the statistics like > tx/rx bytes/packets, retries, signal,power etc which are available via nl80211(iw) > are already available in debugfs. For the sake of completeness and to maintain > uniformity the last rx rate is added to debugfs. I think debugfs does not have (or need) completeness and uniformity, nor is it meant as a repository of statistics. It is to be used for various debugging purposes (as the name implies). I think any additions of info to debugfs that are also available through regular APIs should be considered unnecessary bloat and dropped. - Felix -- 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