mtd: cfi_cmdset_0002: remove redundant assignment to variable timeo
authorColin Ian King <colin.king@canonical.com>
Thu, 25 Mar 2021 17:45:14 +0000 (17:45 +0000)
committerVignesh Raghavendra <vigneshr@ti.com>
Wed, 7 Apr 2021 18:43:19 +0000 (00:13 +0530)
The variable timeo 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: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
Link: https://lore.kernel.org/r/20210325174514.486272-1-colin.king@canonical.com
Addresses-Coverity: ("Unused value")

drivers/mtd/chips/cfi_cmdset_0002.c

index 0bd8b5a..627adc3 100644 (file)
@@ -1653,7 +1653,7 @@ static int __xipram do_write_oneword_once(struct map_info *map,
                                          unsigned long adr, map_word datum,
                                          int mode, struct cfi_private *cfi)
 {
-       unsigned long timeo = jiffies + HZ;
+       unsigned long timeo;
        /*
         * We use a 1ms + 1 jiffies generic timeout for writes (most devices
         * have a max write time of a few hundreds usec). However, we should