diff options
| author | Aya Mahfouz <mahfouz.saif.elyazal@gmail.com> | 2015-02-27 15:11:02 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-03-01 16:57:36 -0800 |
| commit | e20af8acf1b538496827cd1e3b7b29631443528a (patch) | |
| tree | ac09e20983cff689fdbf68b7f2ea56fd79e26ca9 /drivers/staging/dgap | |
| parent | 11107ffe2cd1c1dc5948713fc08a1372185be0d5 (diff) | |
| download | blackbird-op-linux-e20af8acf1b538496827cd1e3b7b29631443528a.tar.gz blackbird-op-linux-e20af8acf1b538496827cd1e3b7b29631443528a.zip | |
staging: dgap: remove unused variable
This patch removes a variable that was simply used to
store the return value of a function call before
returning it.
The issue was detected and resolved using the following
coccinelle script:
@@
identifier len,f;
@@
-int len;
... when != len
when strict
-len =
+return
f(...);
-return len;
Signed-off-by: Aya Mahfouz <mahfouz.saif.elyazal@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/dgap')
| -rw-r--r-- | drivers/staging/dgap/dgap.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 7184747e0652..914e332e13a1 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dgap.c @@ -3979,7 +3979,6 @@ static int dgap_get_modem_info(struct channel_t *ch, unsigned int __user *value) int result; u8 mstat; ulong lock_flags; - int rc; spin_lock_irqsave(&ch->ch_lock, lock_flags); @@ -4004,9 +4003,7 @@ static int dgap_get_modem_info(struct channel_t *ch, unsigned int __user *value) if (mstat & D_CD(ch)) result |= TIOCM_CD; - rc = put_user(result, value); - - return rc; + return put_user(result, value); } /* |
