diff options
| author | Daeseok Youn <daeseok.youn@gmail.com> | 2014-07-11 19:22:04 +0900 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-07-11 17:35:45 -0700 |
| commit | 6299ae5aa7bc82cf0cd43e800c4857dba3e4d6e5 (patch) | |
| tree | a466af93bb985c3c6b157892ea50fe9232d1a7dc /drivers/staging/dgap | |
| parent | 604631ff0b2b9e36d9ad1ebe389687f5ed7b7b90 (diff) | |
staging: dgap: redundant NULL and magic check in dgap_get_modem_info()
The "ch" is already checking in caller.
Signed-off-by: Daeseok Youn <daeseok.youn@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 | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/staging/dgap/dgap.c b/drivers/staging/dgap/dgap.c index 9affc5bf722..79ea11696a2 100644 --- a/drivers/staging/dgap/dgap.c +++ b/drivers/staging/dgap/dgap.c @@ -3061,9 +3061,6 @@ static int dgap_get_modem_info(struct channel_t *ch, unsigned int __user *value) ulong lock_flags; int rc; - if (!ch || ch->magic != DGAP_CHANNEL_MAGIC) - return -EIO; - spin_lock_irqsave(&ch->ch_lock, lock_flags); mstat = readb(&(ch->ch_bs->m_stat)); |