Browse Source

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>
Mark A. Greer 11 years ago
parent
commit
671970f531
1 changed files with 1 additions and 2 deletions
  1. 1 2
      drivers/nfc/trf7970a.c

+ 1 - 2
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)