diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2021-12-06 23:28:00 +0100 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2021-12-09 11:52:23 +0100 |
| commit | 60bf9b33c82c0e040a98272d7ff4f5a52e7469d6 (patch) | |
| tree | f64e489c481820bfd64c8b026463d93244bebc66 /drivers/pci/msi/msi.c | |
| parent | 890337624e1fa2da079fc1c036a62d178c985280 (diff) | |
| download | linux-60bf9b33c82c0e040a98272d7ff4f5a52e7469d6.tar.bz2 | |
PCI/MSI: Move descriptor counting on allocation fail to the legacy code
The irqdomain code already returns the information. Move the loop to the
legacy code.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Tested-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Link: https://lore.kernel.org/r/20211206210225.101336873@linutronix.de
Diffstat (limited to 'drivers/pci/msi/msi.c')
| -rw-r--r-- | drivers/pci/msi/msi.c | 19 |
1 files changed, 1 insertions, 18 deletions
diff --git a/drivers/pci/msi/msi.c b/drivers/pci/msi/msi.c index 443a16c67e1a..8b4d529792a6 100644 --- a/drivers/pci/msi/msi.c +++ b/drivers/pci/msi/msi.c @@ -609,7 +609,7 @@ static int msix_capability_init(struct pci_dev *dev, struct msix_entry *entries, ret = pci_msi_setup_msi_irqs(dev, nvec, PCI_CAP_ID_MSIX); if (ret) - goto out_avail; + goto out_free; /* Check if all MSI entries honor device restrictions */ ret = msi_verify_entries(dev); @@ -634,23 +634,6 @@ static int msix_capability_init(struct pci_dev *dev, struct msix_entry *entries, pcibios_free_irq(dev); return 0; -out_avail: - if (ret < 0) { - /* - * If we had some success, report the number of IRQs - * we succeeded in setting up. - */ - struct msi_desc *entry; - int avail = 0; - - for_each_pci_msi_entry(entry, dev) { - if (entry->irq != 0) - avail++; - } - if (avail != 0) - ret = avail; - } - out_free: free_msi_irqs(dev); |