Commit 0f3083af authored by Mathieu OTHACEHE's avatar Mathieu OTHACEHE Committed by Johan Hovold

USB: serial: fix returnvar.cocci warnings

Remove unneeded variables when "0" can be returned.

Generated by: scripts/coccinelle/misc/returnvar.cocci
Signed-off-by: default avatarMathieu OTHACEHE <m.othacehe@gmail.com>
Signed-off-by: default avatarJohan Hovold <johan@kernel.org>
parent fb571101
...@@ -140,7 +140,6 @@ static int cyberjack_open(struct tty_struct *tty, ...@@ -140,7 +140,6 @@ static int cyberjack_open(struct tty_struct *tty,
{ {
struct cyberjack_private *priv; struct cyberjack_private *priv;
unsigned long flags; unsigned long flags;
int result = 0;
dev_dbg(&port->dev, "%s - usb_clear_halt\n", __func__); dev_dbg(&port->dev, "%s - usb_clear_halt\n", __func__);
usb_clear_halt(port->serial->dev, port->write_urb->pipe); usb_clear_halt(port->serial->dev, port->write_urb->pipe);
...@@ -152,7 +151,7 @@ static int cyberjack_open(struct tty_struct *tty, ...@@ -152,7 +151,7 @@ static int cyberjack_open(struct tty_struct *tty,
priv->wrsent = 0; priv->wrsent = 0;
spin_unlock_irqrestore(&priv->lock, flags); spin_unlock_irqrestore(&priv->lock, flags);
return result; return 0;
} }
static void cyberjack_close(struct usb_serial_port *port) static void cyberjack_close(struct usb_serial_port *port)
......
...@@ -860,7 +860,6 @@ static int process_resetdev_request(struct usb_serial_port *port) ...@@ -860,7 +860,6 @@ static int process_resetdev_request(struct usb_serial_port *port)
static int garmin_clear(struct garmin_data *garmin_data_p) static int garmin_clear(struct garmin_data *garmin_data_p)
{ {
unsigned long flags; unsigned long flags;
int status = 0;
/* flush all queued data */ /* flush all queued data */
pkt_clear(garmin_data_p); pkt_clear(garmin_data_p);
...@@ -870,7 +869,7 @@ static int garmin_clear(struct garmin_data *garmin_data_p) ...@@ -870,7 +869,7 @@ static int garmin_clear(struct garmin_data *garmin_data_p)
garmin_data_p->outsize = 0; garmin_data_p->outsize = 0;
spin_unlock_irqrestore(&garmin_data_p->lock, flags); spin_unlock_irqrestore(&garmin_data_p->lock, flags);
return status; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment