2011-02-18 17:09:06

by Nickolay Ledovskikh

Subject: [PATCH] ath5k: Correct channel setting for AR2317 chip

Correct channel setting function must be used for AR2317.
When I tested ahb patch on bullet2 all seemed to work fine,
but it couldn't connect another host (using ibss for example).
During an analysis I observed that it's transmitting on another
channel. I looked into madwifi code and understood that
the problem is in channel setting function. So atheros RF2317 not
fully handled in the current ath5k version and must be patched.

Signed-off-by: Nikolay Ledovskikh <[email protected]>
---
diff --git a/drivers/net/wireless/ath/ath5k/phy.c b/drivers/net/wireless/ath/ath5k/phy.c
index 78c26fd..c44111f 100644
--- a/drivers/net/wireless/ath/ath5k/phy.c
+++ b/drivers/net/wireless/ath/ath5k/phy.c
@@ -1253,6 +1253,7 @@ static int ath5k_hw_channel(struct ath5k_hw *ah,
case AR5K_RF5111:
ret = ath5k_hw_rf5111_channel(ah, channel);
break;
+ case AR5K_RF2317:
case AR5K_RF2425:
ret = ath5k_hw_rf2425_channel(ah, channel);
break;

2011-02-18 18:27:24

by Bob Copeland

Subject: Re: [PATCH] ath5k: Correct channel setting for AR2317 chip

On Fri, Feb 18, 2011 at 07:59:53PM +0300, Nikolay Ledovskikh wrote:
> Correct channel setting function must be used for AR2317.
> When I tested ahb patch on bullet2 all seemed to work fine,
> but it couldn't connect another host (using ibss for example).
> During an analysis I observed that it's transmitting on another
> channel. I looked into madwifi code and understood that
> the problem is in channel setting function. So atheros RF2317 not
> fully handled in the current ath5k version and must be patched.
>
> Signed-off-by: Nikolay Ledovskikh <[email protected]>

Thanks!

Acked-by: Bob Copeland <[email protected]>

--
Bob Copeland %% http://www.bobcopeland.com