From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] PPC405EX CHIP_21 erratum
Date: Mon, 2 May 2011 13:50:42 +0200 [thread overview]
Message-ID: <201105021350.43119.sr@denx.de> (raw)
In-Reply-To: <4DBAED2A.4030705@harris.com>
Hi Steve,
On Friday 29 April 2011 18:54:02 Steven A. Falco wrote:
> We've been bitten by the PPC405EX CHIP_21 erratum.
How did it affect you exactly? Was an incorrect PVR detected? Did this result
in some problems (Linux etc)?
> I've looked through the
> U-Boot code, but it doesn't appear that there is a work-around for this
> one.
Correct, there currently is no workaround.
> The following patch is my adaptation of AMCC's suggestion as to the fix.
> But I have to say that I don't care for it, if for no other reason that it
> will break if a rev E comes out.
>
> Also, I made it #ifdef'd on CONFIG_405EX, but it will not work on a 405EXr.
> And I have seen a 405EX report its PVR as if it were a 405EXr. It's not
> looking good for the suggested work-around.
>
> AMCC claims this can be fixed in hardware, by always doing a double reset.
> Naturally, that is hard to implement and would mean massive rework of
> existing boards.
Yes, there are many 405EX(r) boards in the field already. Such an hardware
workaround should really be avoided if possible.
> So I am contemplating a different work-around, whereby software always
> resets the board on a cold boot (if such a thing can be reliably
> detected). That would hopefully be the equivalent of the hardware double
> reset, and would not be dependent on specific PVR values, making it more
> "future proof".
>
> Has anyone else run across this? Do you have similar concerns about the
> patch?
No, I have not seen this problem before. And yes, I would also prefer your
"alternative suggestion", with the always reboot after powerup optionally
built into the U-Boot image. But as you already mentioned, we would have to
find a way to reliably detect the powerup reset, so that we don't end in an
reset-loop.
Cheers,
Stefan
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de
next prev parent reply other threads:[~2011-05-02 11:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-29 16:54 [U-Boot] PPC405EX CHIP_21 erratum Steven A. Falco
2011-05-02 11:50 ` Stefan Roese [this message]
[not found] <4DBF1F33.4040101@harris.com>
2011-05-02 21:44 ` Steven A. Falco
2011-05-03 13:07 ` Stefan Roese
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=201105021350.43119.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