On Tue, Jul 18, 2023 at 2:55 PM Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote: > Le mardi 18 juillet 2023 à 13:41 +0200, Thierry Reding a écrit : > > On Tue, Jul 18, 2023 at 10:42:47AM +0200, Paul Cercueil wrote: ... > > Looking at the definition of EXPORT_GPL_DEV_PM_OPS(), it will cause > > an > > EXPORT_SYMBOL_GPL() to be added. So there very well is a change. And > > it's a completely bogus change because no module is ever going to use > > that symbol. If we were to ever support building the pinctrl driver > > as > > a module, then this would perhaps make sense, but we don't. > > In this particular case the EXPORT_SYMBOL_GPL() isn't really important, > the rest of EXPORT_GPL_DEV_PM_OPS() is. > > I don't think having a symbol exported it is a big deal, TBH, if you > use the namespaced version. If you really don't want that, we need a > version of EXPORT_GPL_DEV_PM_OPS() that doesn't export the symbol. Ah, I agree with Thierry and it is another point why I do not like those EXPORT*PM_OPS() macros. Polluting an exported space (even namespaced) is a big deal, so, definitely no from my side. -- With Best Regards, Andy Shevchenko
