From: Mark A. Greer <mgreer@animalcreek.com>
Date: Tue, 23 Sep 2014 18:25:45 +0000 (-0700)
Subject: NFC: trf7970a: Remove useless local variable
X-Git-Tag: v3.18-rc1~115^2~39^2~72^2~14
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=671970f531378e2a22ba0168d85dfca1f4e082c4;p=~emulex%2Finfiniband.git

NFC: trf7970a: Remove useless local variable

The trf7970a_suspend() routine always returns
zero so don't use a local return variable to
hold the return value.  This fixes up a smatch
warning.

Signed-off-by: Mark A. Greer <mgreer@animalcreek.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
---

diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c
index 2a521bb3806..00149a9a50e 100644
--- a/drivers/nfc/trf7970a.c
+++ b/drivers/nfc/trf7970a.c
@@ -2123,7 +2123,6 @@ static int trf7970a_suspend(struct device *dev)
 {
 	struct spi_device *spi = container_of(dev, struct spi_device, dev);
 	struct trf7970a *trf = spi_get_drvdata(spi);
-	int ret = 0;
 
 	dev_dbg(dev, "Suspend\n");
 
@@ -2133,7 +2132,7 @@ static int trf7970a_suspend(struct device *dev)
 
 	mutex_unlock(&trf->lock);
 
-	return ret;
+	return 0;
 }
 
 static int trf7970a_resume(struct device *dev)