diff options
| author | Joerg Roedel <joerg.roedel@amd.com> | 2009-05-04 18:46:34 +0200 |
|---|---|---|
| committer | Joerg Roedel <joerg.roedel@amd.com> | 2009-05-28 18:09:08 +0200 |
| commit | fab6afa30954a0684ef8ac1d9a606e74a6215ab6 (patch) | |
| tree | f397ecccd0294cb714608bccb2fa50b0e9eebfcd /arch | |
| parent | 58492e128892e3b55f1a6ef0cf3c3ab4ce7cc214 (diff) | |
| download | talos-op-linux-fab6afa30954a0684ef8ac1d9a606e74a6215ab6.tar.gz talos-op-linux-fab6afa30954a0684ef8ac1d9a606e74a6215ab6.zip | |
amd-iommu: drop pointless iommu-loop in msi setup code
It is not necessary to loop again over all IOMMUs in this code. So drop
the loop.
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/x86/kernel/amd_iommu_init.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/arch/x86/kernel/amd_iommu_init.c b/arch/x86/kernel/amd_iommu_init.c index 74f4f1fea930..cc99f6092230 100644 --- a/arch/x86/kernel/amd_iommu_init.c +++ b/arch/x86/kernel/amd_iommu_init.c @@ -826,13 +826,6 @@ out_free: static int __init iommu_setup_msi(struct amd_iommu *iommu) { int r; - struct amd_iommu *curr; - - for_each_iommu(curr) { - if (curr->dev == iommu->dev) - curr->int_enabled = true; - } - if (pci_enable_msi(iommu->dev)) return 1; @@ -847,6 +840,7 @@ static int __init iommu_setup_msi(struct amd_iommu *iommu) return 1; } + iommu->int_enabled = true; iommu_feature_enable(iommu, CONTROL_EVT_INT_EN); return 0; |
