[U-Boot] [PATCH] ARM: cpu: Add ARMv7 barrier operations support

Nobuhiro Iwamatsu nobuhiro.iwamatsu.yj at renesas.com
Wed Mar 25 08:47:30 CET 2015


Hi,

2015-03-21 0:16 GMT+09:00 Vladimir Barinov
<vladimir.barinov at cogentembedded.com>:
> From: Valentine Barshak <valentine.barshak at cogentembedded.com>
>
> This enables ARMv7 barrier operations support when
> march=armv7-a is enabled.
>
> Using CP15 barriers causes U-Boot bootm command crash when
> transferring control to the loaded image on Renesas R8A7794 Cortex A7 CPU.
> Using ARMv7 barrier operations instead of the deprecated CP15 barriers
> helps to avoid these issues.
>
> Signed-off-by: Valentine Barshak <valentine.barshak+renesas at cogentembedded.com>
> Signed-off-by: Vladimir Barinov <vladimir.barinov+renesas at cogentembedded.com>
> Reviewed-by: Tom Rini <trini at konsulko.com>

Reviewed-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj at renesas.com>

Best regards,
  Nobuhiro

> ---
>  arch/arm/cpu/armv7/cache_v7.c | 14 +++++++-------
>  arch/arm/include/asm/armv7.h  | 10 ++++++++++
>  2 files changed, 17 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/cache_v7.c b/arch/arm/cpu/armv7/cache_v7.c
> index 0f9d837..e8ee875 100644
> --- a/arch/arm/cpu/armv7/cache_v7.c
> +++ b/arch/arm/cpu/armv7/cache_v7.c
> @@ -68,7 +68,7 @@ static void v7_inval_dcache_level_setway(u32 level, u32 num_sets,
>                 }
>         }
>         /* DSB to make sure the operation is complete */
> -       CP15DSB;
> +       DSB;
>  }
>
>  static void v7_clean_inval_dcache_level_setway(u32 level, u32 num_sets,
> @@ -96,7 +96,7 @@ static void v7_clean_inval_dcache_level_setway(u32 level, u32 num_sets,
>                 }
>         }
>         /* DSB to make sure the operation is complete */
> -       CP15DSB;
> +       DSB;
>  }
>
>  static void v7_maint_dcache_level_setway(u32 level, u32 operation)
> @@ -215,7 +215,7 @@ static void v7_dcache_maint_range(u32 start, u32 stop, u32 range_op)
>         }
>
>         /* DSB to make sure the operation is complete */
> -       CP15DSB;
> +       DSB;
>  }
>
>  /* Invalidate TLB */
> @@ -228,9 +228,9 @@ static void v7_inval_tlb(void)
>         /* Invalidate entire instruction TLB */
>         asm volatile ("mcr p15, 0, %0, c8, c5, 0" : : "r" (0));
>         /* Full system DSB - make sure that the invalidation is complete */
> -       CP15DSB;
> +       DSB;
>         /* Full system ISB - make sure the instruction stream sees it */
> -       CP15ISB;
> +       ISB;
>  }
>
>  void invalidate_dcache_all(void)
> @@ -343,10 +343,10 @@ void invalidate_icache_all(void)
>         asm volatile ("mcr p15, 0, %0, c7, c5, 6" : : "r" (0));
>
>         /* Full system DSB - make sure that the invalidation is complete */
> -       CP15DSB;
> +       DSB;
>
>         /* ISB - make sure the instruction stream sees it */
> -       CP15ISB;
> +       ISB;
>  }
>  #else
>  void invalidate_icache_all(void)
> diff --git a/arch/arm/include/asm/armv7.h b/arch/arm/include/asm/armv7.h
> index a13da23..189f3f0 100644
> --- a/arch/arm/include/asm/armv7.h
> +++ b/arch/arm/include/asm/armv7.h
> @@ -70,6 +70,16 @@
>  #define CP15DSB        asm volatile ("mcr     p15, 0, %0, c7, c10, 4" : : "r" (0))
>  #define CP15DMB        asm volatile ("mcr     p15, 0, %0, c7, c10, 5" : : "r" (0))
>
> +#ifdef __ARM_ARCH_7A__
> +#define ISB    asm volatile ("isb" : : : "memory")
> +#define DSB    asm volatile ("dsb" : : : "memory")
> +#define DMB    asm volatile ("dmb" : : : "memory")
> +#else
> +#define ISB    CP15ISB
> +#define DSB    CP15DSB
> +#define DMB    CP15DMB
> +#endif
> +
>  /*
>   * Workaround for ARM errata # 798870
>   * Set L2ACTLR[7] to reissue any memory transaction in the L2 that has been
> --
> 1.9.3
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-- 
Nobuhiro Iwamatsu


More information about the U-Boot mailing list