USB: serial: fix returnvar.cocci warnings
authorMathieu OTHACEHE <m.othacehe@gmail.com>
Thu, 4 Feb 2016 18:01:28 +0000 (19:01 +0100)
committerJohan Hovold <johan@kernel.org>
Sun, 28 Feb 2016 13:36:53 +0000 (14:36 +0100)
Remove unneeded variables when "0" can be returned.

Generated by: scripts/coccinelle/misc/returnvar.cocci

Signed-off-by: Mathieu OTHACEHE <m.othacehe@gmail.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/cyberjack.c
drivers/usb/serial/garmin_gps.c

index 2916dea3ede84a2a02f8ed58f8b31c18f0fefc1b..5f17a3b9916d79ea28c82b04755e23d5680d4b2d 100644 (file)
@@ -140,7 +140,6 @@ static int  cyberjack_open(struct tty_struct *tty,
 {
        struct cyberjack_private *priv;
        unsigned long flags;
-       int result = 0;
 
        dev_dbg(&port->dev, "%s - usb_clear_halt\n", __func__);
        usb_clear_halt(port->serial->dev, port->write_urb->pipe);
@@ -152,7 +151,7 @@ static int  cyberjack_open(struct tty_struct *tty,
        priv->wrsent = 0;
        spin_unlock_irqrestore(&priv->lock, flags);
 
-       return result;
+       return 0;
 }
 
 static void cyberjack_close(struct usb_serial_port *port)
index 9d4170cd5aac9d93dba89c51757cdc98fecbb9da..97cabf803c2fa91a134c957dba74c4f77b97c712 100644 (file)
@@ -860,7 +860,6 @@ static int process_resetdev_request(struct usb_serial_port *port)
 static int garmin_clear(struct garmin_data *garmin_data_p)
 {
        unsigned long flags;
-       int status = 0;
 
        /* flush all queued data */
        pkt_clear(garmin_data_p);
@@ -870,7 +869,7 @@ static int garmin_clear(struct garmin_data *garmin_data_p)
        garmin_data_p->outsize = 0;
        spin_unlock_irqrestore(&garmin_data_p->lock, flags);
 
-       return status;
+       return 0;
 }