From: Bartosz Golaszewski <brgl@bgdev.pl>
Date: Mon, 19 Mar 2018 09:17:06 +0000 (+0100)
Subject: eeprom: at24: drop redundant variable in at24_read()
X-Git-Tag: for-linus-20180413~38^2~23
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=eb27fde2731b6cb5818493b8ac18e01f427e335f;p=linux-block.git

eeprom: at24: drop redundant variable in at24_read()

We can reuse ret instead of defining a loop-local status variable.

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
Tested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index fb71041971d2..ef9d20f43127 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -395,17 +395,15 @@ static int at24_read(void *priv, unsigned int off, void *val, size_t count)
 	mutex_lock(&at24->lock);
 
 	while (count) {
-		int	status;
-
-		status = at24_regmap_read(at24, buf, off, count);
-		if (status < 0) {
+		ret = at24_regmap_read(at24, buf, off, count);
+		if (ret < 0) {
 			mutex_unlock(&at24->lock);
 			pm_runtime_put(dev);
-			return status;
+			return ret;
 		}
-		buf += status;
-		off += status;
-		count -= status;
+		buf += ret;
+		off += ret;
+		count -= ret;
 	}
 
 	mutex_unlock(&at24->lock);