Woodruff, Richard wrote: >> From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@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. > >
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
