diff options
| author | Bin Meng <bmeng.cn@gmail.com> | 2018-10-08 02:27:44 -0700 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2018-10-22 09:18:49 -0400 |
| commit | ed5df0852f56a3512db283010ad7b6e93cba0612 (patch) | |
| tree | e28915afa8bca698afbdb13fc7a376eb837cbfe0 /drivers/mtd/nand | |
| parent | 7702713ee8d2f6a9fbd66894ec2995856f7507dc (diff) | |
| download | u-boot-ed5df0852f56a3512db283010ad7b6e93cba0612.tar.gz u-boot-ed5df0852f56a3512db283010ad7b6e93cba0612.tar.xz u-boot-ed5df0852f56a3512db283010ad7b6e93cba0612.zip | |
nand: atmel: Initialize pmecc smu with correct size
Currently in pmecc_get_sigma(), the code tries to clear the memory
pointed by smu with wrong size 'sizeof(int16_t) * ARRAY_SIZE(smu)'.
Since smu is actually a pointer, not an array, so ARRAY_SIZE(smu)
does not generate correct size to be cleared.
In fact, GCC 8.1.0 reports a warning against it:
error: division 'sizeof (int16_t * {aka short int *}) / sizeof (int16_t
{aka short int})' does not compute the number of array elements
[-Werror=sizeof-pointer-div]
Fix it by using the correct size.
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Diffstat (limited to 'drivers/mtd/nand')
| -rw-r--r-- | drivers/mtd/nand/raw/atmel_nand.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/nand/raw/atmel_nand.c b/drivers/mtd/nand/raw/atmel_nand.c index a5b76e1aa0..31ad2cfa88 100644 --- a/drivers/mtd/nand/raw/atmel_nand.c +++ b/drivers/mtd/nand/raw/atmel_nand.c @@ -249,7 +249,7 @@ static void pmecc_get_sigma(struct mtd_info *mtd) int diff; /* Init the Sigma(x) */ - memset(smu, 0, sizeof(int16_t) * ARRAY_SIZE(smu)); + memset(smu, 0, sizeof(int16_t) * num * (cap + 2)); dmu_0_count = 0; |