Re: [PATCH] 8xx: add cpm_get_cpmp()

On Sun, Aug 07, 2005 at 12:57:31PM -0300, Marcelo Tosatti wrote:

On Sun, Aug 07, 2005 at 12:44:32PM -0300, Aristeu Sergio Rozanski Filho wrote:

quoted

quoted

It depends how you define the semantics of this function call.
Can you call it any (and all of the) time you need it, or does it
actually perform an ioremap() and you only want to call it
once?

what about don't cache it and call ioremap() from driver? (I guess
ioremap() already check if an area is already mapped, no?)

Yep, ioremap() should be doing virtual address caching already, no?

In general, ioremap() doesn't do any caching currently. Some subarchs 
do some limited caching, e.g. if BATs (classic PPC) or CAMs (e500) are 
available and were used previously for ioremap() mappings. 

Some time ago I made a trivial ioremap cache patch (useful on 4xx, 
which doesn't have BATs nor CAMs), although it wass really a hack and 
it was never merged :).

-- 
Eugene