All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeroen Hofstee <jeroen@myspectrum.nl>
To: u-boot@lists.denx.de
Subject: [U-Boot] ti, am3517: errata 430973 workaround
Date: Mon, 08 Dec 2014 23:19:10 +0100	[thread overview]
Message-ID: <548623DE.10704@myspectrum.nl> (raw)

Hi,

A while ago [1], a RFC was posted to disable workaround for
besides others, errata 430973. It is a bit unclear to me which
revision actually need this workaround, but as suggested in
[2] also enabling this workaround in Linux seem to make some
weird problems go away in linux (signal 4, bad instruction,
11 segfaults etc).

As said, I am a bit in doubt why this works. The board in question
is a tam3517 derived one:

cat /proc/cpuinfo
Processor    : ARMv7 Processor rev 7 (v7l)
BogoMIPS    : 397.57
Features    : swp half thumb fastmult vfp edsp neon vfpv3 tls
CPU implementer    : 0x41
CPU architecture: 7
CPU variant    : 0x1
CPU part    : 0xc08
CPU revision    : 7

Which makes this a r1p7 I assume, and hence the workaround
of linux, CONFIG_ARM_ERRATA_430973, "This option enables the
workaround for the 430973 Cortex-A8 (r1p0..r1p2) erratum",
should not be needed it seems.

On the other hand Andreas Bie?man, wrote at [3]
"I have rev 20.0 from 13-Apr-10. The three mentioned errata
should be fixed in r2p1." note, this mentions r2p1 not r1p2!

Since I don't have access to "ARM Core Cortex-A8 (AT400/AT401)
errata", I cannot look this up. Hence the question, is u-boot
wrong by enabling this workaround for a r1p7 revision or is the
comment in the kernel flawed? (or am I missing something else..)

If someone could shed some light on this it would be appreciated.

Regards,
Jeroen

[1] http://lists.denx.de/pipermail/u-boot/2013-July/158377.html
[2] http://lists.denx.de/pipermail/u-boot/2013-July/158404.html
[3] http://lists.denx.de/pipermail/u-boot/2013-July/158386.html

             reply	other threads:[~2014-12-08 22:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-08 22:19 Jeroen Hofstee [this message]
2014-12-09  8:51 ` [U-Boot] ti, am3517: errata 430973 workaround Andreas Bießmann
2014-12-09 21:45 ` Robert Nelson
2014-12-10 10:25   ` Jeroen Hofstee

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=548623DE.10704@myspectrum.nl \
    --to=jeroen@myspectrum.nl \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.