Return-path: Received: from mail.candelatech.com ([208.74.158.172]:47958 "EHLO ns3.lanforge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754528Ab2CTQrm (ORCPT ); Tue, 20 Mar 2012 12:47:42 -0400 Message-ID: <4F68B4AA.3070306@candelatech.com> (sfid-20120320_174752_494070_07E777BE) Date: Tue, 20 Mar 2012 09:47:38 -0700 From: Ben Greear MIME-Version: 1.0 To: Florian Fainelli CC: linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH v3 4/4] ath9k: Support ethtool getstats api. References: <1332183105-4447-1-git-send-email-greearb@candelatech.com> <1332183105-4447-4-git-send-email-greearb@candelatech.com> <4F684301.9040006@openwrt.org> In-Reply-To: <4F684301.9040006@openwrt.org> Content-Type: text/plain; charset=UTF-8; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 03/20/2012 01:42 AM, Florian Fainelli wrote: > Hi, > > Le 03/19/12 19:51, greearb@candelatech.com a écrit : >> From: Ben Greear >> >> This returns many of the values that formerly could >> only be obtained from debugfs. This should be an >> improvement when trying to access these counters >> programatically. Currently this support is only >> enabled when DEBUGFS is enabled because otherwise >> these stats are not accumulated. >> >> Signed-off-by: Ben Greear >> --- >> >> v3: No changes from v2 >> >> :100644 100644 4a00806... 7261f88... M drivers/net/wireless/ath/ath9k/main.c >> drivers/net/wireless/ath/ath9k/main.c | 134 +++++++++++++++++++++++++++++++++ >> 1 files changed, 134 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c >> index 4a00806..7261f88 100644 >> --- a/drivers/net/wireless/ath/ath9k/main.c >> +++ b/drivers/net/wireless/ath/ath9k/main.c >> @@ -2430,6 +2430,134 @@ static int ath9k_get_antenna(struct ieee80211_hw *hw, u32 *tx_ant, u32 *rx_ant) >> return 0; >> } >> >> +#ifdef CONFIG_ATH9K_DEBUGFS > > These are ethtool knobs, so you might want to introduce a new config symbol dedicated to it, which depends on CONFIG_ATH9K_DEBUGFS eventually. I'd like to gather at least most stats always, so ethtool can work regardless of debugfs. But, that can be follow on patches in my opinion. If it turns out that we need another config option for this, then that is fine too. Thanks, Ben -- Ben Greear Candela Technologies Inc http://www.candelatech.com