]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
misc: eeprom_93xx46: Replace explicit castings with proper specifiers
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 8 May 2024 18:47:01 +0000 (21:47 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 4 Jun 2024 15:47:08 +0000 (17:47 +0200)
There is no need to have an explicit casting when we can simply use
the correct printf() specifier.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Link: https://lore.kernel.org/r/20240508184905.2102633-9-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/eeprom/eeprom_93xx46.c

index 18a3b534ea7341c5c873ade3d67bd60c39bd13af..ac485b2827dbe26559659f941a631c76dbba5d0c 100644 (file)
@@ -162,8 +162,8 @@ static int eeprom_93xx46_read(void *priv, unsigned int off,
                ndelay(250);
 
                if (err) {
-                       dev_err(&edev->spi->dev, "read %zu bytes at %d: err. %d\n",
-                               nbytes, (int)off, err);
+                       dev_err(&edev->spi->dev, "read %zu bytes at %u: err. %d\n",
+                               nbytes, off, err);
                        break;
                }
 
@@ -274,7 +274,8 @@ static int eeprom_93xx46_write(void *priv, unsigned int off,
 {
        struct eeprom_93xx46_dev *edev = priv;
        char *buf = val;
-       int i, ret, step = 1;
+       int ret, step = 1;
+       unsigned int i;
 
        if (unlikely(off >= edev->size))
                return -EFBIG;
@@ -301,8 +302,7 @@ static int eeprom_93xx46_write(void *priv, unsigned int off,
        for (i = 0; i < count; i += step) {
                ret = eeprom_93xx46_write_word(edev, &buf[i], off + i);
                if (ret) {
-                       dev_err(&edev->spi->dev, "write failed at %d: %d\n",
-                               (int)off + i, ret);
+                       dev_err(&edev->spi->dev, "write failed at %u: %d\n", off + i, ret);
                        break;
                }
        }