staging: wilc1000: remove wilc_free_join_params
authorChaehyun Lim <chaehyun.lim@gmail.com>
Thu, 24 Dec 2015 07:52:35 +0000 (16:52 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 3 Feb 2016 23:08:06 +0000 (15:08 -0800)
wilc_free_join_params call kfree. There is no need to use
wrapper function, so use kfree directly.

Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/host_interface.c
drivers/staging/wilc1000/host_interface.h
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c

index 6e5bccc8bc3d0d2bda0299cbae87d5a2b1bef645..724b4bd75ac07db6d3a3d3055a6eb868afa42cca 100644 (file)
@@ -4609,14 +4609,6 @@ static void *host_int_ParseJoinBssParam(tstrNetworkInfo *ptstrNetworkInfo)
        return (void *)pNewJoinBssParam;
 }
 
-void wilc_free_join_params(void *pJoinParams)
-{
-       if ((struct bss_param *)pJoinParams)
-               kfree((struct bss_param *)pJoinParams);
-       else
-               PRINT_ER("Unable to FREE null pointer\n");
-}
-
 s32 wilc_del_all_rx_ba_session(struct wilc_vif *vif, char *pBSSID, char TID)
 {
        s32 result = 0;
index fa5d2925e47a529044900c22455150a2cac8f119..7ac85f27217e6060b67faa8aef36f7c223fa9e31 100644 (file)
@@ -365,9 +365,6 @@ s32 wilc_listen_state_expired(struct wilc_vif *vif, u32 u32SessionID);
 s32 wilc_frame_register(struct wilc_vif *vif, u16 u16FrameType, bool bReg);
 int wilc_set_wfi_drv_handler(struct wilc_vif *vif, int index);
 int wilc_set_operation_mode(struct wilc_vif *vif, u32 mode);
-
-void wilc_free_join_params(void *pJoinParams);
-
 s32 wilc_get_statistics(struct wilc_vif *vif, struct rf_info *pstrStatistics);
 void wilc_resolve_disconnect_aberration(struct wilc_vif *vif);
 int wilc_get_vif_idx(struct wilc_vif *vif);
index 53fb2d4bb0bd11c9038b2b05c42979dee3bd722a..be5704bfc3821b405bc75443e8fe6e3af9d9ffa8 100644 (file)
@@ -196,7 +196,7 @@ static void clear_shadow_scan(void)
                                last_scanned_shadow[last_scanned_cnt].pu8IEs = NULL;
                        }
 
-                       wilc_free_join_params(last_scanned_shadow[i].pJoinParams);
+                       kfree(last_scanned_shadow[i].pJoinParams);
                        last_scanned_shadow[i].pJoinParams = NULL;
                }
                last_scanned_cnt = 0;
@@ -282,7 +282,7 @@ static void remove_network_from_shadow(unsigned long arg)
                        kfree(last_scanned_shadow[i].pu8IEs);
                        last_scanned_shadow[i].pu8IEs = NULL;
 
-                       wilc_free_join_params(last_scanned_shadow[i].pJoinParams);
+                       kfree(last_scanned_shadow[i].pJoinParams);
 
                        for (j = i; (j < last_scanned_cnt - 1); j++)
                                last_scanned_shadow[j] = last_scanned_shadow[j + 1];
@@ -376,7 +376,7 @@ static void add_network_to_shadow(tstrNetworkInfo *pstrNetworkInfo,
        last_scanned_shadow[ap_index].u32TimeRcvdInScanCached = jiffies;
        last_scanned_shadow[ap_index].u8Found = 1;
        if (ap_found != -1)
-               wilc_free_join_params(last_scanned_shadow[ap_index].pJoinParams);
+               kfree(last_scanned_shadow[ap_index].pJoinParams);
        last_scanned_shadow[ap_index].pJoinParams = pJoinParams;
 }