On Wed, 2024-10-16 at 13:38 +0300, Andy Shevchenko wrote: > On Wed, Oct 16, 2024 at 11:49:05AM +0200, Philipp Stanner wrote: > > pcim_ionumap_region() has recently been made a public function and > > does > > not have the disadvantage of having to deal with the legacy iomap > > table, > > as pcim_iounmap_regions() does. > > > > Deprecate pcim_iounmap_regions(). > > ... > > > + * This function is DEPRECATED. Do not use it in new code. > > Interestingly that the syntax of the DEPRECATED is not documented > (yet?), > however the sphinx parser hints us about **DEPRECATED** format — see > Documentation/sphinx/parse-headers.pl:251. In any case the above > seems > like second used (in a form of the full sentence) and will be updated > in accordance with the above mentioned script. Can't completely follow – so one should always write "**DEPRECATED**", correct? Is that a blocker for you? All the docstrings in pci/pci.c and pci/devres.c so far just use "DEPRECATED". P.