Kim Phillips wrote:
> On Mon, 19 Apr 2010 15:19:24 -0500
> Scott Wood <scottw...@freescale.com> wrote:
>> Shouldn't you be using icache_enable(), or at least using HID0_INIT to 
>> do invalidation and lock clearing?
> 
> the invalidation should occur whether or not the cache enable bit is
> set in HID0_INIT,

OK, was thinking of some other caches that don't automatically clear out 
the random junk on power-on -- but e300 manual says it does.

> But I see your point; we should be using a more
> formal approach.  I'll see what I can do - it's just that this patch
> preserved the existing code size, which could be important for e.g.,
> future nand bootstrap development.

Right.  It looks like we could shrink the NAND SPL some more by 
#ifndefing the cache functions in start.S.

-Scott
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to