On Wed, Aug 15, 2018 at 03:26:39PM -0600, Jon Derrick wrote: > --- a/drivers/pci/pci.h > +++ b/drivers/pci/pci.h > @@ -536,4 +536,10 @@ static inline void pci_aer_clear_fatal_status(struct pci_dev *dev) { } > static inline void pci_aer_clear_device_status(struct pci_dev *dev) { } > #endif > > +#ifdef CONFIG_PCIE_DPC > +void pci_no_dpc(void); > +#else > +static inline void pci_no_dpc(void) { } > +#endif <snip> > --- a/include/linux/pci.h > +++ b/include/linux/pci.h > @@ -1473,6 +1473,12 @@ static inline int pci_irqd_intx_xlate(struct irq_domain *d, > static inline bool pci_aer_available(void) { return false; } > #endif > > +#ifdef CONFIG_PCIE_DPC > +bool pci_dpc_available(void); > +#else > +static inline bool pci_dpc_available(void) { return false; } > +#endif Seems like these two sections belong together. Otherwise, looks fine.