diff options
author | Alexander Gordeev <agordeev@redhat.com> | 2014-09-07 14:57:53 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-10-01 14:21:14 -0400 |
commit | 6b2fd7efeb888fa781c1f767de6c36497ac1596b (patch) | |
tree | 5c0034e0cd1db2e36e54140a6f1477948016bc0f /arch/powerpc/sysdev/fsl_msi.c | |
parent | 52addcf9d6669fa439387610bc65c92fa0980cef (diff) |
PCI/MSI/PPC: Remove arch_msi_check_device()
Move MSI checks from arch_msi_check_device() to arch_setup_msi_irqs().
This makes the code more compact and allows removing
arch_msi_check_device() from generic MSI code.
Signed-off-by: Alexander Gordeev <agordeev@redhat.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'arch/powerpc/sysdev/fsl_msi.c')
-rw-r--r-- | arch/powerpc/sysdev/fsl_msi.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/arch/powerpc/sysdev/fsl_msi.c b/arch/powerpc/sysdev/fsl_msi.c index 77efbaec7b9c..b32e79dbef4f 100644 --- a/arch/powerpc/sysdev/fsl_msi.c +++ b/arch/powerpc/sysdev/fsl_msi.c | |||
@@ -109,14 +109,6 @@ static int fsl_msi_init_allocator(struct fsl_msi *msi_data) | |||
109 | return 0; | 109 | return 0; |
110 | } | 110 | } |
111 | 111 | ||
112 | static int fsl_msi_check_device(struct pci_dev *pdev, int nvec, int type) | ||
113 | { | ||
114 | if (type == PCI_CAP_ID_MSIX) | ||
115 | pr_debug("fslmsi: MSI-X untested, trying anyway.\n"); | ||
116 | |||
117 | return 0; | ||
118 | } | ||
119 | |||
120 | static void fsl_teardown_msi_irqs(struct pci_dev *pdev) | 112 | static void fsl_teardown_msi_irqs(struct pci_dev *pdev) |
121 | { | 113 | { |
122 | struct msi_desc *entry; | 114 | struct msi_desc *entry; |
@@ -173,6 +165,9 @@ static int fsl_setup_msi_irqs(struct pci_dev *pdev, int nvec, int type) | |||
173 | struct msi_msg msg; | 165 | struct msi_msg msg; |
174 | struct fsl_msi *msi_data; | 166 | struct fsl_msi *msi_data; |
175 | 167 | ||
168 | if (type == PCI_CAP_ID_MSIX) | ||
169 | pr_debug("fslmsi: MSI-X untested, trying anyway.\n"); | ||
170 | |||
176 | /* | 171 | /* |
177 | * If the PCI node has an fsl,msi property, then we need to use it | 172 | * If the PCI node has an fsl,msi property, then we need to use it |
178 | * to find the specific MSI. | 173 | * to find the specific MSI. |
@@ -527,7 +522,6 @@ static int fsl_of_msi_probe(struct platform_device *dev) | |||
527 | if (!ppc_md.setup_msi_irqs) { | 522 | if (!ppc_md.setup_msi_irqs) { |
528 | ppc_md.setup_msi_irqs = fsl_setup_msi_irqs; | 523 | ppc_md.setup_msi_irqs = fsl_setup_msi_irqs; |
529 | ppc_md.teardown_msi_irqs = fsl_teardown_msi_irqs; | 524 | ppc_md.teardown_msi_irqs = fsl_teardown_msi_irqs; |
530 | ppc_md.msi_check_device = fsl_msi_check_device; | ||
531 | } else if (ppc_md.setup_msi_irqs != fsl_setup_msi_irqs) { | 525 | } else if (ppc_md.setup_msi_irqs != fsl_setup_msi_irqs) { |
532 | dev_err(&dev->dev, "Different MSI driver already installed!\n"); | 526 | dev_err(&dev->dev, "Different MSI driver already installed!\n"); |
533 | err = -ENODEV; | 527 | err = -ENODEV; |