On Fri, 4 Dec 2009 09:51:45 +0100 Stefan Roese <s...@denx.de> wrote: > The memory controller could already be enabled, when spd_sdram() is > called. This could be the case for example, when the SDRAM is initialized > by the JTAG debugger. > > Signed-off-by: Stefan Roese <s...@denx.de> > Cc: Reinhard Arlt <reinhard.a...@esd.eu> > Cc: Kim Phillips <kim.phill...@freescale.com>
thanks for this, Stefan. > + /* > + * First disable the memory controller (could be enabled > + * by the debugger) > + */ > + ddr->sdram_cfg &= ~SDRAM_CFG_MEM_EN; > + asm("sync;isync"); I realize that this file already doesn't use accessor functions to any large extent, but that doesn't mean that we should continue this bad habit. Kim _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot