Yijing Wang
2014-09-29 08:35:45 UTC
Irq and msi_desc will be associated in arch MSI
setup code regardless of MSI or MSI-X type.
The irq_set_msi_desc() in msix_program_entries()
is redundant.
The typical MSI setup flow:
pci_enable_msix()
msix_capability_init()
arch_setup_msi_irqs()
native_setup_msi_irqs() e.g in x86
setup_msi_irq()
irq_set_msi_desc_off() <---
Signed-off-by: Yijing Wang <***@huawei.com>
---
drivers/pci/msi.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c
index e0916ad..48eae97 100644
--- a/drivers/pci/msi.c
+++ b/drivers/pci/msi.c
@@ -673,7 +673,6 @@ static void msix_program_entries(struct pci_dev *dev,
PCI_MSIX_ENTRY_VECTOR_CTRL;
entries[i].vector = entry->irq;
- irq_set_msi_desc(entry->irq, entry);
entry->masked = readl(entry->mask_base + offset);
msix_mask_irq(entry, 1);
i++;
setup code regardless of MSI or MSI-X type.
The irq_set_msi_desc() in msix_program_entries()
is redundant.
The typical MSI setup flow:
pci_enable_msix()
msix_capability_init()
arch_setup_msi_irqs()
native_setup_msi_irqs() e.g in x86
setup_msi_irq()
irq_set_msi_desc_off() <---
Signed-off-by: Yijing Wang <***@huawei.com>
---
drivers/pci/msi.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c
index e0916ad..48eae97 100644
--- a/drivers/pci/msi.c
+++ b/drivers/pci/msi.c
@@ -673,7 +673,6 @@ static void msix_program_entries(struct pci_dev *dev,
PCI_MSIX_ENTRY_VECTOR_CTRL;
entries[i].vector = entry->irq;
- irq_set_msi_desc(entry->irq, entry);
entry->masked = readl(entry->mask_base + offset);
msix_mask_irq(entry, 1);
i++;
--
1.7.1
1.7.1