Use Hal_GetChnlGroup88E() instead of duplicating it's code
in get_rx_power_val_by_reg().

Signed-off-by: Michael Straube <straube.li...@gmail.com>
---
 drivers/staging/rtl8188eu/hal/rf.c              | 17 +++--------------
 .../staging/rtl8188eu/hal/rtl8188e_hal_init.c   |  2 +-
 .../staging/rtl8188eu/include/rtl8188e_hal.h    |  2 ++
 3 files changed, 6 insertions(+), 15 deletions(-)

diff --git a/drivers/staging/rtl8188eu/hal/rf.c 
b/drivers/staging/rtl8188eu/hal/rf.c
index 81e30a1a6bfd..6fe4daea8fd5 100644
--- a/drivers/staging/rtl8188eu/hal/rf.c
+++ b/drivers/staging/rtl8188eu/hal/rf.c
@@ -164,20 +164,9 @@ static void get_rx_power_val_by_reg(struct adapter *adapt, 
u8 channel,
                        /*  increase power diff defined by Realtek for 
regulatory */
                        if (hal_data->pwrGroupCnt == 1)
                                chnlGroup = 0;
-                       if (hal_data->pwrGroupCnt >= hal_data->PGMaxGroup) {
-                               if (channel < 3)
-                                       chnlGroup = 0;
-                               else if (channel < 6)
-                                       chnlGroup = 1;
-                               else if (channel < 9)
-                                       chnlGroup = 2;
-                               else if (channel < 12)
-                                       chnlGroup = 3;
-                               else if (channel < 14)
-                                       chnlGroup = 4;
-                               else if (channel == 14)
-                                       chnlGroup = 5;
-                       }
+                       if (hal_data->pwrGroupCnt >= hal_data->PGMaxGroup)
+                               Hal_GetChnlGroup88E(channel, &chnlGroup);
+
                        write_val = 
hal_data->MCSTxPowerLevelOriginalOffset[chnlGroup][index+(rf ? 8 : 0)] +
                                        ((index < 2) ? powerbase0[rf] : 
powerbase1[rf]);
                        break;
diff --git a/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c 
b/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c
index 31e80d693f32..d5a1cf92ca6f 100644
--- a/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c
+++ b/drivers/staging/rtl8188eu/hal/rtl8188e_hal_init.c
@@ -373,7 +373,7 @@ static void Hal_ReadPowerValueFromPROM_8188E(struct 
txpowerinfo24g *pwrInfo24G,
        }
 }
 
-static void Hal_GetChnlGroup88E(u8 chnl, u8 *group)
+void Hal_GetChnlGroup88E(u8 chnl, u8 *group)
 {
        if (chnl < 3)                   /*  Channel 1-2 */
                *group = 0;
diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_hal.h 
b/drivers/staging/rtl8188eu/include/rtl8188e_hal.h
index a86b07d3c82a..eb4655ecf6e0 100644
--- a/drivers/staging/rtl8188eu/include/rtl8188e_hal.h
+++ b/drivers/staging/rtl8188eu/include/rtl8188e_hal.h
@@ -329,6 +329,8 @@ struct hal_data_8188e {
        u8      UsbRxAggPageTimeout;
 };
 
+void Hal_GetChnlGroup88E(u8 chnl, u8 *group);
+
 /*  rtl8188e_hal_init.c */
 void _8051Reset88E(struct adapter *padapter);
 void rtl8188e_InitializeFirmwareVars(struct adapter *padapter);
-- 
2.19.2

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to