On Tue, Mar 8, 2011 at 10:08 PM, Aneesh V <ane...@ti.com> wrote: > On Wednesday 09 March 2011 10:16 AM, John Rigby wrote: >> >> On Mon, Feb 28, 2011 at 4:46 AM, Aneesh V<ane...@ti.com> 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<ane...@ti.com> >>> --- >> >> >> 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?
The latest Linaro cross toolchain: $ arm-linux-gnueabi-gcc --version arm-linux-gnueabi-gcc (Ubuntu/Linaro 4.5.2-1ubuntu3ppa1) 4.5.2 > > best regards, > Aneesh > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot