intel-iommu: mark intel_dma_ops static
authorChristoph Hellwig <hch@lst.de>
Mon, 17 Sep 2018 17:10:31 +0000 (19:10 +0200)
committerTony Luck <tony.luck@intel.com>
Mon, 17 Sep 2018 17:15:58 +0000 (10:15 -0700)
ia64 currently explicitly assigns it to dma_ops, but that same work is
already done by intel_iommu_init a little later, so we can remove the
duplicate assignment and mark the variable static.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/ia64/kernel/pci-dma.c
drivers/iommu/intel-iommu.c

index 5a5bf5a82ac25cb1d16edb4fc92d9e5bae74d122..fe988c49f01ce6ae844355731ddc50f01fef3df1 100644 (file)
@@ -24,8 +24,6 @@ int force_iommu __read_mostly;
 
 int iommu_pass_through;
 
-extern struct dma_map_ops intel_dma_ops;
-
 static int __init pci_iommu_init(void)
 {
        if (iommu_detected)
@@ -39,8 +37,6 @@ fs_initcall(pci_iommu_init);
 
 void __init pci_iommu_alloc(void)
 {
-       dma_ops = &intel_dma_ops;
-
        /*
         * The order of these functions is important for
         * fall-back/fail-over reasons
index e72efef979245944cc88d3b49ad5fbbc96fde328..0ee6516de41a72d92ce146ca867ac6e30194e517 100644 (file)
@@ -3895,7 +3895,7 @@ static int intel_mapping_error(struct device *dev, dma_addr_t dma_addr)
        return !dma_addr;
 }
 
-const struct dma_map_ops intel_dma_ops = {
+static const struct dma_map_ops intel_dma_ops = {
        .alloc = intel_alloc_coherent,
        .free = intel_free_coherent,
        .map_sg = intel_map_sg,