| Date | Wed, 16 Nov 2022 13:31:10 -0400 | From | Jason Gunthorpe <> | Subject | Re: [patch 31/39] Documentation: PCI: Add reference to PCI/MSI device driver APIs |
| |
On Fri, Nov 11, 2022 at 02:55:04PM +0100, Thomas Gleixner wrote: > From: Ahmed S. Darwish <darwi@linutronix.de> > > All exported device-driver MSI APIs are now grouped in one place at > drivers/pci/msi/api.c with comprehensive kernel-docs added. > > Reference these kernel-docs in the official PCI/MSI howto. > > Signed-off-by: Ahmed S. Darwish <darwi@linutronix.de> > Signed-off-by: Thomas Gleixner <tglx@linutronix.de> > > --- > Documentation/PCI/msi-howto.rst | 10 ++++++++++ > 1 file changed, 10 insertions(+)
I was wondering what the payoff was for moving everything to api.c, this seems pretty good.
Though, in some regards it might be cleaner to teach ":export:" about how to process symbol namespaces and put all the architecture facing exports in some PCI_MSI_ARCH/IRQCHIP namespace which could achieve the same effect for kdoc as moving all the code around and have the bonus of discouraging people from mis-using the APIs inside inappropriate drivers.
But, I like the idea, and the outcome is great so
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Jason
|