Commit d2e639d6 authored Jun 24, 2020 by Dan Carpenter Committed by Miquel Raynal Jul 07, 2020

mtd: rawnand: ingenic: cleanup ARRAY_SIZE() vs sizeof() use

The ARRAY_SIZE() is the number of elements but we want to use sizeof()
here for the number of bytes. Fortunately, they are the same thing
because it's an array of u8 so this has no effect on runtime.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20200624132640.GC9972@mwanda

parent 91e81150

drivers/mtd/nand/raw/ingenic/jz4740_ecc.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -90,8 +90,8 @@ static int jz4740_ecc_calculate(struct ingenic_ecc *ecc,
		* If the written data is completely 0xff, we also want to write 0xff as
		* ECC, otherwise we will get in trouble when doing subpage writes.
		*/
		if (memcmp(ecc_code, empty_block_ecc, ARRAY_SIZE(empty_block_ecc)) == 0)
		memset(ecc_code, 0xff, ARRAY_SIZE(empty_block_ecc));
		if (memcmp(ecc_code, empty_block_ecc, sizeof(empty_block_ecc)) == 0)
		memset(ecc_code, 0xff, sizeof(empty_block_ecc));

		return 0;
		}