Woodruff, Richard wrote:
From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces@lists.denx.de] On Behalf Of Tom Rix Sent: Monday, July 06, 2009 11:48 AM
Since there is only one version of flushing the dcache for arm_cortex8, rename v7_flush_dcache_all to the the generic name flush_dcache. Because the function is intended for only omap3 boards, move the function to the new file cache_flush.S.
Minor point is file name or function name might change to be more reflective of what it does (cache_ops.S, and invalidate_dcache()). As there is not really any 'flush' (cleaning out dirty entries to main memory).
I will change the name to cache_ops.S flush_dcache was chosen to match other boards/cpu's and one of the main reasons for this change. If Jean is ok changing flush_dcache to invalidate_dcache, I will make that change too. Tom
Today the dcache is not even enabled in ARM ports. To do so requires MMU to be enabled to properly map IO regions as non-cached.
As such all we ever need to do is possibly invalidate icache at run time (say you use Kermit to download something and try to execute it).
The dcache invalidate today is only there to setup cache in clean state before calling the kernel.
Later on a dcache-flush/icache-invalidate might need to be added if Dcache is actually enabled.
Regards, Richard W.