Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-09 Thread Jerry Van Baren
Jerry Van Baren wrote: > Liu Dave-R63238 wrote: >>> According to Ira, the DMA method was faster than the cpu method: >>> "It makes the DMA initialization normal speed again. The DMA >>> in the for loop takes the longest (as expected). >>> >>> So yes, strangely it (enabling the icache) makes a HUGE

Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-09 Thread Jerry Van Baren
Liu Dave-R63238 wrote: >> According to Ira, the DMA method was faster than the cpu method: >> "It makes the DMA initialization normal speed again. The DMA >> in the for loop takes the longest (as expected). >> >> So yes, strangely it (enabling the icache) makes a HUGE >> difference. The total tim

Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-08 Thread Ira W. Snyder
On Wed, Jul 08, 2009 at 07:32:26PM -0500, Peter Tyser wrote: > On Thu, 2009-07-09 at 08:24 +0800, Liu Dave-R63238 wrote: > > > When SDRAM ECC is enabled and CONFIG_ECC_INIT_VIA_DDRCONTROLLER is not > > > defined use DMA to set SDRAM to a known state. Previously a > > > sequence of > > > 64-bit st

Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-08 Thread Liu Dave-R63238
> According to Ira, the DMA method was faster than the cpu method: > "It makes the DMA initialization normal speed again. The DMA > in the for loop takes the longest (as expected). > > So yes, strangely it (enabling the icache) makes a HUGE > difference. The total time is <3 seconds now. It is

Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-08 Thread Peter Tyser
On Thu, 2009-07-09 at 08:24 +0800, Liu Dave-R63238 wrote: > > When SDRAM ECC is enabled and CONFIG_ECC_INIT_VIA_DDRCONTROLLER is not > > defined use DMA to set SDRAM to a known state. Previously a > > sequence of > > 64-bit stores was used. > > IIRC, the DMA init SDRAM is slower than the 64bit s

Re: [U-Boot] [PATCH 1/3 v2] 83xx: Default to using DMA to initializeSDRAM

2009-07-08 Thread Liu Dave-R63238
> When SDRAM ECC is enabled and CONFIG_ECC_INIT_VIA_DDRCONTROLLER is not > defined use DMA to set SDRAM to a known state. Previously a > sequence of > 64-bit stores was used. IIRC, the DMA init SDRAM is slower than the 64bit stores. It is why I added these code here. I suggest to keep the way.