Commit 87a64d52 authored by Joerg Roedel's avatar Joerg Roedel

x86/amd-iommu: Remove iommu parameter from dma_ops_domain_alloc

This function doesn't use the parameter anymore so it can be
removed.
Signed-off-by: default avatarJoerg Roedel <joerg.roedel@amd.com>
parent 98fc5a69
...@@ -1188,7 +1188,7 @@ static void dma_ops_domain_free(struct dma_ops_domain *dom) ...@@ -1188,7 +1188,7 @@ static void dma_ops_domain_free(struct dma_ops_domain *dom)
* It also intializes the page table and the address allocator data * It also intializes the page table and the address allocator data
* structures required for the dma_ops interface * structures required for the dma_ops interface
*/ */
static struct dma_ops_domain *dma_ops_domain_alloc(struct amd_iommu *iommu) static struct dma_ops_domain *dma_ops_domain_alloc(void)
{ {
struct dma_ops_domain *dma_dom; struct dma_ops_domain *dma_dom;
...@@ -1443,7 +1443,7 @@ static int device_change_notifier(struct notifier_block *nb, ...@@ -1443,7 +1443,7 @@ static int device_change_notifier(struct notifier_block *nb,
dma_domain = find_protection_domain(devid); dma_domain = find_protection_domain(devid);
if (dma_domain) if (dma_domain)
goto out; goto out;
dma_domain = dma_ops_domain_alloc(iommu); dma_domain = dma_ops_domain_alloc();
if (!dma_domain) if (!dma_domain)
goto out; goto out;
dma_domain->target_dev = devid; dma_domain->target_dev = devid;
...@@ -2085,7 +2085,6 @@ static void prealloc_protection_domains(void) ...@@ -2085,7 +2085,6 @@ static void prealloc_protection_domains(void)
{ {
struct pci_dev *dev = NULL; struct pci_dev *dev = NULL;
struct dma_ops_domain *dma_dom; struct dma_ops_domain *dma_dom;
struct amd_iommu *iommu;
u16 devid; u16 devid;
while ((dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, dev)) != NULL) { while ((dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, dev)) != NULL) {
...@@ -2100,10 +2099,7 @@ static void prealloc_protection_domains(void) ...@@ -2100,10 +2099,7 @@ static void prealloc_protection_domains(void)
devid = get_device_id(&dev->dev); devid = get_device_id(&dev->dev);
iommu = amd_iommu_rlookup_table[devid]; dma_dom = dma_ops_domain_alloc();
if (!iommu)
continue;
dma_dom = dma_ops_domain_alloc(iommu);
if (!dma_dom) if (!dma_dom)
continue; continue;
init_unity_mappings_for_device(dma_dom, devid); init_unity_mappings_for_device(dma_dom, devid);
...@@ -2139,7 +2135,7 @@ int __init amd_iommu_init_dma_ops(void) ...@@ -2139,7 +2135,7 @@ int __init amd_iommu_init_dma_ops(void)
* protection domain will be assigned to the default one. * protection domain will be assigned to the default one.
*/ */
for_each_iommu(iommu) { for_each_iommu(iommu) {
iommu->default_dom = dma_ops_domain_alloc(iommu); iommu->default_dom = dma_ops_domain_alloc();
if (iommu->default_dom == NULL) if (iommu->default_dom == NULL)
return -ENOMEM; return -ENOMEM;
iommu->default_dom->domain.flags |= PD_DEFAULT_MASK; iommu->default_dom->domain.flags |= PD_DEFAULT_MASK;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment