diff mbox

[U-Boot,V6,03/11] ARM: Introduce erratum workaround for 430973

Message ID 1425939129-308-4-git-send-email-nm@ti.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Nishanth Menon March 9, 2015, 10:12 p.m. UTC
430973: Stale prediction on replaced inter working branch causes
	Cortex-A8 to execute in the wrong ARM/Thumb state
Impacts: Every Cortex-A8 processors with revision lower than r2p1
Work around: Set IBE to 1

Based on ARM errata Document revision 20.0 (13 Nov 2010)

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 README                     |    1 +
 arch/arm/cpu/armv7/start.S |   13 +++++++++++++
 2 files changed, 14 insertions(+)

Comments

Tom Rini March 11, 2015, 3:48 p.m. UTC | #1
On Mon, Mar 09, 2015 at 05:12:01PM -0500, Nishanth Menon wrote:

> 430973: Stale prediction on replaced inter working branch causes
> 	Cortex-A8 to execute in the wrong ARM/Thumb state
> Impacts: Every Cortex-A8 processors with revision lower than r2p1
> Work around: Set IBE to 1
> 
> Based on ARM errata Document revision 20.0 (13 Nov 2010)
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>
Tom Rini March 15, 2015, 9:50 p.m. UTC | #2
On Mon, Mar 09, 2015 at 05:12:01PM -0500, Nishanth Menon wrote:

> 430973: Stale prediction on replaced inter working branch causes
> 	Cortex-A8 to execute in the wrong ARM/Thumb state
> Impacts: Every Cortex-A8 processors with revision lower than r2p1
> Work around: Set IBE to 1
> 
> Based on ARM errata Document revision 20.0 (13 Nov 2010)
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>

Applied to u-boot/master, thanks!
diff mbox

Patch

diff --git a/README b/README
index 259dbdbd2075..82f7e10c71d6 100644
--- a/README
+++ b/README
@@ -693,6 +693,7 @@  The following options need to be configured:
 		NOTE: The following can be machine specific errata. These
 		do have ability to provide rudimentary version and machine
 		specific checks, but expect no product checks.
+		CONFIG_ARM_ERRATA_430973
 		CONFIG_ARM_ERRATA_454179
 		CONFIG_ARM_ERRATA_798870
 
diff --git a/arch/arm/cpu/armv7/start.S b/arch/arm/cpu/armv7/start.S
index 8483687879ed..41fb24cf321b 100644
--- a/arch/arm/cpu/armv7/start.S
+++ b/arch/arm/cpu/armv7/start.S
@@ -202,6 +202,19 @@  skip_errata_798870:
 skip_errata_454179:
 #endif
 
+#ifdef CONFIG_ARM_ERRATA_430973
+	cmp	r2, #0x21		@ Only on < r2p1
+	bge	skip_errata_430973
+
+	mrc	p15, 0, r0, c1, c0, 1	@ Read ACR
+	orr	r0, r0, #(0x1 << 6)	@ Set IBE bit
+	push	{r1-r5}			@ Save the cpu info registers
+	bl	v7_arch_cp15_set_acr
+	pop	{r1-r5}			@ Restore the cpu info - fall through
+
+skip_errata_430973:
+#endif
+
 	mov	pc, r5			@ back to my caller
 ENDPROC(cpu_init_cp15)