Hi, [...] > +/* > + * pcie_port_service_get_irq - get irq of the service > + * @dev: PCI Express port the service is associated with > + * @service: For the service to find > + * > + * Get irq number associated with given service on a pci_dev > + */ > +int pcie_port_service_get_irq(struct pci_dev *dev, u32 service) [...] A small nitpick. It would be "IRQ" rather than "irq" in the above kernel-doc. Also, missing periods at the end of sentence. Krzysztof