There's no need to keep the same for loop twice in the code.
Move the txpower cap before the loop to reduce code complexity.

Signed-off-by: Helmut Schaa <helmut.sc...@googlemail.com>
---
 drivers/net/wireless/ath/ath9k/ar9003_phy.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/ar9003_phy.c 
b/drivers/net/wireless/ath/ath9k/ar9003_phy.c
index 2f15cbc..81ab3ca 100644
--- a/drivers/net/wireless/ath/ath9k/ar9003_phy.c
+++ b/drivers/net/wireless/ath/ath9k/ar9003_phy.c
@@ -1844,13 +1844,9 @@ static void ar9003_hw_tx99_set_txpower(struct ath_hw 
*ah, u8 txpower)
        static u8 p_pwr_array[ar9300RateSize] = { 0 };
        unsigned int i;
 
-       if (txpower <= MAX_RATE_POWER) {
-               for (i = 0; i < ar9300RateSize; i++)
-                       p_pwr_array[i] = txpower;
-       } else {
-               for (i = 0; i < ar9300RateSize; i++)
-                       p_pwr_array[i] = MAX_RATE_POWER;
-       }
+       txpower = txpower <= MAX_RATE_POWER ? txpower : MAX_RATE_POWER;
+       for (i = 0; i < ar9300RateSize; i++)
+               p_pwr_array[i] = txpower;
 
        ar9003_hw_tx_power_regwrite(ah, p_pwr_array);
 }
-- 
2.8.1

_______________________________________________
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel

Reply via email to