[U-Boot] [PATCH 17/22] omap4: calculate EMIF register values

Aneesh V aneesh at ti.com
Wed Mar 9 06:08:53 CET 2011


On Wednesday 09 March 2011 10:16 AM, John Rigby wrote:
> On Mon, Feb 28, 2011 at 4:46 AM, Aneesh V<aneesh at 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<aneesh at 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?

best regards,
Aneesh


More information about the U-Boot mailing list