[U-Boot] [PATCH] mx6: drop ARM errata 742230

[U-Boot] [PATCH] mx6: drop ARM errata 742230

[Shawn Guo]

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(-)

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
-- 
1.8.3.2

[U-Boot] [PATCH] mx6: drop ARM errata 742230

[Nitin Garg]

Acked-by: Nitin Garg <nitin.garg@freescale.com>

Regards,
Nitin Garg

-----Original Message-----
From: Shawn Guo [mailto:shawn.guo at freescale.com] 
Sent: Wednesday, June 11, 2014 3:53 AM
To: u-boot at lists.denx.de
Cc: Garg Nitin-B37173; Stefano Babic; Guo Shawn-R65073
Subject: [PATCH] mx6: drop ARM errata 742230

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(-)

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
--
1.8.3.2

[U-Boot] [PATCH] mx6: drop ARM errata 742230

[Stefano Babic]

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

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================