summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrea Merello2008-05-10 13:32:34 +0200
committerJohn W. Linville2008-05-28 22:43:39 +0200
commit0a0ab41e833c8184c6d4ab663f137d5bbd50e049 (patch)
treef39473769febd13131a7cd2234a4ae11c7132fe5
parentrtl8180: fix wrong parameter in sa2400_rf_set_channel (diff)
downloadkernel-qcow2-linux-0a0ab41e833c8184c6d4ab663f137d5bbd50e049.tar.gz
kernel-qcow2-linux-0a0ab41e833c8184c6d4ab663f137d5bbd50e049.tar.xz
kernel-qcow2-linux-0a0ab41e833c8184c6d4ab663f137d5bbd50e049.zip
rtl8180: fix wrong parameter in max2820_rf_set_channel
The max2820 RF code needs to invoke max2820_write_phy_antenna every time the channel is being switch. This should be done passing the channel number to that function. Incorrectly we were passing the same value that is written on the channel RF register. This may cause problems when operating on ch 14. This patch fixes it. Thanks to Alessandro Di Marco who found this issue! Signed-off-by: Andrea Merello <andreamrl@tiscali.it> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/rtl8180_max2820.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/rtl8180_max2820.c b/drivers/net/wireless/rtl8180_max2820.c
index a140c802264a..6c825fd7f3b6 100644
--- a/drivers/net/wireless/rtl8180_max2820.c
+++ b/drivers/net/wireless/rtl8180_max2820.c
@@ -88,7 +88,7 @@ static void max2820_rf_set_channel(struct ieee80211_hw *dev,
* sa2400, for MAXIM we do this directly from BB */
rtl8180_write_phy(dev, 3, txpw);
- max2820_write_phy_antenna(dev, chan);
+ max2820_write_phy_antenna(dev, channel);
write_max2820(dev, 3, chan);
}