diff options
| author | Daeseok Youn <daeseok.youn@gmail.com> | 2016-05-04 05:13:48 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-05-09 12:10:03 (GMT) |
| commit | a5b90ef7a1fed3fc0f7a4bf0ad0159eaddb2acfb (patch) | |
| tree | 09fc2e8490f46bd597dbe0ed90357ea757b6b8f0 /drivers/staging/dgnc | |
| parent | 1b30ccded52fb2b297128b190b47641457ecc8f5 (diff) | |
| download | linux-a5b90ef7a1fed3fc0f7a4bf0ad0159eaddb2acfb.tar.xz | |
staging: dgnc: clean up the dgnc_get_modem_info()
the "ch" in dgnc_get_modem_info() was already checked before calling
this function and also if "ch" is not NULL, dgnc_get_mstat() returns
valid value so it doesn't need to check an error.
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/dgnc')
| -rw-r--r-- | drivers/staging/dgnc/dgnc_tty.c | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index 392d532..a505775 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -2020,17 +2020,7 @@ static inline int dgnc_get_mstat(struct channel_t *ch) static int dgnc_get_modem_info(struct channel_t *ch, unsigned int __user *value) { - int result; - - if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return -ENXIO; - - result = dgnc_get_mstat(ch); - - if (result < 0) - return -ENXIO; - - return put_user(result, value); + return put_user(dgnc_get_mstat(ch), value); } /* |