diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2014-09-23 16:38:28 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-10-01 14:21:46 -0400 |
commit | 5ec094057c7df5ff80f5e7fe282f47ad205fb976 (patch) | |
tree | 6b7769b0acde248c5ca37ebb825b2a8b6e1409e8 /drivers/pci | |
parent | 56b72b40957947f7c08771f030102351d4c906df (diff) |
PCI/MSI: Remove unnecessary temporary variable
The only use of "status" is to hold a value which is immediately returned,
so just return and remove the variable directly.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci')
-rw-r--r-- | drivers/pci/msi.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/pci/msi.c b/drivers/pci/msi.c index 6413c5a08373..2f7c92c4757a 100644 --- a/drivers/pci/msi.c +++ b/drivers/pci/msi.c | |||
@@ -920,7 +920,7 @@ EXPORT_SYMBOL(pci_msix_vec_count); | |||
920 | **/ | 920 | **/ |
921 | int pci_enable_msix(struct pci_dev *dev, struct msix_entry *entries, int nvec) | 921 | int pci_enable_msix(struct pci_dev *dev, struct msix_entry *entries, int nvec) |
922 | { | 922 | { |
923 | int status, nr_entries; | 923 | int nr_entries; |
924 | int i, j; | 924 | int i, j; |
925 | 925 | ||
926 | if (!pci_msi_supported(dev, nvec)) | 926 | if (!pci_msi_supported(dev, nvec)) |
@@ -951,8 +951,7 @@ int pci_enable_msix(struct pci_dev *dev, struct msix_entry *entries, int nvec) | |||
951 | dev_info(&dev->dev, "can't enable MSI-X (MSI IRQ already assigned)\n"); | 951 | dev_info(&dev->dev, "can't enable MSI-X (MSI IRQ already assigned)\n"); |
952 | return -EINVAL; | 952 | return -EINVAL; |
953 | } | 953 | } |
954 | status = msix_capability_init(dev, entries, nvec); | 954 | return msix_capability_init(dev, entries, nvec); |
955 | return status; | ||
956 | } | 955 | } |
957 | EXPORT_SYMBOL(pci_enable_msix); | 956 | EXPORT_SYMBOL(pci_enable_msix); |
958 | 957 | ||