On Tue, Mar 8, 2011 at 10:08 PM, Aneesh V aneesh@ti.com wrote:
On Wednesday 09 March 2011 10:16 AM, John Rigby wrote:
On Mon, Feb 28, 2011 at 4:46 AM, Aneesh Vaneesh@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 Vaneesh@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