[U-Boot] [PATCH 4/6] da850: revert cache disable patch

[Laurence Withers]

On Fri, Sep 30, 2011 at 04:54:25PM +0530, nagabhushana.netagunte at ti.com wrote:
> From: Nagabhushana Netagunte <nagabhushana.netagunte at ti.com>
> 
> revert commit bd65d006a6088bcb857e079447d7549e2cd7054d as cache
> disabling is no more needed. Subsequent patches to new cache
> management framework has fixed EMAC issue with cache coherency.
> 
> Signed-off-by: Nagabhushana Netagunte <nagabhushana.netagunte at ti.com>
> ---
>  include/configs/da850_l138evm.h |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/include/configs/da850_l138evm.h b/include/configs/da850_l138evm.h
> index 347893c..0bd630f 100644
> --- a/include/configs/da850_l138evm.h
> +++ b/include/configs/da850_l138evm.h
> @@ -43,9 +43,6 @@
>  #define CONFIG_SYS_HZ			1000
>  #define CONFIG_SKIP_LOWLEVEL_INIT
>  #define CONFIG_SYS_TEXT_BASE		0xc1080000
> -#define CONFIG_SYS_ICACHE_OFF
> -#define CONFIG_SYS_DCACHE_OFF
> -#define CONFIG_SYS_L2CACHE_OFF
>  
>  /*
>   * Memory Info

Am I correct in thinking that what you mean is that caches are now disabled
by default, and therefore you don't need to explicitly disable them? I guess
so, since I couldn't find any posted patches to fix the EMAC driver (though I
would be very glad if I was wrong about this!).

If so, we haven't actually "fixed EMAC issue with cache coherency" - the
driver is still broken if caches are enabled, and the commit message should
reflect that. The same would apply to the dm644X and dm36x cache disable
patches.

Bye for now,
-- 
Laurence Withers, <lwithers at guralp.com>                http://www.guralp.com/
Direct tel:+447753988197 or tel:+443333408643               Software Engineer
General support queries: <support at guralp.com>         CMG-DCM CMG-EAM CMG-NAM