spi: spidev: Remove redundant initialization of variable status
authorJay Fang <f.fangjian@huawei.com>
Wed, 9 Sep 2020 06:08:24 +0000 (14:08 +0800)
committerMark Brown <broonie@kernel.org>
Wed, 9 Sep 2020 15:27:46 +0000 (16:27 +0100)
In spidev_read() and spidev_write(), the variable status is being
initialized with a value that is never read and it is being updated
later with a new value. The initialization is redundant and can be
removed.

Signed-off-by: Jay Fang <f.fangjian@huawei.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/1599631704-53232-1-git-send-email-f.fangjian@huawei.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spidev.c

index 455e99c..859910e 100644 (file)
@@ -146,7 +146,7 @@ static ssize_t
 spidev_read(struct file *filp, char __user *buf, size_t count, loff_t *f_pos)
 {
        struct spidev_data      *spidev;
-       ssize_t                 status = 0;
+       ssize_t                 status;
 
        /* chipselect only toggles at start or end of operation */
        if (count > bufsiz)
@@ -176,7 +176,7 @@ spidev_write(struct file *filp, const char __user *buf,
                size_t count, loff_t *f_pos)
 {
        struct spidev_data      *spidev;
-       ssize_t                 status = 0;
+       ssize_t                 status;
        unsigned long           missing;
 
        /* chipselect only toggles at start or end of operation */