Re: [PATCH v1 0/4] Introduce pcim_alloc_irq_vectors()

From: Krzysztof Wilczyński
Date: Mon Feb 15 2021 - 16:19:29 EST


Hi Dejin,

Thank you for working on this series!

Do you have a link to the conversation that prompted addition of this
new function? If so, then it would be nice to include a reference to it
here (as a link to http://lore.kernel.org/) in the cover letter for
reference, if possible, of course.

Generally, it would also be nice to expand on things a little bit and
explain why do you want to add this new function, and what problems does
it solve.

[...]
> Introduce pcim_alloc_irq_vectors(), a explicit device-managed version of
> pci_alloc_irq_vectors(). and use the correct name of device-managed
> function to alloc irq vectors in i2c drivers.

Did you want to use a comma, instead of a period, in the sentence
above? You could also probably drop the word "explicit".

Krzysztof