[U-Boot] [PATCH] powerpc/83xx: fix sdram initialization for keymile boards

Heiko Schocher hs at denx.de
Thu Nov 10 15:38:13 CET 2011


Hello Holger,

Holger Brunck wrote:
> Hi Heiko,
> 
> On 11/10/2011 12:59 PM, Heiko Schocher wrote:
>> Holger Brunck wrote:
>>> From: Andreas Huber <andreas.huber at keymile.com>
>>>
>>> commit b11f53f3 (keymile: Fix Coding style issues for keymile boards)
>>> introduces a bug according the SDRAM initialization for all
>>> km83xx boards.
>>>
>>> im->ddr.sdram_cfg |= SDRAM_CFG_MEM_EN;
>>> was replaced with
>>> out_be32(&im->ddr.sdram_cfg, SDRAM_CFG_MEM_EN);
>>> and this is wrong, because this overwrites the intial value
>>> CONFIG_SYS_DDR_SDRAM_CFG.
>>>
>>> Signed-off-by: Andreas Huber <andreas.huber at keymile.com>
>>> Signed-off-by: Holger Brunck <holger.brunck at keymile.com>
>>> cc: Heiko Schocher <hs at denx.de>
>>> cc: Kim Phillips <kim.phillips at freescale.com>
>>> ---
>>>  board/keymile/km83xx/km83xx.c |    3 ++-
>>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/board/keymile/km83xx/km83xx.c b/board/keymile/km83xx/km83xx.c
>>> index 17560c8..c0238c8 100644
>>> --- a/board/keymile/km83xx/km83xx.c
>>> +++ b/board/keymile/km83xx/km83xx.c
>>> @@ -217,7 +217,8 @@ int fixed_sdram(void)
>>>  	out_be32(&im->ddr.sdram_interval, CONFIG_SYS_DDR_INTERVAL);
>>>  	out_be32(&im->ddr.sdram_clk_cntl, CONFIG_SYS_DDR_CLK_CNTL);
>>>  	udelay(200);
>>> -	out_be32(&im->ddr.sdram_cfg, SDRAM_CFG_MEM_EN);
>>> +	out_be32(&im->ddr.sdram_cfg,
>>> +		SDRAM_CFG_MEM_EN | CONFIG_SYS_DDR_SDRAM_CFG);
>> Wouldn't a
>>
>> setbits_be32(&im->ddr.sdram_cfg, SDRAM_CFG_MEM_EN);
>>
>> be better?
>>
> 
> yes this works too and is slightly nicer, I will send an updated version.

Ok, thanks! Please fell free to add my

Acked-by: Heiko Schocher<hs at denx.de>

to it!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list