staging: rtl8188eu: make some arrays static const
authorMichael Straube <straube.linux@gmail.com>
Sun, 24 May 2020 10:15:14 +0000 (12:15 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 27 May 2020 08:16:34 +0000 (10:16 +0200)
Make some arrays in phy_iq_calibrate() static const and adjust
the functions that take these arrays as parameters accordingly.
Reduces object file size by 84 bytes (GCC 9.3.1 x86_64).

Suggested-by: Joe Perches <joe@perches.com>
Signed-off-by: Michael Straube <straube.linux@gmail.com>
Link: https://lore.kernel.org/r/20200524101514.20557-2-straube.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/hal/phy.c

index a0522d6764d6ff8239c0804025e3ec062aac7122..920688fc9e9fdb5c41d2f9aa8baecd4217456586 100644 (file)
@@ -786,7 +786,7 @@ static void pathb_fill_iqk(struct adapter *adapt, bool iqkok, s32 result[][8],
        }
 }
 
-static void save_adda_registers(struct adapter *adapt, u32 *addareg,
+static void save_adda_registers(struct adapter *adapt, const u32 *addareg,
                                u32 *backup, u32 register_num)
 {
        u32 i;
@@ -795,7 +795,7 @@ static void save_adda_registers(struct adapter *adapt, u32 *addareg,
                backup[i] = phy_query_bb_reg(adapt, addareg[i], bMaskDWord);
 }
 
-static void save_mac_registers(struct adapter *adapt, u32 *mac_reg,
+static void save_mac_registers(struct adapter *adapt, const u32 *mac_reg,
                               u32 *backup)
 {
        u32 i;
@@ -806,7 +806,7 @@ static void save_mac_registers(struct adapter *adapt, u32 *mac_reg,
        backup[i] = usb_read32(adapt, mac_reg[i]);
 }
 
-static void reload_adda_reg(struct adapter *adapt, u32 *adda_reg,
+static void reload_adda_reg(struct adapter *adapt, const u32 *adda_reg,
                            u32 *backup, u32 regiester_num)
 {
        u32 i;
@@ -815,8 +815,8 @@ static void reload_adda_reg(struct adapter *adapt, u32 *adda_reg,
                phy_set_bb_reg(adapt, adda_reg[i], bMaskDWord, backup[i]);
 }
 
-static void reload_mac_registers(struct adapter *adapt,
-                                u32 *mac_reg, u32 *backup)
+static void reload_mac_registers(struct adapter *adapt, const u32 *mac_reg,
+                                u32 *backup)
 {
        u32 i;
 
@@ -826,7 +826,7 @@ static void reload_mac_registers(struct adapter *adapt,
        usb_write32(adapt, mac_reg[i], backup[i]);
 }
 
-static void path_adda_on(struct adapter *adapt, u32 *adda_reg,
+static void path_adda_on(struct adapter *adapt, const u32 *adda_reg,
                         bool is_path_a_on, bool is2t)
 {
        u32 path_on;
@@ -844,7 +844,8 @@ static void path_adda_on(struct adapter *adapt, u32 *adda_reg,
                phy_set_bb_reg(adapt, adda_reg[i], bMaskDWord, path_on);
 }
 
-static void mac_setting_calibration(struct adapter *adapt, u32 *mac_reg, u32 *backup)
+static void mac_setting_calibration(struct adapter *adapt, const u32 *mac_reg,
+                                   u32 *backup)
 {
        u32 i = 0;
 
@@ -952,7 +953,7 @@ static void phy_iq_calibrate(struct adapter *adapt, s32 result[][8],
        struct odm_dm_struct *dm_odm = &adapt->HalData->odmpriv;
        u32 i;
        u8 path_a_ok, path_b_ok;
-       u32 adda_reg[IQK_ADDA_REG_NUM] = {
+       static const u32 adda_reg[IQK_ADDA_REG_NUM] = {
                rFPGA0_XCD_SwitchControl, rBlue_Tooth,
                rRx_Wait_CCA, rTx_CCK_RFON,
                rTx_CCK_BBON, rTx_OFDM_RFON,
@@ -962,12 +963,12 @@ static void phy_iq_calibrate(struct adapter *adapt, s32 result[][8],
                rRx_TO_Rx, rStandby,
                rSleep, rPMPD_ANAEN
        };
-       u32 iqk_mac_reg[IQK_MAC_REG_NUM] = {
+       static const u32 iqk_mac_reg[IQK_MAC_REG_NUM] = {
                REG_TXPAUSE, REG_BCN_CTRL,
                REG_BCN_CTRL_1, REG_GPIO_MUXCFG
        };
        /* since 92C & 92D have the different define in IQK_BB_REG */
-       u32 iqk_bb_reg_92c[IQK_BB_REG_NUM] = {
+       static const u32 iqk_bb_reg_92c[IQK_BB_REG_NUM] = {
                rOFDM0_TRxPathEnable, rOFDM0_TRMuxPar,
                rFPGA0_XCD_RFInterfaceSW, rConfig_AntA, rConfig_AntB,
                rFPGA0_XAB_RFInterfaceSW, rFPGA0_XA_RFInterfaceOE,