staging: rtl8723au: core: rtw_cmd: Remove unused functions
authorRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Sun, 18 Jan 2015 00:59:09 +0000 (01:59 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 25 Jan 2015 11:59:18 +0000 (19:59 +0800)
Removes some functions that are not used anywhere:
rtw_set_ch_cmd23a() rtw_cmd_clr_isr23a()

This was partially found by using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723au/core/rtw_cmd.c
drivers/staging/rtl8723au/include/rtw_cmd.h

index 60e0ded8ae02727736fd8404a7f96571f7d7a0c9..2447a56df8381c48eddac40fe3d7fcd1c05c6f6f 100644 (file)
@@ -245,11 +245,6 @@ exit:
        return res;
 }
 
-void rtw_cmd_clr_isr23a(struct cmd_priv *pcmdpriv)
-{
-       pcmdpriv->cmd_done_cnt++;
-}
-
 void rtw_free_cmd_obj23a(struct cmd_obj *pcmd)
 {
 
@@ -852,62 +847,6 @@ exit:
        return res;
 }
 
-/*
- * This is only ever called from on_action_spct23a_ch_switch () which isn't
- * called from anywhere itself
- */
-int rtw_set_ch_cmd23a(struct rtw_adapter *padapter, u8 ch, u8 bw, u8 ch_offset,
-                     u8 enqueue)
-{
-       struct cmd_obj *pcmdobj;
-       struct set_ch_parm *set_ch_parm;
-       struct cmd_priv *pcmdpriv = &padapter->cmdpriv;
-       int res = _SUCCESS;
-
-       DBG_8723A("%s(%s): ch:%u, bw:%u, ch_offset:%u\n", __func__,
-                 padapter->pnetdev->name, ch, bw, ch_offset);
-
-       /* check input parameter */
-
-       /* prepare cmd parameter */
-       set_ch_parm = kzalloc(sizeof(*set_ch_parm), GFP_KERNEL);
-       if (!set_ch_parm) {
-               res = _FAIL;
-               goto exit;
-       }
-       set_ch_parm->ch = ch;
-       set_ch_parm->bw = bw;
-       set_ch_parm->ch_offset = ch_offset;
-
-       if (enqueue) {
-               /* need enqueue, prepare cmd_obj and enqueue */
-               pcmdobj = kzalloc(sizeof(struct cmd_obj), GFP_KERNEL);
-               if (!pcmdobj) {
-                       kfree(set_ch_parm);
-                       res = _FAIL;
-                       goto exit;
-               }
-
-               init_h2fwcmd_w_parm_no_rsp(pcmdobj, set_ch_parm,
-                                          GEN_CMD_CODE(_SetChannel));
-               res = rtw_enqueue_cmd23a(pcmdpriv, pcmdobj);
-       } else {
-               /* no need to enqueue, do the cmd hdl directly and
-                  free cmd parameter */
-               if (H2C_SUCCESS != set_ch_hdl23a(padapter, (u8 *)set_ch_parm))
-                       res = _FAIL;
-
-               kfree(set_ch_parm);
-       }
-
-       /* do something based on res... */
-exit:
-
-       DBG_8723A("%s(%s): res:%u\n", __func__, padapter->pnetdev->name, res);
-
-       return res;
-}
-
 static void traffic_status_watchdog(struct rtw_adapter *padapter)
 {
        u8 bEnterPS;
index 92c8ec4e3bdbf745b170df49c451aa4e0c70aba6..775dcdc1e7b9a823765ecbfc49278be385b4c848 100644 (file)
@@ -99,7 +99,6 @@ int rtw_init_cmd_priv23a(struct cmd_priv *pcmdpriv);
 
 u32 rtw_init_evt_priv23a (struct evt_priv *pevtpriv);
 void rtw_free_evt_priv23a (struct evt_priv *pevtpriv);
-void rtw_cmd_clr_isr23a(struct cmd_priv *pcmdpriv);
 void rtw_evt_notify_isr(struct evt_priv *pevtpriv);
 
 enum rtw_drvextra_cmd_id
@@ -713,7 +712,6 @@ int rtw_ps_cmd23a(struct rtw_adapter*padapter);
 int rtw_chk_hi_queue_cmd23a(struct rtw_adapter*padapter);
 #endif
 
-int rtw_set_ch_cmd23a(struct rtw_adapter*padapter, u8 ch, u8 bw, u8 ch_offset, u8 enqueue);
 int rtw_set_chplan_cmd(struct rtw_adapter*padapter, u8 chplan, u8 enqueue);
 int rtw_led_blink_cmd(struct rtw_adapter*padapter, struct led_8723a *pLed);
 int rtw_set_csa_cmd(struct rtw_adapter*padapter, u8 new_ch_no);