mtd: cfi_cmdset_0002: remove redundant assignment to variable ret
authorColin Ian King <colin.i.king@gmail.com>
Wed, 2 Oct 2024 17:22:58 +0000 (18:22 +0100)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Mon, 21 Oct 2024 09:57:43 +0000 (11:57 +0200)
Variable ret is being assigned a value that is never read, the following
goto statement jumps to a statement that assigns ret a return from the
call to function do_write_oneword_once. The assignment is redundant
and can be removed.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20241002172258.958113-1-colin.i.king@gmail.com
drivers/mtd/chips/cfi_cmdset_0002.c

index 9f2223d..7c91429 100644 (file)
@@ -1779,10 +1779,8 @@ static int __xipram do_write_oneword_retry(struct map_info *map,
                map_write(map, CMD(0xF0), chip->start);
                /* FIXME - should have reset delay before continuing */
 
-               if (++retry_cnt <= MAX_RETRIES) {
-                       ret = 0;
+               if (++retry_cnt <= MAX_RETRIES)
                        goto retry;
-               }
        }
        xip_enable(map, chip, adr);