public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Benoit Monin <benoit.monin@laposte.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Add the "mfill" and "mcheck" memory commands.
Date: Fri,  6 Mar 2009 17:17:13 +0100 (CET)	[thread overview]
Message-ID: <6148322.23690.1236356233840.JavaMail.www@wwinf8204> (raw)

> De : "Wolfgang Denk"
> 
> Dear Benoit Monin,
> 
> In message <28907262.11403.1236090567026.JavaMail.www@wwinf8303> you wrote:
> > 
> > The mfill command writes the value of a counter to a memory area.
> > The mcheck command verifies the memory area against the counter value.
> > Those commands are useful for debugging memory/framebuffer controller.
> > 
> > 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
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.

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 ?

> Best regards,
> 
> Wolfgang Denk
> 
Sincerely,
Beno?t.

--


 Cr?ez votre adresse ?lectronique prenom.nom at laposte.net 
 1 Go d'espace de stockage, anti-spam et anti-virus int?gr?s.

             reply	other threads:[~2009-03-06 16:17 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-06 16:17 Benoit Monin [this message]
2009-03-08 22:20 ` [U-Boot] [PATCH] Add the "mfill" and "mcheck" memory commands Wolfgang Denk
  -- strict thread matches above, loose matches on Subject: below --
2009-03-03 14:29 Benoit Monin
2009-03-05 23:55 ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6148322.23690.1236356233840.JavaMail.www@wwinf8204 \
    --to=benoit.monin@laposte.net \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox