On Mon, Jun 16, 2014 at 5:41 PM, Vasili Galka <vvv...@gmail.com> wrote:
> TOP860 configuration assumes at most 128 flash sectors. Thus, the
> AMLV256U flash can't be supported. The existing code could result in
> memory corruption when writing to the flash_info->start[] array.
>
> Signed-off-by: Vasili Galka <vvv...@gmail.com>
> Cc: Wolfgang Denk <w...@denx.de>
> ---
> board/emk/common/flash.c | 4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/board/emk/common/flash.c b/board/emk/common/flash.c
> index ae5777c..4119b3b 100644
> --- a/board/emk/common/flash.c
> +++ b/board/emk/common/flash.c
> @@ -324,6 +324,7 @@ ulong flash_get_size (FPWV *addr, flash_info_t *info)
> }
> break;
> }
> +#ifndef CONFIG_TOP860
> if ((FPW)addr[FLASH_ID3] == (FPW)AMD_ID_LV256U_2 &&
> (FPW)addr[FLASH_ID4] == (FPW)AMD_ID_LV256U_3)
> {
> @@ -337,7 +338,8 @@ ulong flash_get_size (FPWV *addr, flash_info_t *info)
> }
> break;
> }
> -
> +#endif
> +
> /* fall thru to here ! */
> default:
> printf ("unknown AMD device=%x %x %x",
> --
> 1.7.9
>
>
Any review?
This was inspired by a a compiler warning. I'm still getting this warning
on the latest master.
Best,
Vasili
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
