[U-Boot] [PATCH] ARM: mx6: define CONFIG_ARM_ERRATA_742230

[U-Boot] [PATCH] ARM: mx6: define CONFIG_ARM_ERRATA_742230

[Shawn Guo]

The ARM errata 742230 - "ARM errata: DMB operation may be faulty" is
claimed for Cortex-A9 (r1p0..r2p2).  Though i.MX6 uses a newer revision
than r2p2, we are seeing a reboot failure on i.MX6 SMP build that can be
fixed by applying the workaround for this errata.  So for safety, let's
define CONFIG_ARM_ERRATA_742230 to enable the workaround on i.MX6.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
---
 include/configs/mx6_common.h |    1 +
 1 file changed, 1 insertion(+)

diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
index b333937..674bcd3 100644
--- a/include/configs/mx6_common.h
+++ b/include/configs/mx6_common.h
@@ -17,6 +17,7 @@
 #ifndef __MX6_COMMON_H
 #define __MX6_COMMON_H
 
+#define CONFIG_ARM_ERRATA_742230
 #define CONFIG_ARM_ERRATA_743622
 #define CONFIG_ARM_ERRATA_751472
 
-- 
1.7.9.5

[U-Boot] [PATCH] ARM: mx6: define CONFIG_ARM_ERRATA_742230

[Stefano Babic]

On 16/04/2013 16:58, Shawn Guo wrote:
> The ARM errata 742230 - "ARM errata: DMB operation may be faulty" is
> claimed for Cortex-A9 (r1p0..r2p2).  Though i.MX6 uses a newer revision
> than r2p2, we are seeing a reboot failure on i.MX6 SMP build that can be
> fixed by applying the workaround for this errata.  So for safety, let's
> define CONFIG_ARM_ERRATA_742230 to enable the workaround on i.MX6.
> 
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>

Hi Shawn,

> ---
>  include/configs/mx6_common.h |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/configs/mx6_common.h b/include/configs/mx6_common.h
> index b333937..674bcd3 100644
> --- a/include/configs/mx6_common.h
> +++ b/include/configs/mx6_common.h
> @@ -17,6 +17,7 @@
>  #ifndef __MX6_COMMON_H
>  #define __MX6_COMMON_H
>  
> +#define CONFIG_ARM_ERRATA_742230
>  #define CONFIG_ARM_ERRATA_743622
>  #define CONFIG_ARM_ERRATA_751472
>  

This is a fix, but the release is coming (end of the week). What do you
think if I push this to -next ?

Best regards,
Stefano


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

[U-Boot] [PATCH] ARM: mx6: define CONFIG_ARM_ERRATA_742230

[Shawn Guo]

On Tue, Apr 16, 2013 at 05:05:30PM +0200, Stefano Babic wrote:
> This is a fix, but the release is coming (end of the week). What do you
> think if I push this to -next ?

No pressure on this, so -next is fine.  Thanks, Stefano.

Shawn

[U-Boot] [PATCH] ARM: mx6: define CONFIG_ARM_ERRATA_742230

[Stefano Babic]

On 16/04/2013 16:58, Shawn Guo wrote:
> The ARM errata 742230 - "ARM errata: DMB operation may be faulty" is
> claimed for Cortex-A9 (r1p0..r2p2).  Though i.MX6 uses a newer revision
> than r2p2, we are seeing a reboot failure on i.MX6 SMP build that can be
> fixed by applying the workaround for this errata.  So for safety, let's
> define CONFIG_ARM_ERRATA_742230 to enable the workaround on i.MX6.
> 
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> ---

Applied to u-boot-imx, -next branch, 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
=====================================================================