From: Siarhei Siamashka <siarhei.siamashka@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] OMAP3: workaround for ARM Cortex-A8 erratum 725233
Date: Sun, 7 Feb 2010 19:54:06 +0200 [thread overview]
Message-ID: <201002071954.06342.siarhei.siamashka@gmail.com> (raw)
In-Reply-To: <4B6ED210.4050602@windriver.com>
On Sunday 07 February 2010, Tom wrote:
> Siarhei Siamashka wrote:
> > 725233: PLD instructions executed with PLD data forwarding
> > enabled can result in a processor deadlock
> >
> > Signed-off-by: Siarhei Siamashka <siarhei.siamashka@gmail.com>
>
> Please add a detailed comment on the errata.
The short summary is pretty much complete (PLD data forwarding is not working
correctly and needs to be turned off to prevent problems). Though it really
makes sense to add that:
1. only r1pX revisions of Cortex-A8 are affected
2. performance impact is practically non-existant
The detailed description of this erratum is available in the official
Cortex-A8 errata list. I'm not sure about directly copying full erratum
description text here. Anyway, this workaround would be better submitted
by somebody TI. I'm just "guilty" of triggering this deadlock with some
ARM NEON optimizations from pixman library, so feel a bit of responsibility
for it too.
> Also looks like this jumping to ROM code.
> Can this be done without a ROM code call ?
I just added it to the part of code which is already doing similar things
(sets L1NEON and other bits from AUXCR with some icky looking assembly via
ROM code call). Most likely it can be also done in some other way.
--
Best regards,
Siarhei Samashka
next prev parent reply other threads:[~2010-02-07 17:54 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-07 2:19 [U-Boot] [PATCH] OMAP3: workaround for ARM Cortex-A8 erratum 725233 Siarhei Siamashka
2010-02-07 2:19 ` [U-Boot] [PATCH] OMAP3: remove useless ASA bit from AUXCR Siarhei Siamashka
2010-02-07 14:45 ` [U-Boot] [PATCH] OMAP3: workaround for ARM Cortex-A8 erratum 725233 Tom
2010-02-07 17:54 ` Siarhei Siamashka [this message]
2010-02-08 13:15 ` Måns Rullgård
2010-02-08 15:27 ` Siarhei Siamashka
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=201002071954.06342.siarhei.siamashka@gmail.com \
--to=siarhei.siamashka@gmail.com \
--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