diff options
| author |   Mark A. Greer <mgreer@animalcreek.com> | 2014-09-23 11:25:45 -0700 |
|---|---|---|
| committer |   Samuel Ortiz <sameo@linux.intel.com> | 2014-09-24 00:28:22 +0200 |
| commit | 671970f531378e2a22ba0168d85dfca1f4e082c4 (patch) | |
| tree | d78532003fd70da4ffaf1a36358595274fd5c3de /drivers/nfc | |
| parent | NFC: st21nfcb: Remove useless headers (diff) | |
| download | linux-dev-671970f531378e2a22ba0168d85dfca1f4e082c4.tar.xz linux-dev-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>
Diffstat (limited to 'drivers/nfc')
| -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) |