summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark A. Greer <mgreer@animalcreek.com>2014-09-23 11:25:45 -0700
committerSamuel Ortiz <sameo@linux.intel.com>2014-09-24 00:28:22 +0200
commit671970f531378e2a22ba0168d85dfca1f4e082c4 (patch)
treed78532003fd70da4ffaf1a36358595274fd5c3de
parent890c165261f92e23005d84762acc98892acaa504 (diff)
downloadlinux-671970f531378e2a22ba0168d85dfca1f4e082c4.tar.gz
linux-671970f531378e2a22ba0168d85dfca1f4e082c4.tar.bz2
linux-671970f531378e2a22ba0168d85dfca1f4e082c4.zip
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>
-rw-r--r--drivers/nfc/trf7970a.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c
index 2a521bb38060..00149a9a50ef 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)