[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