[U-Boot] [PATCH 1/5] 8xxx: Add 'ecc' command

Wolfgang Denk wd at denx.de
Sat Oct 24 19:21:10 CEST 2009


Dear Peter Tyser,

In message <1256258353-25589-1-git-send-email-ptyser at xes-inc.com> you wrote:
> Add a new 'ecc' command to interact with the 85xx and 86xx DDR ECC
> registers.  The 'ecc' command can inject data/ECC errors to simulate
> errors and provides an 'info' subcommand which displays ECC error
> information such as failure address, read vs expected data/ECC,
> physical signal which failed, single-bit error count, and multiple bit
> error occurrence.  An example of the 'ecc info' command follows:

We already have similar commands for other architectures, see for
example cpu/mpc83xx/ecc.c

I'm not sure if it's possible to use a common implementation, but I
would like to ask you to check if this is possible.

In any case I ask that we use a common user interface for both
implementations. It makes no sense that the same command name behaves
differently on different boards (even from the same vendor).

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
"...one of the main causes of the fall of the Roman Empire was  that,
lacking  zero,  they had no way to indicate successful termination of
their C programs."                                     - Robert Firth


More information about the U-Boot mailing list