From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Sakoman Date: Sun, 26 Sep 2010 07:57:24 -0700 Subject: [U-Boot] [PATCH] ARMV7: OMAP4: Calculate SDRAM size In-Reply-To: <20100926143707.AFC3E134F52@gemini.denx.de> References: <1285276376.21276.30.camel@quadra> <4C9E0E7E.7090900@gmail.com> <4C9F3E4F.3040403@gmail.com> <20100926143707.AFC3E134F52@gemini.denx.de> Message-ID: <1285513044.24562.10.camel@quadra> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sun, 2010-09-26 at 16:37 +0200, Wolfgang Denk wrote: > Dear Nishanth Menon, > > In message <4C9F3E4F.3040403@gmail.com> you wrote: > > > > > I think you are referring to SDRAM initialization. Yes, it is assumed > > > that SDRAM initialization(at least the DMM part of it) is done before > > > the call to sdram_size(). The right location for this seems > > > to be the initial part of sdram_init(). > > > > thanks for the clarification.. NOR boot is usually going to have initial > > part in XIP -> we'd traditionally relocate to SDRAM before the call to > > sdram_size() happens. > > Yes, this was the way how many ARM boards id, and this is what I'm > trying to explain: this is WRONG. > > SDRAM configuration must be done while still running from NOR, i. e. > before relocation. Only then free reconfiguration, auto-sizing etc. is > possible. I'll make sure that this is done properly when(if?) a patch series for an OMAP4 board with NOR boot is prepared. Getting back to the subject patch, though :-) This patch fixes a bug in the current Panda board support in rc2. Without this patch U-boot will return an erroneous size for available RAM. We should consider it for inclusion as soon as practical. Regards, Steve