OSDN Git Service

complete ->[sg]et_serial() switchover
authorAl Viro <viro@zeniv.linux.org.uk>
Wed, 12 Sep 2018 11:46:51 +0000 (07:46 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Sat, 13 Oct 2018 04:50:43 +0000 (00:50 -0400)
Reviewed-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
drivers/tty/tty_io.c
drivers/usb/serial/usb-serial.c

index b96bfd0..15d60ba 100644 (file)
@@ -2474,7 +2474,7 @@ static int tty_tiocsserial(struct tty_struct *tty, struct serial_struct __user *
                pr_warn("%s: '%s' is using deprecated serial flags (with no effect): %.8x\n",
                        __func__, get_task_comm(comm, current), flags);
        if (!tty->ops->set_serial)
-               return -ENOIOCTLCMD;
+               return -ENOTTY;
        return tty->ops->set_serial(tty, &v);
 }
 
@@ -2485,7 +2485,7 @@ static int tty_tiocgserial(struct tty_struct *tty, struct serial_struct __user *
 
        memset(&v, 0, sizeof(struct serial_struct));
        if (!tty->ops->get_serial)
-               return -ENOIOCTLCMD;
+               return -ENOTTY;
        err = tty->ops->get_serial(tty, &v);
        if (!err && copy_to_user(ss, &v, sizeof(struct serial_struct)))
                err = -EFAULT;
@@ -2621,15 +2621,9 @@ long tty_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                }
                break;
        case TIOCSSERIAL:
-               retval = tty_tiocsserial(tty, p);
-               if (retval != -ENOIOCTLCMD)
-                       return retval;
-               break;
+               return tty_tiocsserial(tty, p);
        case TIOCGSERIAL:
-               retval = tty_tiocgserial(tty, p);
-               if (retval != -ENOIOCTLCMD)
-                       return retval;
-               break;
+               return tty_tiocgserial(tty, p);
        case TIOCGPTPEER:
                /* Special because the struct file is needed */
                return ptm_open_peer(file, tty, (int)arg);
index 0f96d82..7e89efb 100644 (file)
@@ -402,7 +402,7 @@ static int serial_get_serial(struct tty_struct *tty, struct serial_struct *ss)
 
        if (port->serial->type->get_serial)
                return port->serial->type->get_serial(tty, ss);
-       return -ENOIOCTLCMD;
+       return -ENOTTY;
 }
 
 static int serial_set_serial(struct tty_struct *tty, struct serial_struct *ss)
@@ -411,7 +411,7 @@ static int serial_set_serial(struct tty_struct *tty, struct serial_struct *ss)
 
        if (port->serial->type->set_serial)
                return port->serial->type->set_serial(tty, ss);
-       return -ENOIOCTLCMD;
+       return -ENOTTY;
 }
 
 static int serial_ioctl(struct tty_struct *tty,