diff options
| author | Peter Senna Tschudin | 2014-05-26 16:08:50 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2014-05-26 19:32:15 +0200 |
| commit | 4764ca981b040048766e4f39a45a4b9c5cecff9c (patch) | |
| tree | b4743d812ce65e2be8ca8e9190a0385029bff358 /drivers/staging/rtl8192e/rtllib_softmac.c | |
| parent | Staging: Silicom: bpctl_mod.c & bypass.c: Adding blank lines (diff) | |
| download | kernel-qcow2-linux-4764ca981b040048766e4f39a45a4b9c5cecff9c.tar.gz kernel-qcow2-linux-4764ca981b040048766e4f39a45a4b9c5cecff9c.tar.xz kernel-qcow2-linux-4764ca981b040048766e4f39a45a4b9c5cecff9c.zip | |
drivers/staging: Remove useless return variables
This patch remove variables that are initialized with a constant,
are never updated, and are only used as parameter of return.
Return the constant instead of using a variable.
Verified by compilation only.
The coccinelle script that find and fixes this issue is:
// <smpl>
@@
type T;
constant C;
identifier ret;
@@
- T ret = C;
... when != ret
when strict
return
- ret
+ C
;
// </smpl>
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/rtl8192e/rtllib_softmac.c')
| -rw-r--r-- | drivers/staging/rtl8192e/rtllib_softmac.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c index d65cba428941..684ceed17232 100644 --- a/drivers/staging/rtl8192e/rtllib_softmac.c +++ b/drivers/staging/rtl8192e/rtllib_softmac.c @@ -3239,7 +3239,6 @@ static int rtllib_wpa_set_auth_algs(struct rtllib_device *ieee, int value) struct rtllib_security sec = { .flags = SEC_AUTH_MODE, }; - int ret = 0; if (value & AUTH_ALG_SHARED_KEY) { sec.auth_mode = WLAN_AUTH_SHARED_KEY; @@ -3259,7 +3258,7 @@ static int rtllib_wpa_set_auth_algs(struct rtllib_device *ieee, int value) if (ieee->set_security) ieee->set_security(ieee->dev, &sec); - return ret; + return 0; } static int rtllib_wpa_set_param(struct rtllib_device *ieee, u8 name, u32 value) |