Removed useless return value variables. Signed-off By: Trevor Pace <trevor.pace@xxxxxx> ================================================================================ diff --git a/drivers/usb/serial/usb-serial.c b/drivers/usb/serial/usb-serial.c index bd7581b..faec1d1 100644 --- a/drivers/usb/serial/usb-serial.c +++ b/drivers/usb/serial/usb-serial.c @@ -362,10 +362,9 @@ static int serial_write(struct tty_struct *tty, const unsigned char *buf, int count) { struct usb_serial_port *port = tty->driver_data; - int retval = -ENODEV; if (port->serial->dev->state == USB_STATE_NOTATTACHED) - goto exit; + return -ENODEV; dbg("%s - port %d, %d byte(s)", __func__, port->number, count); @@ -374,10 +373,7 @@ static int serial_write(struct tty_struct *tty, const unsigned char *buf, WARN_ON(!port->port.count); /* pass on to the driver specific version of this function */ - retval = port->serial->type->write(tty, port, buf, count); - -exit: - return retval; + return port->serial->type->write(tty, port, buf, count); } static int serial_write_room(struct tty_struct *tty) @@ -429,7 +425,6 @@ static int serial_ioctl(struct tty_struct *tty, struct file *file, unsigned int cmd, unsigned long arg) { struct usb_serial_port *port = tty->driver_data; - int retval = -ENODEV; dbg("%s - port %d, cmd 0x%.4x", __func__, port->number, cmd); @@ -437,11 +432,9 @@ static int serial_ioctl(struct tty_struct *tty, struct file *file, /* pass on to the driver specific version of this function if it is available */ - if (port->serial->type->ioctl) { - retval = port->serial->type->ioctl(tty, file, cmd, arg); - } else - retval = -ENOIOCTLCMD; - return retval; + if (port->serial->type->ioctl) + return port->serial->type->ioctl(tty, file, cmd, arg); + return -ENOIOCTLCMD; } static void serial_set_termios(struct tty_struct *tty, struct ktermios *old) @@ -1174,7 +1167,7 @@ int usb_serial_suspend(struct usb_interface *intf, pm_message_t message) { struct usb_serial *serial = usb_get_intfdata(intf); struct usb_serial_port *port; - int i, r = 0; + int i; serial->suspending = 1; @@ -1185,24 +1178,21 @@ int usb_serial_suspend(struct usb_interface *intf, pm_message_t message) } if (serial->type->suspend) - r = serial->type->suspend(serial, message); + return serial->type->suspend(serial, message); - return r; + return 0; } EXPORT_SYMBOL(usb_serial_suspend); int usb_serial_resume(struct usb_interface *intf) { struct usb_serial *serial = usb_get_intfdata(intf); - int rv; serial->suspending = 0; - if (serial->type->resume) - rv = serial->type->resume(serial); - else - rv = usb_serial_generic_resume(serial); - return rv; + return (serial->type->resume) + ? serial->type->resume(serial) + : usb_serial_generic_resume(serial); } EXPORT_SYMBOL(usb_serial_resume); -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html