From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] cmd_mem: Decode the mtest start and end values from fdt
Date: Fri, 01 Mar 2013 23:48:41 +0100 [thread overview]
Message-ID: <20130301224841.CE7A0200527@gemini.denx.de> (raw)
In-Reply-To: <ff1257d2-b332-477f-915d-0851e5ef944f@CO9EHSMHS030.ehs.local>
Dear Jagannadha Sutradharudu Teki,
In message <ff1257d2-b332-477f-915d-0851e5ef944f@CO9EHSMHS030.ehs.local> you wrote:
>
> I think the entire change are not good as it touches the environment concept,
> Can you suggest any better way we could do or it may not be possible/require the mem test values
> alter though devicetree.
May I ask _why_ you would want to do this in the first place?
It does not seem to make sense to me.
Please also note that "mtest" is basicly useless. Totally useless,
to be honest. It performs plain read / write cycles, and unless you
have very basic memory issues this will always work - if you have
such, these get detected when running get_ram_size().
"mtest" was once added as an explicit requirement of a customer to
meet some formal specification requests. It is a ZERO practical use.
If you want to do memory testing, then use the memory test functions
of the POST framework. OK, these will not exercise any burst mode
accesses either, but they are at least a bit more useful.
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
The evolution of the human race will not be accomplished in the ten
thousand years of tame animals, but in the million years of wild
animals, because man is and will always be a wild animal.
- Charles Galton Darwin
next prev parent reply other threads:[~2013-03-01 22:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-01 14:32 [U-Boot] [PATCH] cmd_mem: Decode the mtest start and end values from fdt Jagannadha Sutradharudu Teki
2013-03-01 16:06 ` Wolfgang Denk
2013-03-01 19:15 ` Tom Rini
2013-03-01 19:24 ` Simon Glass
[not found] ` <ff1257d2-b332-477f-915d-0851e5ef944f@CO9EHSMHS030.ehs.local>
2013-03-01 22:48 ` Wolfgang Denk [this message]
[not found] ` <51f05678-4118-4be8-bb94-ad295c407602@DB3EHSMHS015.ehs.local>
2013-03-04 9:44 ` Wolfgang Denk
2013-03-08 19:08 ` [U-Boot] [PATCH] Feature Removal: disable "mtest" command by default Wolfgang Denk
2013-03-08 19:41 ` Ira W. Snyder
2013-03-08 20:50 ` Wolfgang Denk
2013-03-08 20:51 ` [U-Boot] [PATCH v2] " Wolfgang Denk
2013-03-11 19:57 ` Tom Rini
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=20130301224841.CE7A0200527@gemini.denx.de \
--to=wd@denx.de \
--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