From: "Thomas Schäfer" <tschaefer@giga-stream.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] ECC DIMM's with walnut board
Date: Mon, 28 Apr 2003 16:07:54 +0200 [thread overview]
Message-ID: <FIEJJAKAABLMHFKFECNJGEAMGPAA.tschaefer@giga-stream.de> (raw)
Hi all,
I tried to boot a walnut board with a 256 ECC DIMM using u-boot 0.3.0, but
didn't succeed. When examining the code in the function spd_sdram in
cpu/ppc4xx/spd_sdram, I found that the ECC configuration code is probably
wrong. I suppose, that the following lines
/* now check for ECC ability of module. We only support ECC
* on 32 bit wide devices with 8 bit ECC.
*/
if ( (read_spd(11)==2) && ((read_spd(6)==40) || (read_spd(14)==8)) ){
sdram0_ecccfg=0xf<<SDRAM0_ECCCFG_SHIFT;
ecc_on = 1;
should be replaced with
/* now check for ECC ability of module. We only support ECC
* on 32 bit wide devices with 8 bit ECC.
*/
==> if ( (read_spd(11)==2) && ((read_spd(6)==40) &&
read_spd(14)==8)) ){ <==
sdram0_ecccfg=0xf<<SDRAM0_ECCCFG_SHIFT;
ecc_on = 1;
The reason is, that the if statement in the first code is also true for 72
bit ECC devices. So ECC is enabled for such a device which fails in the
following. The changed code works for me with several DIMM types (32MB
without ECC, 256 MB with ECC and without ECC), but I'm not an expert with
all types of SDRAM and so I'm not sure if I fully understood the code in
that place. Any ideas, comments?
Best regards
Thomas Sch?fer
____________________________________
GIGA STREAM - UMTS Technologies GmbH
Konrad-Zuse-Str. 7
66115 Saarbr?cken
Tel.: + 49 (0)681 / 95916 - 203
Fax: + 49 (0)681 / 95916 - 100
E-mail: tschaefer@giga-stream.de
next reply other threads:[~2003-04-28 14:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-04-28 14:07 Thomas Schäfer [this message]
2003-04-28 17:18 ` [U-Boot-Users] ECC DIMM's with walnut board Kenneth Johansson
2003-05-22 13:10 ` Wolfgang Denk
-- strict thread matches above, loose matches on Subject: below --
2003-04-29 8:23 Thomas Schäfer
2003-04-29 8:19 Thomas Schäfer
2003-04-25 13:15 Thomas Schäfer
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=FIEJJAKAABLMHFKFECNJGEAMGPAA.tschaefer@giga-stream.de \
--to=tschaefer@giga-stream.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