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

2009-07-14 Thread Peter Tyser
On Tue, 2009-07-14 at 12:32 -0500, Kim Phillips wrote: > On Tue, 14 Jul 2009 09:42:00 -0500 > Peter Tyser wrote: > > > > I'm assuming Kim would prefer not to use DMA and thus this patch should > > be discarded. > > yeah, I prefer speed :) It looked like Ira tested using the DDR controller for

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

2009-07-14 Thread Kim Phillips
On Tue, 14 Jul 2009 09:42:00 -0500 Peter Tyser wrote: > I'm assuming Kim would prefer not to use DMA and thus this patch should > be discarded. yeah, I prefer speed :) > Kim, let me know if this is the case and I'll resubmit a patch just to > fix the build error. Patches 2 and 3 are still app

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

2009-07-14 Thread Wolfgang Denk
Dear Peter Tyser, In message <1247582520.30723.4.ca...@localhost.localdomain> you wrote: > > There was some discussion about whether to use DMA to initialize SDRAM > on the 83xx architecture (which this patch does). After some > benchmarking by Ira Synder it turns out that using DMA is actually >

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

2009-07-14 Thread Peter Tyser
On Tue, 2009-07-14 at 11:22 +0200, Wolfgang Denk wrote: > Dear Kim, > > In message <1247098309-29122-2-git-send-email-pty...@xes-inc.com> Peter Tyse > 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

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

2009-07-14 Thread Kumar Gala
On Jul 14, 2009, at 4:22 AM, Wolfgang Denk wrote: > Dear Kim, > > In message <1247098309-29122-2-git-send-email-pty...@xes-inc.com> > Peter Tyse 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

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

2009-07-14 Thread Wolfgang Denk
Dear Kim, In message <1247098309-29122-2-git-send-email-pty...@xes-inc.com> Peter Tyse 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. > > Signed-off-by: Peter Tys

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

2009-07-09 Thread Jerry Van Baren
Ira W. Snyder wrote: > On Wed, Jul 08, 2009 at 05:58:50PM -0700, Ira W. Snyder wrote: >> 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: [snip] >> Both the DMA and CPU methods are definitely on the same order of >> magnitu

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

2009-07-09 Thread Ira W. Snyder
On Wed, Jul 08, 2009 at 05:58:50PM -0700, Ira W. Snyder wrote: > 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 S