diff mbox

[U-Boot] mx6: drop ARM errata 742230

Message ID 1402476767-5260-1-git-send-email-shawn.guo@freescale.com
State Awaiting Upstream
Delegated to: Stefano Babic
Headers show

Commit Message

Shawn Guo June 11, 2014, 8:52 a.m. UTC
Commit e9fd66defd7e (ARM: mx6: define CONFIG_ARM_ERRATA_742230) enables
errata 742230 for imx6, because it helps remove one reboot issue.
However, this errata does not really apply on imx6, because Cortex-A9
on imx6 is r2p10 while the errata only applies to revisions r1p0..r2p2.

At a later time, commit f71cbfe3ca5d (ARM: Add workaround for Cortex-A9
errata 794072) adds support of errata 794072, which applies to all
Cortex-A9 revisions.  As the workaround for both errata are exactly
same, it makes a lot more sense to select 794072 instead of 742230 for
imx6.  Since we already enable 794072 for imx6, it's time to drop
errata 742230 to avoid confusion.

Signed-off-by: Shawn Guo <shawn.guo@freescale.com>
---
 include/configs/mx6_common.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Stefano Babic June 17, 2014, 2:34 p.m. UTC | #1
Hi Shawn,

On 11/06/2014 10:52, Shawn Guo wrote:
> Commit e9fd66defd7e (ARM: mx6: define CONFIG_ARM_ERRATA_742230) enables
> errata 742230 for imx6, because it helps remove one reboot issue.
> However, this errata does not really apply on imx6, because Cortex-A9
> on imx6 is r2p10 while the errata only applies to revisions r1p0..r2p2.
> 
> At a later time, commit f71cbfe3ca5d (ARM: Add workaround for Cortex-A9
> errata 794072) adds support of errata 794072, which applies to all
> Cortex-A9 revisions.  As the workaround for both errata are exactly
> same, it makes a lot more sense to select 794072 instead of 742230 for
> imx6.  Since we already enable 794072 for imx6, it's time to drop
> errata 742230 to avoid confusion.
> 
> Signed-off-by: Shawn Guo <shawn.guo@freescale.com>
> ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic
diff mbox

Patch

diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
index 8a8920f6cc8a..e4a5cc5be1a6 100644
--- a/include/configs/mx6_common.h
+++ b/include/configs/mx6_common.h
@@ -17,7 +17,6 @@ 
 #ifndef __MX6_COMMON_H
 #define __MX6_COMMON_H
 
-#define CONFIG_ARM_ERRATA_742230
 #define CONFIG_ARM_ERRATA_743622
 #define CONFIG_ARM_ERRATA_751472
 #define CONFIG_ARM_ERRATA_794072