[U-Boot] [PATCH] MPC512x: workaround data corruption for unaligned local bus accesses

Anatolij Gustschin agust at denx.de
Tue Jun 29 15:34:06 CEST 2010


Dear Wolfgang,

On Tue, 29 Jun 2010 14:47:14 +0200
Wolfgang Denk <wd at denx.de> wrote:

...
> > BTW, shouldn't we fix print_buffer() also? do_mem_md() doesn't use
> > memcpy() and the issue with corrupted dumps still remains here.
> > I'm testing a patch to fix it. Will submit it soon.
> 
> I though about this, too. But then I decided against it. "md" is
> intended to to exactly what the user requests - as is, we can easily
> demonstrate the issue. I consider this an extremely useful debug
> feature. If I command the memory to be read in units of 32 bits I
> really mean that. A "fix" ins print_buffer() whould hush up the
> symptoms, so the problem would be much harder to detect.

Thanks for the explanation, I didn't thought about it. This is
indeed a reasonable debug feature.

Best regards,
Anatolij


More information about the U-Boot mailing list