staging/rtl8723bs: Fix some coding style issues in rtw_odm.c.
authorTom Gardi <gardi@gmx.at>
Sat, 12 Aug 2017 00:13:02 +0000 (02:13 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Aug 2017 22:57:17 +0000 (15:57 -0700)
WARNING: line over 80 characters
WARNING: static const char * array should probably be static const char * const
CHECK: Unnecessary parentheses around hal_data->odmpriv

Signed-off-by: Tom Gardi <gardi@gmx.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_odm.c

index 3144e8ec2fa215cb29600c3d6faa1d8f957b65a4..edbcaeb9f8c24bf1463d37b521d3b1ea9a539756 100644 (file)
@@ -18,7 +18,7 @@
 #include <rtw_odm.h>
 #include <hal_data.h>
 
-static const char *odm_comp_str[] = {
+static const char * const odm_comp_str[] = {
        /* BIT0 */"ODM_COMP_DIG",
        /* BIT1 */"ODM_COMP_RA_MASK",
        /* BIT2 */"ODM_COMP_DYNAMIC_TXPWR",
@@ -55,7 +55,7 @@ static const char *odm_comp_str[] = {
 
 #define RTW_ODM_COMP_MAX 32
 
-static const char *odm_ability_str[] = {
+static const char * const odm_ability_str[] = {
        /* BIT0 */"ODM_BB_DIG",
        /* BIT1 */"ODM_BB_RA_MASK",
        /* BIT2 */"ODM_BB_DYNAMIC_TXPWR",
@@ -87,7 +87,7 @@ static const char *odm_ability_str[] = {
 
 #define RTW_ODM_ABILITY_MAX 27
 
-static const char *odm_dbg_level_str[] = {
+static const char * const odm_dbg_level_str[] = {
        NULL,
        "ODM_DBG_OFF",
        "ODM_DBG_SERIOUS",
@@ -127,7 +127,8 @@ void rtw_odm_dbg_level_msg(void *sel, struct adapter *adapter)
        DBG_871X_SEL_NL(sel, "odm.DebugLevel = %u\n", dbg_level);
        for (i = 0; i < RTW_ODM_DBG_LEVEL_NUM; i++) {
                if (odm_dbg_level_str[i])
-                       DBG_871X_SEL_NL(sel, "%u %s\n", i, odm_dbg_level_str[i]);
+                       DBG_871X_SEL_NL(sel, "%u %s\n",
+                                       i, odm_dbg_level_str[i]);
        }
 }
 
@@ -161,20 +162,23 @@ void rtw_odm_adaptivity_parm_msg(void *sel, struct adapter *adapter)
        struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
        DM_ODM_T *odm = &pHalData->odmpriv;
 
-       DBG_871X_SEL_NL(sel, "%10s %16s %8s %10s %11s %14s\n"
-               , "TH_L2H_ini", "TH_EDCCA_HL_diff", "IGI_Base", "ForceEDCCA", "AdapEn_RSSI", "IGI_LowerBound");
-       DBG_871X_SEL_NL(sel, "0x%-8x %-16d 0x%-6x %-10d %-11u %-14u\n"
-               , (u8)odm->TH_L2H_ini
-               , odm->TH_EDCCA_HL_diff
-               , odm->IGI_Base
-               , odm->ForceEDCCA
-               , odm->AdapEn_RSSI
-               , odm->IGI_LowerBound
+       DBG_871X_SEL_NL(sel, "%10s %16s %8s %10s %11s %14s\n",
+                       "TH_L2H_ini", "TH_EDCCA_HL_diff", "IGI_Base",
+                       "ForceEDCCA", "AdapEn_RSSI", "IGI_LowerBound");
+       DBG_871X_SEL_NL(sel, "0x%-8x %-16d 0x%-6x %-10d %-11u %-14u\n",
+                       (u8)odm->TH_L2H_ini,
+                       odm->TH_EDCCA_HL_diff,
+                       odm->IGI_Base,
+                       odm->ForceEDCCA,
+                       odm->AdapEn_RSSI,
+                       odm->IGI_LowerBound
        );
 }
 
-void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_EDCCA_HL_diff,
-       s8 IGI_Base, bool ForceEDCCA, u8 AdapEn_RSSI, u8 IGI_LowerBound)
+void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini,
+                                s8 TH_EDCCA_HL_diff, s8 IGI_Base,
+                                bool ForceEDCCA, u8 AdapEn_RSSI,
+                                u8 IGI_LowerBound)
 {
        struct hal_com_data *pHalData = GET_HAL_DATA(adapter);
        DM_ODM_T *odm = &pHalData->odmpriv;
@@ -190,8 +194,8 @@ void rtw_odm_adaptivity_parm_set(struct adapter *adapter, s8 TH_L2H_ini, s8 TH_E
 void rtw_odm_get_perpkt_rssi(void *sel, struct adapter *adapter)
 {
        struct hal_com_data *hal_data = GET_HAL_DATA(adapter);
-       DM_ODM_T *odm = &(hal_data->odmpriv);
+       DM_ODM_T *odm = &hal_data->odmpriv;
 
        DBG_871X_SEL_NL(sel, "RxRate = %s, RSSI_A = %d(%%), RSSI_B = %d(%%)\n",
-       HDATA_RATE(odm->RxRate), odm->RSSI_A, odm->RSSI_B);
+                       HDATA_RATE(odm->RxRate), odm->RSSI_A, odm->RSSI_B);
 }