mtd: rawnand: remove redundant assignment to variable bbtblocks

ColinIanKing · miquelraynal · commit 9367043f1c02 · 2024-02-26T11:36:11.000+01:00
The variable bbtblocks is being assigned a value that is never read. The assignment is redundant and can be removed. Cleans up clang scan build warning: drivers/mtd/nand/raw/nand_bbt.c:579:3: warning: Value stored to 'bbtblocks' is never read [deadcode.DeadStores] 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/20240209174019.3933233-1-colin.i.king@gmail.com
diff --git a/drivers/mtd/nand/raw/nand_bbt.c b/drivers/mtd/nand/raw/nand_bbt.c
@@ -576,7 +576,6 @@ static int search_bbt(struct nand_chip *this, uint8_t *buf,
 		startblock &= bbtblocks - 1;
 	} else {
 		chips = 1;
-		bbtblocks = mtd->size >> this->bbt_erase_shift;
 	}
 
 	for (i = 0; i < chips; i++) {

Original file line number	Diff line number	Diff line change
`@@ -576,7 +576,6 @@ static int search_bbt(struct nand_chip this, uint8_t buf,`
`576`	`576`	`startblock &= bbtblocks - 1;`
`577`	`577`	`} else {`
`578`	`578`	`chips = 1;`
`579`		`- bbtblocks = mtd->size >> this->bbt_erase_shift;`
`580`	`579`	`}`
`581`	`580`
`582`	`581`	`for (i = 0; i < chips; i++) {`