Date: Fri, 26 Aug 2011 10:34:34 +0530
From: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
To: Felix Fietkau <nbd@openwrt.org>
CC: <linville@tuxdriver.com>, <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH 1/2] ath9k_hw: Fix rx latency of 11a mode
Message-ID: <20110826050415.GA7416@vmraj-lnx.users.atheros.com> (sfid-20110826_070412_927378_84E6736E)
References: <1314283522-6143-1-git-send-email-rmanohar@qca.qualcomm.com>
 <4E566AFA.8050203@openwrt.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
In-Reply-To: <4E566AFA.8050203@openwrt.org>
Sender: linux-wireless-owner@vger.kernel.org

On Thu, Aug 25, 2011 at 05:32:10PM +0200, Felix Fietkau wrote:
> On 2011-08-25 4:45 PM, Rajkumar Manoharan wrote:
> >Rx latecy to start signal(usec) of 11a is 41 not 37 and
> >also corrected the rx delay in quarter rate.
> >
> >Signed-off-by: Rajkumar Manoharan<rmanohar@qca.qualcomm.com>
> >---
> >  drivers/net/wireless/ath/ath9k/hw.c |    7 +++++--
> >  1 files changed, 5 insertions(+), 2 deletions(-)
> >
> >diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
> >index 88100cc..d4dce6b 100644
> >--- a/drivers/net/wireless/ath/ath9k/hw.c
> >+++ b/drivers/net/wireless/ath/ath9k/hw.c
> >@@ -974,7 +974,10 @@ void ath9k_hw_init_global_settings(struct ath_hw *ah)
> >  	if (ah->misc_mode != 0)
> >  		REG_SET_BIT(ah, AR_PCU_MISC, ah->misc_mode);
> >
> >-	rx_lat = 37;
> >+	if (IS_CHAN_A_FAST_CLOCK(ah, chan))
> >+		rx_lat = 41;
> >+	else
> >+		rx_lat = 37;
> >  	tx_lat = 54;
> >
> >  	if (IS_CHAN_HALF_RATE(chan)) {
> >@@ -988,7 +991,7 @@ void ath9k_hw_init_global_settings(struct ath_hw *ah)
> >  		sifstime = 32;
> >  	} else if (IS_CHAN_QUARTER_RATE(chan)) {
> >  		eifs = 340;
> >-		rx_lat *= 4;
> >+		rx_lat *= 4 - 1;
> That looks a bit weird. Did you mean to change it to *= 3 or was
> this just mistyped because you wanted to subtract one after the
> multiplication?
>
Oops.. it is rx_lat = (rx_lat * 4) - 1.

--
Rajkumar