Add support for Cortex-A57 erratum 833471 workaround
authorSandrine Bailleux <sandrine.bailleux@arm.com>
Thu, 21 Apr 2016 10:10:52 +0000 (11:10 +0100)
committerSandrine Bailleux <sandrine.bailleux@arm.com>
Thu, 21 Apr 2016 11:59:59 +0000 (12:59 +0100)
Change-Id: I86ac81ffd7cd094ce68c4cceb01c16563671a063

docs/cpu-specific-build-macros.md
include/lib/cpus/aarch64/cortex_a57.h
lib/cpus/aarch64/cortex_a57.S
lib/cpus/cpu-ops.mk

index c41cbff697d093151a90c0ad115118413b6a1d62..202cea70457217938445c4ff8222142a18bb7413 100644 (file)
@@ -72,6 +72,9 @@ For Cortex-A57, following errata build flags are defined :
 *   `ERRATA_A57_829520`: This applies errata 829520 workaround to Cortex-A57
      CPU. This needs to be enabled only for revision <= r1p2 of the CPU.
 
+*   `ERRATA_A57_833471`: This applies errata 833471 workaround to Cortex-A57
+     CPU. This needs to be enabled only for revision <= r1p2 of the CPU.
+
 3.  CPU Specific optimizations
 ------------------------------
 
index 685b87b2e3d0e86e2607ccc55f9cd9740d3b987f..ac4ae57017bea84a33860256f513344b0f148367 100644 (file)
@@ -66,6 +66,7 @@
 #define CPUACTLR_DIS_OVERREAD          (1 << 52)
 #define CPUACTLR_NO_ALLOC_WBWA         (1 << 49)
 #define CPUACTLR_DCC_AS_DCCI           (1 << 44)
+#define CPUACTLR_FORCE_FPSCR_FLUSH     (1 << 38)
 #define CPUACTLR_DIS_STREAMING         (3 << 27)
 #define CPUACTLR_DIS_L1_STREAMING      (3 << 25)
 #define CPUACTLR_DIS_INDIRECT_PREDICTOR        (1 << 4)
index 63cb7579293fdfe85879322299587e8a8e0e9a67..60929a0506f22eab26b41a4b5f52f060294c7577 100644 (file)
@@ -277,6 +277,32 @@ apply_829520:
        ret
 endfunc errata_a57_829520_wa
 
+       /* ---------------------------------------------------
+        * Errata Workaround for Cortex A57 Errata #833471.
+        * This applies only to revision <= r1p2 of Cortex A57.
+        * Inputs:
+        * x0: variant[4:7] and revision[0:3] of current cpu.
+        * Clobbers : x0 - x5
+        * ---------------------------------------------------
+        */
+func errata_a57_833471_wa
+       /*
+        * Compare x0 against revision r1p2
+        */
+       cmp     x0, #0x12
+       b.ls    apply_833471
+#if LOG_LEVEL >= LOG_LEVEL_VERBOSE
+       b       print_revision_warning
+#else
+       ret
+#endif
+apply_833471:
+       mrs     x1, CPUACTLR_EL1
+       orr     x1, x1, #CPUACTLR_FORCE_FPSCR_FLUSH
+       msr     CPUACTLR_EL1, x1
+       ret
+endfunc errata_a57_833471_wa
+
        /* -------------------------------------------------
         * The CPU Ops reset function for Cortex-A57.
         * Clobbers: x0-x5, x15, x19, x30
@@ -330,6 +356,11 @@ func cortex_a57_reset_func
        bl      errata_a57_829520_wa
 #endif
 
+#if ERRATA_A57_833471
+       mov     x0, x15
+       bl      errata_a57_833471_wa
+#endif
+
        /* ---------------------------------------------
         * Enable the SMP bit.
         * ---------------------------------------------
index aa808b61b7cea9b93436b064aac3d13ebf147c7c..0659bff9c6d8a216a42866fdef3d78bea5ca71d2 100644 (file)
@@ -90,6 +90,10 @@ ERRATA_A57_828024    ?=0
 # only to revision <= r1p2 of the Cortex A57 cpu.
 ERRATA_A57_829520      ?=0
 
+# Flag to apply erratum 833471 workaround during reset. This erratum applies
+# only to revision <= r1p2 of the Cortex A57 cpu.
+ERRATA_A57_833471      ?=0
+
 # Process ERRATA_A53_826319 flag
 $(eval $(call assert_boolean,ERRATA_A53_826319))
 $(eval $(call add_define,ERRATA_A53_826319))
@@ -121,3 +125,7 @@ $(eval $(call add_define,ERRATA_A57_828024))
 # Process ERRATA_A57_829520 flag
 $(eval $(call assert_boolean,ERRATA_A57_829520))
 $(eval $(call add_define,ERRATA_A57_829520))
+
+# Process ERRATA_A57_833471 flag
+$(eval $(call assert_boolean,ERRATA_A57_833471))
+$(eval $(call add_define,ERRATA_A57_833471))