On 10/07/2011 08:40 AM, Heiko Schocher wrote:
> commit 8d3fcb5e60b6c8e1d530dbc2e2e33ec6a44670da breaks post
> memory support for sdram base != 0x00000000. Fix this.
>
> Signed-off-by: Heiko Schocher <h...@denx.de>
> Cc: Valentin Longchamp <valentin.longch...@keymile.com>
> Cc: Holger Brunck <holger.bru...@keymile.com>
> ---
> $ /home/git/linux-denx/scripts/checkpatch.pl
> 0001-arm-post-memory-fix-bug-if-sdram-base-0x00000000.patch
> total: 0 errors, 0 warnings, 14 lines checked
>
> 0001-arm-post-memory-fix-bug-if-sdram-base-0x00000000.patch has no obvious
> style problems and is ready for submission.
>
> post/drivers/memory.c | 7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/post/drivers/memory.c b/post/drivers/memory.c
> index 69973c0..d25e323 100644
> --- a/post/drivers/memory.c
> +++ b/post/drivers/memory.c
> @@ -466,10 +466,11 @@ static int memory_post_test_regions(unsigned long
> start, unsigned long size)
>
> for (i = 0; i < (size >> 20) && (!ret); i++) {
> if (!ret)
> - ret = memory_post_test_patterns(i << 20, 0x800);
> - if (!ret)
> - ret = memory_post_test_patterns((i << 20) + 0xff800,
> + ret = memory_post_test_patterns(start + (i << 20),
> 0x800);
> + if (!ret)
> + ret = memory_post_test_patterns(start + (i << 20) +
> + 0xff800, 0x800);
> }
>
> return ret;
Thanks for fixing that Heiko: we only have boards starting @0x00000000 so I did
not notice this bug.
Ackey-by: Valentin Longchamp <valentin.longch...@keymile.com>
Valentin
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
