All of lore.kernel.org
 help / color / mirror / Atom feed
From: shawn.guo@freescale.com (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: add revision info for PL310 errata 588369 and 727915
Date: Mon, 7 Jul 2014 22:07:09 +0800	[thread overview]
Message-ID: <20140707140708.GU30239@dragon> (raw)
In-Reply-To: <1404285816-3763-1-git-send-email-shawn.guo@freescale.com>

On Wed, Jul 02, 2014 at 03:23:36PM +0800, Shawn Guo wrote:
> Add revision info for PL310_ERRATA_588369 and PL310_ERRATA_727915 to
> help people understand if they need to enable the errata for their
> hardware.
> 
> Signed-off-by: Shawn Guo <shawn.guo@freescale.com>

Any comments on this patch?  Otherwise, I'm going to throw it into
the patch system.

Shawn

> ---
>  arch/arm/mm/Kconfig | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig
> index eda0dd0ab97b..9fb8a02b7e35 100644
> --- a/arch/arm/mm/Kconfig
> +++ b/arch/arm/mm/Kconfig
> @@ -907,8 +907,8 @@ config PL310_ERRATA_588369
>  	   They are architecturally defined to behave as the execution of a
>  	   clean operation followed immediately by an invalidate operation,
>  	   both performing to the same memory location. This functionality
> -	   is not correctly implemented in PL310 as clean lines are not
> -	   invalidated as a result of these operations.
> +	   is not correctly implemented in PL310 prior to r2p0 (fixed in r2p0)
> +	   as clean lines are not invalidated as a result of these operations.
>  
>  config PL310_ERRATA_727915
>  	bool "PL310 errata: Background Clean & Invalidate by Way operation can cause data corruption"
> @@ -919,7 +919,8 @@ config PL310_ERRATA_727915
>  	  PL310 can handle normal accesses while it is in progress. Under very
>  	  rare circumstances, due to this erratum, write data can be lost when
>  	  PL310 treats a cacheable write transaction during a Clean &
> -	  Invalidate by Way operation.
> +	  Invalidate by Way operation.  Revisions prior to r3p1 are affected by
> +	  this errata (fixed in r3p1).
>  
>  config PL310_ERRATA_753970
>  	bool "PL310 errata: cache sync operation may be faulty"
> -- 
> 1.8.3.2
> 

  reply	other threads:[~2014-07-07 14:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-02  7:23 [PATCH] ARM: add revision info for PL310 errata 588369 and 727915 Shawn Guo
2014-07-07 14:07 ` Shawn Guo [this message]
2014-07-07 15:44   ` Russell King - ARM Linux
2014-07-08  2:09     ` Shawn Guo

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=20140707140708.GU30239@dragon \
    --to=shawn.guo@freescale.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.