| Message ID | 20191009225354.181018-1-helgaas@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | PCI/ATS: Clean up unnecessary stubs and exports | expand |
Hi Bjorn, On Wed, Oct 09, 2019 at 05:53:51PM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > > Most of the ATS/PRI/PASID interfaces are only used by IOMMU drivers that > can only be built statically, not as modules. A couple are only used by > the PCI core and don't need to be visible outside at all. > > These are intended to be cleanup only, but let me know if they would break > something. > > Bjorn Helgaas (3): > PCI/ATS: Remove unused PRI and PASID stubs > PCI/ATS: Remove unnecessary EXPORT_SYMBOL_GPL() > PCI/ATS: Make pci_restore_pri_state(), pci_restore_pasid_state() > private The series looks good to me, for the whole series: Reviewed-by: Joerg Roedel <jroedel@suse.de>
On Wed, Oct 09, 2019 at 05:53:51PM -0500, Bjorn Helgaas wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > > Most of the ATS/PRI/PASID interfaces are only used by IOMMU drivers that > can only be built statically, not as modules. A couple are only used by > the PCI core and don't need to be visible outside at all. > > These are intended to be cleanup only, but let me know if they would break > something. > > Bjorn Helgaas (3): > PCI/ATS: Remove unused PRI and PASID stubs > PCI/ATS: Remove unnecessary EXPORT_SYMBOL_GPL() > PCI/ATS: Make pci_restore_pri_state(), pci_restore_pasid_state() > private > > drivers/pci/ats.c | 14 -------------- > drivers/pci/pci.h | 4 ++++ > include/linux/pci-ats.h | 15 --------------- > 3 files changed, 4 insertions(+), 29 deletions(-) I applied these to pci/virtualization for v5.5, with Kuppuswamy's Reviewed-by on the first and Joerg's on all three. Thank you both for taking a look! Bjorn
From: Bjorn Helgaas <bhelgaas@google.com> Most of the ATS/PRI/PASID interfaces are only used by IOMMU drivers that can only be built statically, not as modules. A couple are only used by the PCI core and don't need to be visible outside at all. These are intended to be cleanup only, but let me know if they would break something. Bjorn Helgaas (3): PCI/ATS: Remove unused PRI and PASID stubs PCI/ATS: Remove unnecessary EXPORT_SYMBOL_GPL() PCI/ATS: Make pci_restore_pri_state(), pci_restore_pasid_state() private drivers/pci/ats.c | 14 -------------- drivers/pci/pci.h | 4 ++++ include/linux/pci-ats.h | 15 --------------- 3 files changed, 4 insertions(+), 29 deletions(-)