Hi Alok, On 2/10/2023 12:45 PM, ALOK TIWARI wrote: > shall we need to define this function under -> #ifndef CONFIG_PCI_MSI > > #ifndef CONFIG_PCI_MSI > > +static inline struct msi_map > +pci_msix_alloc_irq_at(struct pci_dev *dev, unsigned int index, > + const struct irq_affinity_desc *affdesc) > +{ > + struct msi_map map = { .index = -ENOSYS }; > + > + return map; > +} > + > +static inline void pci_msix_free_irq(struct pci_dev *pdev, struct msi_map map) > +{ > +} > +#endif No need. include/linux/pci.h already has those definitions. include/linux/pci.h already has: #ifdef CONFIG_PCI_MSI ... #else ... /* new function definitions will be inserted here */ ... #endif Reinette