staging: wfx: Remove local variable used only in return statement.
authorNachammai Karuppiah <nachukannan@gmail.com>
Tue, 5 Nov 2019 09:21:31 +0000 (01:21 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 5 Nov 2019 16:50:21 +0000 (17:50 +0100)
Remove unnecessary local variable that is used in return statement and
return the expression directly. Issue found using coccinelle.

Signed-off-by: Nachammai Karuppiah <nachukannan@gmail.com>
Link: https://lore.kernel.org/r/1572945691-109992-1-git-send-email-nachukannan@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wfx/sta.c

index 70a00b8..2ab5ba9 100644 (file)
@@ -114,8 +114,6 @@ end:
 static int wfx_set_uapsd_param(struct wfx_vif *wvif,
                           const struct wfx_edca_params *arg)
 {
-       int ret;
-
        /* Here's the mapping AC [queue, bit]
         *  VO [0,3], VI [1, 2], BE [2, 1], BK [3, 0]
         */
@@ -148,8 +146,7 @@ static int wfx_set_uapsd_param(struct wfx_vif *wvif,
        wvif->uapsd_info.max_auto_trigger_interval = 0;
        wvif->uapsd_info.auto_trigger_step = 0;
 
-       ret = hif_set_uapsd_info(wvif, &wvif->uapsd_info);
-       return ret;
+       return hif_set_uapsd_info(wvif, &wvif->uapsd_info);
 }
 
 int wfx_fwd_probe_req(struct wfx_vif *wvif, bool enable)