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