public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

             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