diff options
author | Tobin C. Harding <me@tobin.cc> | 2017-03-13 12:37:32 +1100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-14 06:27:31 +0800 |
commit | 93cf5f43421513ee6c07da68903221003a490f11 (patch) | |
tree | 66eee3e62a4c00def4297906b3c79684d4465976 /drivers/staging/dgnc/dgnc_tty.c | |
parent | cb52fce16027d79a97d0870a1110d4a4493fa0a8 (diff) |
staging: dgnc: return error code directly
In various functions a return code variable is defined at the top of
function, for example;
rc = -ENODEV;
and then the variable is returned. This makes it harder to read since
it separates the error code from the return site.
Return the error code directly instead of using a variable.
Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/dgnc/dgnc_tty.c')
-rw-r--r-- | drivers/staging/dgnc/dgnc_tty.c | 68 |
1 files changed, 32 insertions, 36 deletions
diff --git a/drivers/staging/dgnc/dgnc_tty.c b/drivers/staging/dgnc/dgnc_tty.c index ffbe5a22169f..cbf90e0e91e7 100644 --- a/drivers/staging/dgnc/dgnc_tty.c +++ b/drivers/staging/dgnc/dgnc_tty.c @@ -1669,20 +1669,20 @@ static int dgnc_tty_tiocmget(struct tty_struct *tty) { struct channel_t *ch; struct un_t *un; - int rc = -EIO; + int rc; unsigned char mstat = 0; unsigned long flags; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -EIO; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -EIO; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -EIO; spin_lock_irqsave(&ch->ch_lock, flags); @@ -1720,23 +1720,22 @@ static int dgnc_tty_tiocmset(struct tty_struct *tty, struct dgnc_board *bd; struct channel_t *ch; struct un_t *un; - int rc = -EIO; unsigned long flags; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -EIO; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -EIO; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -EIO; bd = ch->ch_bd; if (!bd || bd->magic != DGNC_BOARD_MAGIC) - return rc; + return -EIO; spin_lock_irqsave(&ch->ch_lock, flags); @@ -1769,23 +1768,22 @@ static int dgnc_tty_send_break(struct tty_struct *tty, int msec) struct dgnc_board *bd; struct channel_t *ch; struct un_t *un; - int rc = -EIO; unsigned long flags; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -EIO; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -EIO; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -EIO; bd = ch->ch_bd; if (!bd || bd->magic != DGNC_BOARD_MAGIC) - return rc; + return -EIO; switch (msec) { case -1: @@ -1877,11 +1875,11 @@ static void dgnc_tty_send_xchar(struct tty_struct *tty, char c) static inline int dgnc_get_mstat(struct channel_t *ch) { unsigned char mstat; - int rc = -ENXIO; unsigned long flags; + int rc; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -ENXIO; spin_lock_irqsave(&ch->ch_lock, flags); @@ -1990,21 +1988,20 @@ static int dgnc_tty_digigeta(struct tty_struct *tty, struct un_t *un; struct digi_t tmp; unsigned long flags; - int rc = -EFAULT; if (!retinfo) - return rc; + return -EFAULT; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -EFAULT; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -EFAULT; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -EFAULT; memset(&tmp, 0, sizeof(tmp)); @@ -2013,7 +2010,7 @@ static int dgnc_tty_digigeta(struct tty_struct *tty, spin_unlock_irqrestore(&ch->ch_lock, flags); if (copy_to_user(retinfo, &tmp, sizeof(*retinfo))) - return rc; + return -EFAULT; return 0; } @@ -2031,25 +2028,24 @@ static int dgnc_tty_digiseta(struct tty_struct *tty, struct un_t *un; struct digi_t new_digi; unsigned long flags; - int rc = -EFAULT; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -EFAULT; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -EFAULT; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -EFAULT; bd = ch->ch_bd; if (!bd || bd->magic != DGNC_BOARD_MAGIC) - return rc; + return -EFAULT; if (copy_from_user(&new_digi, new_info, sizeof(new_digi))) - return rc; + return -EFAULT; spin_lock_irqsave(&ch->ch_lock, flags); @@ -2358,24 +2354,24 @@ static int dgnc_tty_ioctl(struct tty_struct *tty, unsigned int cmd, struct board_ops *ch_bd_ops; struct channel_t *ch; struct un_t *un; - int rc = -ENODEV; + int rc; unsigned long flags; void __user *uarg = (void __user *)arg; if (!tty || tty->magic != TTY_MAGIC) - return rc; + return -ENODEV; un = tty->driver_data; if (!un || un->magic != DGNC_UNIT_MAGIC) - return rc; + return -ENODEV; ch = un->un_ch; if (!ch || ch->magic != DGNC_CHANNEL_MAGIC) - return rc; + return -ENODEV; bd = ch->ch_bd; if (!bd || bd->magic != DGNC_BOARD_MAGIC) - return rc; + return -ENODEV; ch_bd_ops = bd->bd_ops; @@ -2401,7 +2397,7 @@ static int dgnc_tty_ioctl(struct tty_struct *tty, unsigned int cmd, rc = tty_check_change(tty); spin_unlock_irqrestore(&ch->ch_lock, flags); if (rc) - return rc; + return -ENODEV; rc = ch_bd_ops->drain(tty, 0); @@ -2427,7 +2423,7 @@ static int dgnc_tty_ioctl(struct tty_struct *tty, unsigned int cmd, rc = tty_check_change(tty); spin_unlock_irqrestore(&ch->ch_lock, flags); if (rc) - return rc; + return -ENODEV; rc = ch_bd_ops->drain(tty, 0); if (rc) @@ -2445,7 +2441,7 @@ static int dgnc_tty_ioctl(struct tty_struct *tty, unsigned int cmd, rc = tty_check_change(tty); spin_unlock_irqrestore(&ch->ch_lock, flags); if (rc) - return rc; + return -ENODEV; rc = ch_bd_ops->drain(tty, 0); if (rc) |