On Sun, May 21, 2017 at 03:00:46PM +0000, Gabriele Paoloni wrote: > > I thought that in your previous review you meant to say that we should > avoid 2 instances of the same comment (in this patchset I only have this > instance as I have removed the same comment from patch 2/2...) No, there is no pointin the comment. The whole point of pci_irq_vector is to abstract away from how to get at the interrupt in detail. There is no need to comment on this at the callsite. If you think the existing documentation for the function is not good enough send a patch to improve it there instead of commenting the callsites.