From: Nachammai Karuppiah Date: Tue, 5 Nov 2019 09:21:31 +0000 (-0800) Subject: staging: wfx: Remove local variable used only in return statement. X-Git-Tag: v5.10.7~3799^2~106 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=78ce93f3e80bf8dff320a6d084c9933e81aa4a3f;p=platform%2Fkernel%2Flinux-rpi.git staging: wfx: Remove local variable used only in return statement. Remove unnecessary local variable that is used in return statement and return the expression directly. Issue found using coccinelle. Signed-off-by: Nachammai Karuppiah Link: https://lore.kernel.org/r/1572945691-109992-1-git-send-email-nachukannan@gmail.com Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c index 70a00b8..2ab5ba9 100644 --- a/drivers/staging/wfx/sta.c +++ b/drivers/staging/wfx/sta.c @@ -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)