[U-Boot] [RFC PATCH 1/2] ARM: cache: Introduce weak dram_bank_setup function

Stefan Roese sr at denx.de
Tue Jan 8 12:52:42 CET 2013


On 01/08/2013 12:18 PM, R Sricharan wrote:
> Introduce a weak version of dram_bank_setup function
> to allow a platform specific redefinition.
> 
> This is used in the subsequent patch to setup dram region
> without 'XN' attribute in order to enable the region
> under client permissions.
> 
> Signed-off-by: R Sricharan <r.sricharan at ti.com>
> Cc: Vincent Stehle <v-stehle at ti.com>
> Cc: Tom Rini <trini at ti.com>
> ---
>  arch/arm/include/asm/cache.h |    1 +
>  arch/arm/lib/cache-cp15.c    |    7 ++++++-
>  2 files changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/include/asm/cache.h b/arch/arm/include/asm/cache.h
> index eef6a5a..93811d2 100644
> --- a/arch/arm/include/asm/cache.h
> +++ b/arch/arm/include/asm/cache.h
> @@ -42,6 +42,7 @@ static inline void invalidate_l2_cache(void)
>  void l2_cache_enable(void);
>  void l2_cache_disable(void);
>  
> +void dram_bank_mmu_setup(int bank);
>  /*
>   * The current upper bound for ARM L1 data cache line sizes is 64 bytes.  We
>   * use that value for aligning DMA buffers unless the board config has specified
> diff --git a/arch/arm/lib/cache-cp15.c b/arch/arm/lib/cache-cp15.c
> index 6edf815..843078e 100644
> --- a/arch/arm/lib/cache-cp15.c
> +++ b/arch/arm/lib/cache-cp15.c
> @@ -23,6 +23,8 @@
>  
>  #include <common.h>
>  #include <asm/system.h>
> +#include <asm/cache.h>
> +#include <linux/compiler.h>
>  
>  #if !(defined(CONFIG_SYS_ICACHE_OFF) && defined(CONFIG_SYS_DCACHE_OFF))
>  
> @@ -77,7 +79,7 @@ void mmu_set_region_dcache_behaviour(u32 start, int size,
>  	mmu_page_table_flush((u32)&page_table[start], (u32)&page_table[end]);
>  }
>  
> -static inline void dram_bank_mmu_setup(int bank)
> +void __dram_bank_mmu_setup(int bank)
>  {
>  	bd_t *bd = gd->bd;
>  	int	i;
> @@ -94,6 +96,9 @@ static inline void dram_bank_mmu_setup(int bank)
>  	}
>  }
>  
> +void dram_bank_mmu_setup(int bank)
> +	__attribute__((weak, alias("__dram_bank_mmu_setup")));

Please use this instead (you already included linux/compiler.h for this):

__weak void dram_bank_mmu_setup(int bank)
{
...


Thanks,
Stefan



More information about the U-Boot mailing list