diff options
author | Joerg Roedel <joerg.roedel@amd.com> | 2009-05-04 12:46:34 -0400 |
---|---|---|
committer | Joerg Roedel <joerg.roedel@amd.com> | 2009-05-28 12:09:08 -0400 |
commit | fab6afa30954a0684ef8ac1d9a606e74a6215ab6 (patch) | |
tree | f397ecccd0294cb714608bccb2fa50b0e9eebfcd /arch/x86/kernel/amd_iommu_init.c | |
parent | 58492e128892e3b55f1a6ef0cf3c3ab4ce7cc214 (diff) |
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/x86/kernel/amd_iommu_init.c')
-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: | |||
826 | static int __init iommu_setup_msi(struct amd_iommu *iommu) | 826 | static int __init iommu_setup_msi(struct amd_iommu *iommu) |
827 | { | 827 | { |
828 | int r; | 828 | int r; |
829 | struct amd_iommu *curr; | ||
830 | |||
831 | for_each_iommu(curr) { | ||
832 | if (curr->dev == iommu->dev) | ||
833 | curr->int_enabled = true; | ||
834 | } | ||
835 | |||
836 | 829 | ||
837 | if (pci_enable_msi(iommu->dev)) | 830 | if (pci_enable_msi(iommu->dev)) |
838 | return 1; | 831 | return 1; |
@@ -847,6 +840,7 @@ static int __init iommu_setup_msi(struct amd_iommu *iommu) | |||
847 | return 1; | 840 | return 1; |
848 | } | 841 | } |
849 | 842 | ||
843 | iommu->int_enabled = true; | ||
850 | iommu_feature_enable(iommu, CONTROL_EVT_INT_EN); | 844 | iommu_feature_enable(iommu, CONTROL_EVT_INT_EN); |
851 | 845 | ||
852 | return 0; | 846 | return 0; |