From mboxrd@z Thu Jan 1 00:00:00 1970 From: Aneesh V Date: Wed, 09 Mar 2011 10:38:53 +0530 Subject: [U-Boot] [PATCH 17/22] omap4: calculate EMIF register values In-Reply-To: References: <1298893591-17636-1-git-send-email-aneesh@ti.com> <1298893591-17636-18-git-send-email-aneesh@ti.com> Message-ID: <4D770B65.5050502@ti.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 Wednesday 09 March 2011 10:16 AM, John Rigby wrote: > On Mon, Feb 28, 2011 at 4:46 AM, Aneesh V wrote: >> Calculate EMIF register values based on AC timing parameters >> from the SDRAM datasheet and the DDR frequency rather than >> using the hard-coded values. >> >> For a new board the user doen't have to go through the tedious >> process of calculating the register values. Instead, just >> provide the AC timings from the device data sheet as input >> and the driver will automatically calculate the register values. >> >> Signed-off-by: Aneesh V >> --- > > > I get some warnings from arch/arm/cpu/armv7/omap4/emif.c: > emif.c: In function ?sdram_init?: > emif.c:1164:43: warning: ?section_map? may be used uninitialized in > this function > emif.c:1166:27: warning: ?lis_map_regs_calculated$dmm_lisa_map_3? may > be used uninitialized in this function > emif.c: In function ?sdram_init?: > emif.c:1166:27: warning: ?lis_map_regs_calculated$dmm_lisa_map_3? may > be used uninitialized in this function > > Looking at the code I see that these are initialized but the compiler > can't firgure that out. I don't see the warnings with my tool-chain ((Sourcery G++ Lite 2010q1-202) GCC 4.4.1) I tried immediately after this patch and also after applying all the patches. Which compiler are you using? best regards, Aneesh