eeprom: at24: drop redundant variable in at24_write()
authorBartosz Golaszewski <brgl@bgdev.pl>
Mon, 19 Mar 2018 09:17:07 +0000 (10:17 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Mar 2018 15:25:00 +0000 (16:25 +0100)
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>
drivers/misc/eeprom/at24.c

index ef9d20f..2d8f36c 100644 (file)
@@ -443,18 +443,16 @@ static int at24_write(void *priv, unsigned int off, void *val, size_t count)
        gpiod_set_value_cansleep(at24->wp_gpio, 0);
 
        while (count) {
-               int status;
-
-               status = at24_regmap_write(at24, buf, off, count);
-               if (status < 0) {
+               ret = at24_regmap_write(at24, buf, off, count);
+               if (ret < 0) {
                        gpiod_set_value_cansleep(at24->wp_gpio, 1);
                        mutex_unlock(&at24->lock);
                        pm_runtime_put(dev);
-                       return status;
+                       return ret;
                }
-               buf += status;
-               off += status;
-               count -= status;
+               buf += ret;
+               off += ret;
+               count -= ret;
        }
 
        gpiod_set_value_cansleep(at24->wp_gpio, 1);