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

[John Rigby]

On Tue, Mar 8, 2011 at 10:08 PM, Aneesh V <aneesh at 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<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?

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
>