Thread (8 messages) 8 messages, 4 authors, 2011-08-30

Re: [PATCH 3.1] ath9k_hw: fix calibration on 5 ghz

From: Adrian Chadd <hidden>
Date: 2011-08-29 09:01:09

Can we please hold off of this until we know if this is indeed the correct fix?

Ie, I'm not sure yet whether 5ghz fast clock (which clocks the mac/bb
like it was doing 2.4ghz) needs the dual ADCs enabled or not.
It may be that something is being incorrectly disabled when the
fastclock initvals are being written over the normal 5ghz values.

Thanks,


Adrian

On 29 August 2011 16:06, Felix Fietkau [off-list ref] wrote:
quoted hunk ↗ jump to hunk
ADC calibrations cannot run on 5 GHz with fast clock enabled. They
need to be disabled, otherwise they'll hang and IQ mismatch calibration
will not be run either.

Signed-off-by: Felix Fietkau <redacted>
Reported-by: Adrian Chadd <redacted>
Cc: stable@kernel.org
---
 drivers/net/wireless/ath/ath9k/ar9002_calib.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/drivers/net/wireless/ath/ath9k/ar9002_calib.c b/drivers/net/wireless/ath/ath9k/ar9002_calib.c
index 2d4c091..2d394af 100644
--- a/drivers/net/wireless/ath/ath9k/ar9002_calib.c
+++ b/drivers/net/wireless/ath/ath9k/ar9002_calib.c
@@ -41,7 +41,8 @@ static bool ar9002_hw_is_cal_supported(struct ath_hw *ah,
       case ADC_DC_CAL:
               /* Run ADC Gain Cal for non-CCK & non 2GHz-HT20 only */
               if (!IS_CHAN_B(chan) &&
-                   !(IS_CHAN_2GHZ(chan) && IS_CHAN_HT20(chan)))
+                   !((IS_CHAN_2GHZ(chan) || IS_CHAN_A_FAST_CLOCK(ah, chan)) &&
+                     IS_CHAN_HT20(chan)))
                       supported = true;
               break;
       }
--
1.7.3.2
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help