mtd: rawnand: ingenic: cleanup ARRAY_SIZE() vs sizeof() use
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 24 Jun 2020 13:26:40 +0000 (16:26 +0300)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Tue, 7 Jul 2020 18:44:43 +0000 (20:44 +0200)
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
drivers/mtd/nand/raw/ingenic/jz4740_ecc.c

index 13fea645c7f0bcb81367fb1385bc309d60dcf02f..54e377754a6cc50fa6082b522ec0b387564ac1a4 100644 (file)
@@ -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;
 }