Commit c0324619 authored by Colin Ian King's avatar Colin Ian King Committed by Kalle Valo

wl3501_cs: remove redundant variable rc

The variable rc is being initialized with a value that is never
read and it is being updated later with a new value that is returned.
The variable is redundant and can be replaced with a return 0 as
there are no other return points in this function.

Addresses-Coverity: ("Unused value")
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent 4c8a4685
...@@ -1226,7 +1226,6 @@ static int wl3501_init_firmware(struct wl3501_card *this) ...@@ -1226,7 +1226,6 @@ static int wl3501_init_firmware(struct wl3501_card *this)
static int wl3501_close(struct net_device *dev) static int wl3501_close(struct net_device *dev)
{ {
struct wl3501_card *this = netdev_priv(dev); struct wl3501_card *this = netdev_priv(dev);
int rc = -ENODEV;
unsigned long flags; unsigned long flags;
struct pcmcia_device *link; struct pcmcia_device *link;
link = this->p_dev; link = this->p_dev;
...@@ -1241,10 +1240,9 @@ static int wl3501_close(struct net_device *dev) ...@@ -1241,10 +1240,9 @@ static int wl3501_close(struct net_device *dev)
/* Mask interrupts from the SUTRO */ /* Mask interrupts from the SUTRO */
wl3501_block_interrupt(this); wl3501_block_interrupt(this);
rc = 0;
printk(KERN_INFO "%s: WL3501 closed\n", dev->name); printk(KERN_INFO "%s: WL3501 closed\n", dev->name);
spin_unlock_irqrestore(&this->lock, flags); spin_unlock_irqrestore(&this->lock, flags);
return rc; 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