| Message ID | 20220916140329.679633-1-yangyingliang@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | [-next] PCI/P2PDMA: Switch to use for_each_pci_dev() helper | expand |
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@huawei.com> Reviewed-by: Logan Gunthorpe <logang@deltatee.com> 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; >
On Fri, Sep 16, 2022 at 10:03:29PM +0800, Yang Yingliang wrote: > Use for_each_pci_dev() instead of open-coding it. > No functional change. > > Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> Applied with Logan's Reviewed-by to pci/misc for v6.1, thanks! > --- > 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; > > -- > 2.25.1 >
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;
Use for_each_pci_dev() instead of open-coding it. No functional change. Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> --- drivers/pci/p2pdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)