From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] ppc4xx: Don't use last 256 bytes of SDRAM, workaround for 440EPx CHIP 11 errata
Date: Thu, 20 Mar 2008 11:03:30 +0100 [thread overview]
Message-ID: <20080320100330.B00C624A87@gemini.denx.de> (raw)
In-Reply-To: Your message of "Thu, 20 Mar 2008 10:46:37 +0100." <1206006397-16582-1-git-send-email-sr@denx.de>
Hello Stefan,
in message <1206006397-16582-1-git-send-email-sr@denx.de> you wrote:
> Since 440EPx/GRx has problems with accessing the last 256 bytes of SDRAM via
> the Denali DDR/DDR2 controller, we set CONFIG_PRAM to 1 and reserve 1kByte
> of protected RAM. This way this memory will not get "touched" by U-Boot. And
> by passing "mem=${mem}" to the Linux kernel, Linux will not use this area
> either.
Note that this workaround is incomplete.
If you enable shared (between U-Boot and Linux) log buffer, then the
log buffer will go at the end of memory, and the pRAM area will be
allocated below. In this case the problems is not only still present,
but also very likely to hit pretty soon.
> This patch adds this workaround for the following 440EPx boards:
> sequoia, lwmon5. Others should probably follow this example.
OK, the default configs for Sequoia doesn't use shared log buffer, and
the lwmon5 uses CONFIG_ALT_LB_ADDR (i. e. it puts the log buffer in
OCM), so these boards will work.
But any changes to the configuration may easily break this again.
I don't like the idea that each affected board has to fix this
(probably differently) in it's board config file. I would prefer a
fix (common to all affected boards) in one central location.
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
"Unix is simple, but it takes a genius to understand the simplicity."
- Dennis Ritchie
next prev parent reply other threads:[~2008-03-20 10:03 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-20 9:46 [U-Boot-Users] [PATCH] ppc4xx: Don't use last 256 bytes of SDRAM, workaround for 440EPx CHIP 11 errata Stefan Roese
2008-03-20 10:03 ` Wolfgang Denk [this message]
2008-03-20 10:29 ` Stefan Roese
2008-03-20 11:16 ` Wolfgang Denk
2008-03-20 12:55 ` Kumar Gala
2008-03-20 13:57 ` Stefan Roese
2008-03-20 11:50 ` Dave Littell
2008-03-20 14:11 ` Stefan Roese
2008-03-20 16:06 ` Larry Johnson
2008-03-20 21:12 ` Stefan Roese
2008-03-27 2:50 ` Larry Johnson
2008-03-27 6:21 ` Stefan Roese
2008-03-27 18:36 ` Larry Johnson
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=20080320100330.B00C624A87@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