diff mbox

spi: spidev: Remove unneeded variable

Message ID 1431187039-10573-1-git-send-email-festevam@gmail.com (mailing list archive)
State Accepted
Commit 99472cc08a8bda2c56bceec3693193cc87a33ef1
Headers show

Commit Message

Fabio Estevam May 9, 2015, 3:57 p.m. UTC 
From: Fabio Estevam <fabio.estevam@freescale.com>

Remove unneeded variable used to store return value.

The semantic patch that makes this change is available
in scripts/coccinelle/misc/returnvar.cocci.

More information about semantic patching is available at
http://coccinelle.lip6.fr/

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 drivers/spi/spidev.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Mark Brown May 11, 2015, 3:15 p.m. UTC | #1 
On Sat, May 09, 2015 at 12:57:19PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Remove unneeded variable used to store return value.

Applied, thanks.
diff mbox

Patch

diff --git a/drivers/spi/spidev.c b/drivers/spi/spidev.c
index b00ae96..dd616ff 100644
--- a/drivers/spi/spidev.c
+++ b/drivers/spi/spidev.c
@@ -635,7 +635,6 @@  err_find_dev:
 static int spidev_release(struct inode *inode, struct file *filp)
 {
 	struct spidev_data	*spidev;
-	int			status = 0;
 
 	mutex_lock(&device_list_lock);
 	spidev = filp->private_data;
@@ -664,7 +663,7 @@  static int spidev_release(struct inode *inode, struct file *filp)
 	}
 	mutex_unlock(&device_list_lock);
 
-	return status;
+	return 0;
 }
 
 static const struct file_operations spidev_fops = {