diff options
author | Mark A. Greer <mgreer@animalcreek.com> | 2014-09-23 14:25:45 -0400 |
---|---|---|
committer | Samuel Ortiz <sameo@linux.intel.com> | 2014-09-23 18:28:22 -0400 |
commit | 671970f531378e2a22ba0168d85dfca1f4e082c4 (patch) | |
tree | d78532003fd70da4ffaf1a36358595274fd5c3de | |
parent | 890c165261f92e23005d84762acc98892acaa504 (diff) |
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) | |||
2123 | { | 2123 | { |
2124 | struct spi_device *spi = container_of(dev, struct spi_device, dev); | 2124 | struct spi_device *spi = container_of(dev, struct spi_device, dev); |
2125 | struct trf7970a *trf = spi_get_drvdata(spi); | 2125 | struct trf7970a *trf = spi_get_drvdata(spi); |
2126 | int ret = 0; | ||
2127 | 2126 | ||
2128 | dev_dbg(dev, "Suspend\n"); | 2127 | dev_dbg(dev, "Suspend\n"); |
2129 | 2128 | ||
@@ -2133,7 +2132,7 @@ static int trf7970a_suspend(struct device *dev) | |||
2133 | 2132 | ||
2134 | mutex_unlock(&trf->lock); | 2133 | mutex_unlock(&trf->lock); |
2135 | 2134 | ||
2136 | return ret; | 2135 | return 0; |
2137 | } | 2136 | } |
2138 | 2137 | ||
2139 | static int trf7970a_resume(struct device *dev) | 2138 | static int trf7970a_resume(struct device *dev) |