From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Re: [Patch] JSE SDRAM size probing
Date: Sat, 21 Jan 2006 13:24:56 +0100 [thread overview]
Message-ID: <200601211324.56506.sr@denx.de> (raw)
In-Reply-To: <dqs4i8$srl$1@sea.gmane.org>
Hi Steve,
On Saturday 21 January 2006 02:59, Stephen Williams wrote:
> > In message <dqrtqm$bcg$1@sea.gmane.org> you wrote:
> >>> This patch adds to JSE support a probe for various plausible SDRAM
> >>> configurations available for the board.
> >
> > I know that this is board-specific code, but I tend to reject this
> > patch anyway. Why don't you use the existing code to detet memory
> > sizes or to test memory for errors?
>
> Uh, because I can't find it? This board doesn't have an SPD bus, if
> that is what you mean.
Please take a look at cpu/ppc4xx/sdram.c. It handles right now only the first
SDRAM bank (405 and DDR on 440) by defining "CONFIG_SDRAM_BANK0" in your
board config file. This is done with memory size autodetection using the
functions Wolfgang pointed out.
A few remarks:
- We will add some additional defines in the near future to allow to
configure further SDRAM parameters like CAS latency. With these setting
the timing regs (TR1, RTR) will be generated depending on the SDRAM
clock.
- You would have to enhance the code for multiple SDRAM banks.
- The config table "CFG_SDRAM_TABLE" has to be extended for 256MByte support.
Best regards,
Stefan
prev parent reply other threads:[~2006-01-21 12:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-20 23:05 [U-Boot-Users] [Patch] JSE SDRAM size probing Stephen Williams
2006-01-20 23:09 ` Wolfgang Denk
2006-01-21 0:04 ` [U-Boot-Users] " Stephen Williams
2006-01-21 1:39 ` Wolfgang Denk
2006-01-21 1:59 ` Stephen Williams
2006-01-21 10:12 ` Wolfgang Denk
2006-01-21 13:16 ` [U-Boot-Users] New Boards uboot at sundmangroup.com
2006-01-21 13:38 ` Wolfgang Denk
2006-01-21 16:32 ` [U-Boot-Users] Re: [Patch] JSE SDRAM size probing Stephen Williams
2006-01-21 17:59 ` Wolfgang Denk
2006-01-21 18:49 ` Stephen Williams
2006-01-21 20:55 ` Wolfgang Denk
2006-01-21 12:24 ` Stefan Roese [this message]
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=200601211324.56506.sr@denx.de \
--to=sr@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