On 12/03/18 09:28 PM, Sinan Kaya wrote:
Maybe, dev parameter should also be struct pci_dev so that you can get rid of
all to_pci_dev() calls in this code including find_parent_pci_dev() function.
No, this was mentioned in v2. find_parent_pci_dev is necessary because
the calling drivers aren't likely to have a bunch of struct pci_dev's
for all the devices they are working with lying around. It's a much
nicer from an API stand point to take struct devs and not worth it just
to have a PCI API only taking struct pci_devs.