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

[Hadli, Manjunath]

Laurence,
On Sat, Oct 08, 2011 at 18:26:39, Laurence Withers wrote:
> On Sat, Oct 08, 2011 at 02:58:36PM +0530, manjunath.hadli at ti.com wrote:
> > revert commit bd65d006a6088bcb857e079447d7549e2cd7054d as the 
> > disabling of cache need not be done explicitly. Subsequent patches to 
> > new cache management framework has fixed it.
> > EMAC issue with cache coherency still exists when cahces are enabled.
> 
> Hi Manjunath,
> 
> A small thing, but I'd recommend you drop the sentence "Subsequent patches to new cache management framework has fixed it.", as it implies things were broken (true) and are now fixed (still not true).

> 
> In fact, would it be better to simply drop this patch from the patch series altogether, leaving the caches explicitly disabled, until such a time as the EMAC driver is fixed (I guess we are mainly waiting to see if anyone wants to tackle cache ops for the ARM926EJS) and the code has been verified with caches enabled?
Sure. I am dropping this patch for now. We will tackle (or may be Yonuk) it 
Again soon.

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