Re: [PATCH] ioremap_cached()

Arjan van de Ven
2006-03-31T11:20:49Z

>
> It seems clear to me that ioremap() and ioremap_nocache() are the
> wrong way around. The default needs to be uncached -- and the obvious
> (rare) alternative becomes ioremap_cached().
>
> So here's a patch for i386 to add ioremap_cached() and make ioremap()
> uncached. ioremap_nocache() remains as an alias for ioremap() so we
> don't needlessly break old drivers. Architecture maintainers will need
> to fix up their ports if this patch is accepted.

I'd actually suggest deprecating ioremap() and moving away from it to
make sure all users think of which variant they want. Explicit naming is
always better than "some" unknown behavior (especially for old drivers
that work on multiple kernels... while you allow them to keep compiling,
you change the world under them so I think eventually break them
compiling is actually better than having them limp along broken)