staging: rtl8723au: Use ARRAY_SIZE macro for sizes of arrays
authorJanani Ravichandran <janani.rvchndrn@gmail.com>
Thu, 11 Feb 2016 23:50:40 +0000 (18:50 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 12 Feb 2016 04:03:56 +0000 (20:03 -0800)
Use ARRAY_SIZE to calculate the size of an array to make code concise.
The semantic patch used can be found here:
https://github.com/coccinelle/coccinellery/commit/9cbab452a3a2e18439e8386d6c4a68ee42c3ee2b

Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723au/core/rtw_cmd.c
drivers/staging/rtl8723au/core/rtw_mlme_ext.c
drivers/staging/rtl8723au/hal/HalHWImg8723A_BB.c
drivers/staging/rtl8723au/hal/HalHWImg8723A_MAC.c

index 3035bb8..cd4e0f0 100644 (file)
@@ -295,8 +295,7 @@ static void rtw_cmd_work(struct work_struct *work)
 
 post_process:
        /* call callback function for post-processed */
-       if (pcmd->cmdcode < (sizeof(rtw_cmd_callback) /
-                            sizeof(struct _cmd_callback))) {
+       if (pcmd->cmdcode < ARRAY_SIZE(rtw_cmd_callback)) {
                pcmd_callback = rtw_cmd_callback[pcmd->cmdcode].callback;
                if (!pcmd_callback) {
                        RT_TRACE(_module_rtl871x_cmd_c_, _drv_info_,
index 7cd0052..a39e441 100644 (file)
@@ -2154,8 +2154,7 @@ OnAction23a(struct rtw_adapter *padapter, struct recv_frame *precv_frame)
 
        category = mgmt->u.action.category;
 
-       for (i = 0;
-            i < sizeof(OnAction23a_tbl) / sizeof(struct action_handler); i++) {
+       for (i = 0; i < ARRAY_SIZE(OnAction23a_tbl); i++) {
                ptable = &OnAction23a_tbl[i];
 
                if (category == ptable->num)
index e8cab9e..8d3ea6c 100644 (file)
@@ -219,7 +219,7 @@ void ODM_ReadAndConfig_AGC_TAB_1T_8723A(struct dm_odm_t *pDM_Odm)
        u32 i;
        u8 platform = 0x04;
        u8 board = pDM_Odm->BoardType;
-       u32 ArrayLen = sizeof(Array_AGC_TAB_1T_8723A)/sizeof(u32);
+       u32 ArrayLen = ARRAY_SIZE(Array_AGC_TAB_1T_8723A);
        u32 *Array = Array_AGC_TAB_1T_8723A;
 
        hex = board;
@@ -467,7 +467,7 @@ void ODM_ReadAndConfig_PHY_REG_1T_8723A(struct dm_odm_t *pDM_Odm)
        u32 i = 0;
        u8  platform = 0x04;
        u8  board = pDM_Odm->BoardType;
-       u32 ArrayLen = sizeof(Array_PHY_REG_1T_8723A)/sizeof(u32);
+       u32 ArrayLen = ARRAY_SIZE(Array_PHY_REG_1T_8723A);
        u32 *Array = Array_PHY_REG_1T_8723A;
 
        hex += board;
@@ -523,7 +523,7 @@ void ODM_ReadAndConfig_PHY_REG_MP_8723A(struct dm_odm_t *pDM_Odm)
        u32 i;
        u8 platform = 0x04;
        u8 board = pDM_Odm->BoardType;
-       u32 ArrayLen = sizeof(Array_PHY_REG_MP_8723A)/sizeof(u32);
+       u32 ArrayLen = ARRAY_SIZE(Array_PHY_REG_MP_8723A);
        u32 *Array = Array_PHY_REG_MP_8723A;
 
        hex += board;
index 93b2d18..9bf6859 100644 (file)
@@ -145,7 +145,7 @@ void ODM_ReadAndConfig_MAC_REG_8723A(struct dm_odm_t *pDM_Odm)
        u32     i           = 0;
        u8     platform    = 0x04;
        u8     board       = pDM_Odm->BoardType;
-       u32     ArrayLen    = sizeof(Array_MAC_REG_8723A)/sizeof(u32);
+       u32     ArrayLen    = ARRAY_SIZE(Array_MAC_REG_8723A);
        u32 *Array       = Array_MAC_REG_8723A;
 
        hex += board;