From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Wed, 7 Apr 2010 11:25:01 -0500
Subject: [U-Boot] [PATCH] POST: Added ECC memory test for mpc83xx.
In-Reply-To: <o2led82fe3e1004070730gf4646359kbbcf7a62ae391b3a@mail.gmail.com>
References: <1270219748-4930-1-git-send-email-michael.zaidman@gmail.com>
	<o2hed82fe3e1004021722hd9d884aai7abf87954de0b259@mail.gmail.com>
	<g2k660c0f821004060228l59c38ad5mf8590a4d22d9d644@mail.gmail.com>
	<w2ved82fe3e1004060740q5a37c4a7j786b4c7f3bc536f@mail.gmail.com>
	<t2k660c0f821004060829j139fdd7ao9fef84c77a30e96e@mail.gmail.com>
	<20100406194050.f4606e7a.kim.phillips@freescale.com>
	<o2led82fe3e1004070730gf4646359kbbcf7a62ae391b3a@mail.gmail.com>
Message-ID: <20100407162501.GA9249@loki.buserror.net>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Wed, Apr 07, 2010 at 09:30:55AM -0500, Timur Tabi wrote:
> On Tue, Apr 6, 2010 at 7:40 PM, Kim Phillips <kim.phillips@freescale.com> wrote:
> 
> >> > You should add a comment explaining why you're not using I/O
> >> > accessors.
> >>
> >> Ok, I will add the explanation.
> >
> > use raw i/o accessors please.
> 
> But what about his performance concern?  Shouldn't it be okay for him
> to do direct writes and then follow-up with a sync(), as long as he
> comments it?

That's exactly what raw accessors are for.  They don't include
synchronization (and unfortunately this somehow gets tied up in whether
endian conversions are done).

-Scott