summaryrefslogtreecommitdiff
path: root/board/flagadm/flash.c
diff options
context:
space:
mode:
authorMasahiro Yamada <yamada.m@jp.panasonic.com>2014-07-22 10:57:18 +0900
committerTom Rini <trini@ti.com>2014-07-22 09:46:48 -0400
commit61f06b143e92de70272627b4cd23d7b1e5af256e (patch)
treebd67b5814899af6b84ac73d091f71cf33b7f3243 /board/flagadm/flash.c
parente8078046b4f6484f2e5bcc65cab0f449acd1d208 (diff)
downloadu-boot-imx-61f06b143e92de70272627b4cd23d7b1e5af256e.zip
u-boot-imx-61f06b143e92de70272627b4cd23d7b1e5af256e.tar.gz
u-boot-imx-61f06b143e92de70272627b4cd23d7b1e5af256e.tar.bz2
m68k: fix an undefined behavior warning of M5253DEMO board
The latest GCC is so clever that it reports more warnings than old ones did: ------------------------------>8------------------------------ board/freescale/m5253demo/flash.c: In function 'flash_get_offsets': board/freescale/m5253demo/flash.c:65:23: warning: iteration 2047u invokes undefined behavior [-Waggressive-loop-optimizations] info->start[k + 1] = info->start[k] + CONFIG_SYS_SST_SECTSZ; ^ board/freescale/m5253demo/flash.c:64:3: note: containing loop for (k = 0, j = 0; j < CONFIG_SYS_SST_SECT; j++, k++) { ^ ------------------------------8<------------------------------ The cause of the warning is like this: The for statement iterates 2048 times in flash_get_offsets() func. (Notice CONFIG_SYS_SST_SECT is defined as 2048) The last iteration does info->start[2048] = info->start[2047] + CONFIG_SYS_SST_SECTSZ; causing an undefined behavior. (Please note the array size of info->start is 2048. CONFIG_SYS_MAX_FLASH_SECT is defined as 2048 for this board.) This commit fixes that so as not to overrun the info->start array. Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com> Cc: Jason Jin <Jason.jin@freescale.com>
Diffstat (limited to 'board/flagadm/flash.c')
0 files changed, 0 insertions, 0 deletions