diff options
author | Mark A. Greer <mgreer@animalcreek.com> | 2014-09-23 22:25:45 +0400 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2014-09-24 02:28:22 +0400 |
commit | 671970f531378e2a22ba0168d85dfca1f4e082c4 (patch) | |
tree | d78532003fd70da4ffaf1a36358595274fd5c3de | |
parent | 890c165261f92e23005d84762acc98892acaa504 (diff) | |
download | linux-671970f531378e2a22ba0168d85dfca1f4e082c4.tar.xz |
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.c | 3 |
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) |