| From 1288aa4e80145d9f4196df32f717b4c1cf6aab61 Mon Sep 17 00:00:00 2001 |
| From: Larry Finger <Larry.Finger@lwfinger.net> |
| Date: Sat, 9 Jul 2011 13:15:58 -0500 |
| Subject: rtlwifi: rtl8192cu: Fix duplicate if test |
| |
| From: Larry Finger <Larry.Finger@lwfinger.net> |
| |
| commit 1288aa4e80145d9f4196df32f717b4c1cf6aab61 upstream. |
| |
| A typo causes routine rtl92cu_phy_rf6052_set_cck_txpower() to test the |
| same condition twice. The problem was found using cppcheck-1.49, and the |
| proper fix was verified against the pre-mac80211 version of the code. |
| |
| Reported-by: David Binderman <dcb314@hotmail.com> |
| Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net> |
| Signed-off-by: John W. Linville <linville@tuxdriver.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> |
| |
| --- |
| drivers/net/wireless/rtlwifi/rtl8192cu/rf.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c |
| +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/rf.c |
| @@ -104,7 +104,7 @@ void rtl92cu_phy_rf6052_set_cck_txpower( |
| tx_agc[RF90_PATH_A] = 0x10101010; |
| tx_agc[RF90_PATH_B] = 0x10101010; |
| } else if (rtlpriv->dm.dynamic_txhighpower_lvl == |
| - TXHIGHPWRLEVEL_LEVEL1) { |
| + TXHIGHPWRLEVEL_LEVEL2) { |
| tx_agc[RF90_PATH_A] = 0x00000000; |
| tx_agc[RF90_PATH_B] = 0x00000000; |
| } else{ |
