* [PATCH] ath5k: retries in ath5k_hw_setup_4word_tx_desc
@ 2010-03-15 20:00 Andrew Blaich
2010-03-16 12:12 ` Bob Copeland
0 siblings, 1 reply; 2+ messages in thread
From: Andrew Blaich @ 2010-03-15 20:00 UTC (permalink / raw)
To: Bob Copeland
Cc: John W. Linville, linux-wireless, Jiri Slaby, Nick Kossifidis,
Luis R. Rodriguez
Removing AR5K_TUNE_HWTXTRIES from the Ath5k code, it adds 4
retries to tx_tries0. By removing AR5K_TUNE_HWTXTRIES, the rate
control algorithm is given full control over the retry count.
Signed-off-by:: Andrew Blaich <ablaich@gmail.com>
---
diff --git a/drivers/net/wireless/ath/ath5k/ath5k.h
b/drivers/net/wireless/ath/ath5k/ath5k.h
index ac67f02..1c5834a 100644
--- a/drivers/net/wireless/ath/ath5k/ath5k.h
+++ b/drivers/net/wireless/ath/ath5k/ath5k.h
@@ -202,7 +202,6 @@
#define AR5K_TUNE_MAX_TXPOWER 63
#define AR5K_TUNE_DEFAULT_TXPOWER 25
#define AR5K_TUNE_TPC_TXPOWER false
-#define AR5K_TUNE_HWTXTRIES 4
#define AR5K_INIT_CARR_SENSE_EN 1
diff --git a/drivers/net/wireless/ath/ath5k/desc.c
b/drivers/net/wireless/ath/ath5k/desc.c
index dc30a2b..c18d8d4 100644
--- a/drivers/net/wireless/ath/ath5k/desc.c
+++ b/drivers/net/wireless/ath/ath5k/desc.c
@@ -229,7 +229,7 @@ static int ath5k_hw_setup_4word_tx_desc(struct ath5k_hw *ah,
AR5K_REG_SM(antenna_mode, AR5K_4W_TX_DESC_CTL0_ANT_MODE_XMIT);
tx_ctl->tx_control_1 |= AR5K_REG_SM(type,
AR5K_4W_TX_DESC_CTL1_FRAME_TYPE);
- tx_ctl->tx_control_2 = AR5K_REG_SM(tx_tries0 + AR5K_TUNE_HWTXTRIES,
+ tx_ctl->tx_control_2 = AR5K_REG_SM(tx_tries0,
AR5K_4W_TX_DESC_CTL2_XMIT_TRIES0);
tx_ctl->tx_control_3 = tx_rate0 & AR5K_4W_TX_DESC_CTL3_XMIT_RATE0;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ath5k: retries in ath5k_hw_setup_4word_tx_desc
2010-03-15 20:00 [PATCH] ath5k: retries in ath5k_hw_setup_4word_tx_desc Andrew Blaich
@ 2010-03-16 12:12 ` Bob Copeland
0 siblings, 0 replies; 2+ messages in thread
From: Bob Copeland @ 2010-03-16 12:12 UTC (permalink / raw)
To: Andrew Blaich
Cc: John W. Linville, linux-wireless, Jiri Slaby, Nick Kossifidis,
Luis R. Rodriguez
On Mon, Mar 15, 2010 at 04:00:49PM -0400, Andrew Blaich wrote:
> Removing AR5K_TUNE_HWTXTRIES from the Ath5k code, it adds 4
> retries to tx_tries0. By removing AR5K_TUNE_HWTXTRIES, the rate
> control algorithm is given full control over the retry count.
>
> Signed-off-by:: Andrew Blaich <ablaich@gmail.com>
I believe John has already picked up the patch into wireless-testing.
Thanks for the submission!
--
Bob Copeland %% www.bobcopeland.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-03-16 12:13 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-15 20:00 [PATCH] ath5k: retries in ath5k_hw_setup_4word_tx_desc Andrew Blaich
2010-03-16 12:12 ` Bob Copeland
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).