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

2021-02-15 Thread Krzysztof WilczyƄski
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


[PATCH v1 0/4] Introduce pcim_alloc_irq_vectors()

2021-02-15 Thread Dejin Zheng
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.

Dejin Zheng (4):
  PCI: Introduce pcim_alloc_irq_vectors()
  Documentation: devres: add pcim_alloc_irq_vectors()
  i2c: designware: Use the correct name of device-managed function
  i2c: thunderx: Use the correct name of device-managed function

 .../driver-api/driver-model/devres.rst|  1 +
 drivers/i2c/busses/i2c-designware-pcidrv.c|  2 +-
 drivers/i2c/busses/i2c-thunderx-pcidrv.c  |  2 +-
 drivers/pci/pci.c | 19 +++
 include/linux/pci.h   |  3 +++
 5 files changed, 25 insertions(+), 2 deletions(-)

-- 
2.25.0