diff options
| author |   Nachammai Karuppiah <nachukannan@gmail.com> | 2019-11-05 01:21:31 -0800 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-11-05 17:50:21 +0100 |
| commit | 78ce93f3e80bf8dff320a6d084c9933e81aa4a3f (patch) | |
| tree | 6eddb7c82129995b6d2b0fa4ede9ccc0081ae64d /drivers/staging/wfx | |
| parent | staging: wfx: replace 1 by true (diff) | |
| download | linux-dev-78ce93f3e80bf8dff320a6d084c9933e81aa4a3f.tar.xz linux-dev-78ce93f3e80bf8dff320a6d084c9933e81aa4a3f.zip | |
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 <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>
Diffstat (limited to 'drivers/staging/wfx')
| -rw-r--r-- | drivers/staging/wfx/sta.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c index 70a00b84ac5f..2ab5ba9742ad 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) |