[U-Boot] [PATCH 3/3] mpc83xx: turn on icache in core initialization to improve u-boot boot time

Scott Wood scottwood at freescale.com
Tue Apr 20 00:35:52 CEST 2010


Kim Phillips wrote:
> On Mon, 19 Apr 2010 15:19:24 -0500
> Scott Wood <scottwood at 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


More information about the U-Boot mailing list