On Thu, Sep 23, 2021 at 10:27 AM Ben Widawsky <[email protected]> wrote: > > Reduce maintenance burden of DVSEC query implementation by using the > centralized PCI core implementation. > > Cc: [email protected] > Cc: David Woodhouse <[email protected]> > Cc: Lu Baolu <[email protected]> > Signed-off-by: Ben Widawsky <[email protected]> > --- > drivers/iommu/intel/iommu.c | 15 +-------------- > 1 file changed, 1 insertion(+), 14 deletions(-) > > diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c > index d75f59ae28e6..30c97181f0ae 100644 > --- a/drivers/iommu/intel/iommu.c > +++ b/drivers/iommu/intel/iommu.c > @@ -5398,20 +5398,7 @@ static int intel_iommu_disable_sva(struct device *dev) > */ > static int siov_find_pci_dvsec(struct pci_dev *pdev) > { > - int pos; > - u16 vendor, id; > - > - pos = pci_find_next_ext_capability(pdev, 0, 0x23); > - while (pos) { > - pci_read_config_word(pdev, pos + 4, &vendor); > - pci_read_config_word(pdev, pos + 8, &id); > - if (vendor == PCI_VENDOR_ID_INTEL && id == 5) > - return pos; > - > - pos = pci_find_next_ext_capability(pdev, pos, 0x23); > - } > - > - return 0; > + return pci_find_dvsec_capability(pdev, PCI_VENDOR_ID_INTEL, 5); > }
Same comments as the CXL patch, siov_find_pci_dvsec() doesn't seem to have a reason to exist anymore. What is 5? _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
