On 2022-09-16 08:03, Yang Yingliang wrote: > Use for_each_pci_dev() instead of open-coding it. > No functional change. > > Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx> Reviewed-by: Logan Gunthorpe <logang@xxxxxxxxxxxx> Thanks! Logan > --- > drivers/pci/p2pdma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c > index 4496a7c5c478..88dc66ee1c46 100644 > --- a/drivers/pci/p2pdma.c > +++ b/drivers/pci/p2pdma.c > @@ -649,7 +649,7 @@ struct pci_dev *pci_p2pmem_find_many(struct device **clients, int num_clients) > if (!closest_pdevs) > return NULL; > > - while ((pdev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, pdev))) { > + for_each_pci_dev(pdev) { > if (!pci_has_p2pmem(pdev)) > continue; >