From mboxrd@z Thu Jan 1 00:00:00 1970 From: Valentin Longchamp Date: Fri, 07 Oct 2011 08:53:59 +0200 Subject: [U-Boot] [PATCH] arm, post, memory: fix bug if sdram base != 0x00000000 In-Reply-To: <1317969600-32496-1-git-send-email-hs@denx.de> References: <1317969600-32496-1-git-send-email-hs@denx.de> Message-ID: <4E8EA207.90401@keymile.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > Cc: Valentin Longchamp > Cc: Holger Brunck > --- > $ /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