On Tue, Aug 26, 2014 at 01:35:34PM +0200, Arnd Bergmann wrote: > On Tuesday 26 August 2014 12:20:13 Thierry Reding wrote: > > On Tue, Aug 26, 2014 at 11:54:43AM +0200, Arnd Bergmann wrote: > > > > I'm not sure if it's really worth it. One thing we might do is just > > > remove all definitions of ioremap_nocache and add a wrapper to > > > include/linux/io.h, to make it more obvious what is going on. > > > > Yes, I suppose that would work too. I still think there's an advantage > > in being explicit and avoid aliases like this. Perhaps a __deprecated > > annotation would help with that? > > I fear adding __deprecated would be too controversial, because that > would add hundreds of new warnings to code that is not actually wrong. Right. __deprecated is enabled via Kconfig, though, so people could turn that off if they don't want to see the warnings. I don't mind very much either way. Thierry
Attachment:
pgpo5O2Lvv22n.pgp
Description: PGP signature
