Dear Benoit Monin, In message <6148322.23690.1236356233840.javamail....@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: w...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot