[U-Boot] [PATCH] Add the "mfill" and "mcheck" memory commands.

[Wolfgang Denk]

Dear Benoit Monin,

In message <6148322.23690.1236356233840.JavaMail.www at wwinf8204> you wrote:
> 
> > > The configuration option is CONFIG_MFILL and this only takes effect if
> > > the memory commands are activated globally (CONFIG_CMD_MEM).
> > 
> > What exactly is the purpose of this patch? Looks like an extremely
> > simple memory test to me. Don't we have already a rich selection of
> > better ones available yet?
> > 
> I should have detailed more the intent of those commands. We have here

Indeed.

> a video controller embedded in an fpga with its own video ram. We use
> mfill to fill the video ram as fast as possible with variable data for
> each vram address. And we use mcheck to verify (as fast as possible)
> the content of the vram.
> 
> So this is not intended to test the main memory but it is a development
> and debug tool for peripheral containing memory.

You could use the existing memory tests to do the same.

> I've sent a patch because they are useful to us and generic enough,
> and might be handy for debugging accesses to memory mapped behind
> programmable logic.
> 
> But maybe this debug tool is beyond the scope of U-Boot ?

I think it duplicates existing functionality.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Men don't talk peace unless they're ready to back it up with war.
	-- Col. Green, "The Savage Curtain", stardate 5906.4