diff options
author | Peter Senna Tschudin <peter.senna@gmail.com> | 2014-05-31 12:12:26 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2014-06-19 15:49:12 -0400 |
commit | 69253b6108102834c7cac826d4c34a0414b83a3e (patch) | |
tree | 5633f762fe4ba3f9f62d0c91c13cd25c5d9c376b /drivers/net/wireless/cw1200 | |
parent | 12f32370068d539bdc2fc9cf22c37dbdc2bae158 (diff) |
cw1200: 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: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/cw1200')
-rw-r--r-- | drivers/net/wireless/cw1200/sta.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/cw1200/sta.c b/drivers/net/wireless/cw1200/sta.c index cd0cad7f7759..5b84664db13b 100644 --- a/drivers/net/wireless/cw1200/sta.c +++ b/drivers/net/wireless/cw1200/sta.c | |||
@@ -2289,7 +2289,6 @@ static int cw1200_upload_null(struct cw1200_common *priv) | |||
2289 | 2289 | ||
2290 | static int cw1200_upload_qosnull(struct cw1200_common *priv) | 2290 | static int cw1200_upload_qosnull(struct cw1200_common *priv) |
2291 | { | 2291 | { |
2292 | int ret = 0; | ||
2293 | /* TODO: This needs to be implemented | 2292 | /* TODO: This needs to be implemented |
2294 | 2293 | ||
2295 | struct wsm_template_frame frame = { | 2294 | struct wsm_template_frame frame = { |
@@ -2306,7 +2305,7 @@ static int cw1200_upload_qosnull(struct cw1200_common *priv) | |||
2306 | dev_kfree_skb(frame.skb); | 2305 | dev_kfree_skb(frame.skb); |
2307 | 2306 | ||
2308 | */ | 2307 | */ |
2309 | return ret; | 2308 | return 0; |
2310 | } | 2309 | } |
2311 | 2310 | ||
2312 | static int cw1200_enable_beaconing(struct cw1200_common *priv, | 2311 | static int cw1200_enable_beaconing(struct cw1200_common *priv, |