Return-path: Received: from nebensachen.de ([195.34.83.29]:60334 "EHLO mail.nebensachen.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751663AbYJZUrq (ORCPT ); Sun, 26 Oct 2008 16:47:46 -0400 From: Elias Oltmanns To: "Nick Kossifidis" Cc: "John W. Linville" , "Jiri Slaby" , "Luis R. Rodriguez" , linux-wireless@vger.kernel.org Subject: Re: [PATCH] ath5k: Fix reset sequence for AR5212 in general and RF5111 in particular References: <87prlr9mna.fsf@denkblock.local> <40f31dec0810251621l3d720a69mf21b5253e9135010@mail.gmail.com> <87abcrtfa8.fsf@denkblock.local> <40f31dec0810261132w2747d66em2a2a53ff9e1ecc04@mail.gmail.com> Date: Sun, 26 Oct 2008 21:47:25 +0100 Message-ID: <87wsfvt76q.fsf@denkblock.local> (sfid-20081026_214753_546080_121969CE) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-wireless-owner@vger.kernel.org List-ID: "Nick Kossifidis" wrote: > 2008/10/26 Elias Oltmanns : >> > >> Let me quote from legacy-hal: >> >> ar5212/ar5212_reset.c: >> 373: OS_REG_RMW_FIELD(ah, AR_PHY_DAG_CTRLCCK, >> 374: AR_PHY_DAG_CTRLCCK_RSSI_THR, 2); >> >> ah_internal.h: >> 598: #define OS_REG_RMW_FIELD(_a, _r, _f, _v) \ >> 599: OS_REG_WRITE(_a, _r, \ >> 600: (OS_REG_READ(_a, _r) &~ _f) | (((_v) << _f##_S) & _f)) >> >> Please note the last argument to OS_REG_RMW_FIELD() which is 2 in this >> case. Unless I've made a mistake, this translates into the >> >> data |= 0x00000800; >> >> in my patch. >> >> There definitely is no way for me to connect to an ap (or even get a >> list of aps) with current 2.6.27.4. Perhaps we'll hear something from >> Nils on that matter too. >> > > Some lines above... > > /* Overwrite INI values for revised chipsets */ > if (AH_PRIVATE(ah)->ah_phyRev >= AR_PHY_CHIP_ID_REV_2) { > > so > > /* Add barker RSSI thresh enable as disabled */ > OS_REG_CLR_BIT(ah, AR_PHY_DAG_CTRLCCK, > AR_PHY_DAG_CTRLCCK_EN_RSSI_THR); > OS_REG_RMW_FIELD(ah, AR_PHY_DAG_CTRLCCK, > AR_PHY_DAG_CTRLCCK_RSSI_THR, 2); > > is for phy (BB) revisions > 2 since you have RF5111 this code should > not run but (and here is our bug) instead of checking bb revision we > run this code for all AR5212 chips > > /* > * Write some more initial register settings > */ > if (ah->ah_version == AR5K_AR5212) { > [...] > So can you please change the check on ath5k_hw_reset + add initial > rssi threshold setting and retry ? Is this what you had in mind? It works for me. Regards, Elias -------- From: Elias Oltmanns Subject: ath5k: Fix reset sequence for AR5212 in general and RF5111 in particular Take care to handle register 0xa228 exactly as in the HAL released by Atheros. This change is required to make ath5k work again on my system since commit 2203d6be (ath5k: Misc hw_reset updates), thus fixing a regression in 2.6.27 and therefore hopefully eligible for inclusion into a stable release. v2: Only overwrite initial register values on later revisions of AR5212 chips. Cc: stable Signed-off-by: Elias Oltmanns --- drivers/net/wireless/ath5k/hw.c | 4 +++- drivers/net/wireless/ath5k/initvals.c | 2 ++ 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/drivers/net/wireless/ath5k/hw.c b/drivers/net/wireless/ath5k/hw.c index ad1a5b4..68cf2ae 100644 --- a/drivers/net/wireless/ath5k/hw.c +++ b/drivers/net/wireless/ath5k/hw.c @@ -828,7 +828,8 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum ieee80211_if_types op_mode, /* * Write some more initial register settings */ - if (ah->ah_version == AR5K_AR5212) { + if (ah->ah_version == AR5K_AR5212 && + ah->ah_radio > AR5K_RF5111) { ath5k_hw_reg_write(ah, 0x0002a002, 0x982c); if (channel->hw_value == CHANNEL_G) @@ -858,6 +859,7 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum ieee80211_if_types op_mode, data = ath5k_hw_reg_read(ah, 0xa228); data &= 0xfffe03ff; + data |= 0x00000800; ath5k_hw_reg_write(ah, data, 0xa228); data = 0; diff --git a/drivers/net/wireless/ath5k/initvals.c b/drivers/net/wireless/ath5k/initvals.c index 2806b21..cf7ebd1 100644 --- a/drivers/net/wireless/ath5k/initvals.c +++ b/drivers/net/wireless/ath5k/initvals.c @@ -810,6 +810,8 @@ static const struct ath5k_ini_mode ar5212_rf5111_ini_mode_end[] = { { 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000 } }, { AR5K_PHY(642), { 0xd03e6788, 0xd03e6788, 0xd03e6788, 0xd03e6788, 0xd03e6788 } }, + { 0xa228, + { 0x000001b5, 0x000001b5, 0x000001b5, 0x000001b5, 0x000001b5 } }, { 0xa23c, { 0x13c889af, 0x13c889af, 0x13c889af, 0x13c889af, 0x13c889af } }, };