[U-Boot] [PATCH] arm, post, memory: fix bug if sdram base != 0x00000000
Valentin Longchamp
valentin.longchamp at keymile.com
Fri Oct 7 08:53:59 CEST 2011
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 <hs at denx.de>
> Cc: Valentin Longchamp <valentin.longchamp at keymile.com>
> Cc: Holger Brunck <holger.brunck at 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.longchamp at keymile.com>
Valentin
More information about the U-Boot
mailing list