diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-29 12:55:40 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-12-06 00:14:20 -0500 |
commit | 6db16718c964d6d7d26cfd03ae1e0e7f9c484862 (patch) | |
tree | 47c6fca52ac16cc332904eab87796ec93b44837f /drivers/net/ethernet/pasemi | |
parent | 04901cea2192ae2ea5a65165ed3174187d19d5c8 (diff) |
net-PA Semi: Deletion of unnecessary checks before the function call "pci_dev_put"
The pci_dev_put() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call
is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Acked-by: Olof Johansson <olof@lixom.net>
Acked-by: Luis R. Rodriguez <mcgrof@suse.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/pasemi')
-rw-r--r-- | drivers/net/ethernet/pasemi/pasemi_mac.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/ethernet/pasemi/pasemi_mac.c b/drivers/net/ethernet/pasemi/pasemi_mac.c index 30d934d66356..44e8d7d25547 100644 --- a/drivers/net/ethernet/pasemi/pasemi_mac.c +++ b/drivers/net/ethernet/pasemi/pasemi_mac.c | |||
@@ -1837,10 +1837,8 @@ pasemi_mac_probe(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
1837 | return err; | 1837 | return err; |
1838 | 1838 | ||
1839 | out: | 1839 | out: |
1840 | if (mac->iob_pdev) | 1840 | pci_dev_put(mac->iob_pdev); |
1841 | pci_dev_put(mac->iob_pdev); | 1841 | pci_dev_put(mac->dma_pdev); |
1842 | if (mac->dma_pdev) | ||
1843 | pci_dev_put(mac->dma_pdev); | ||
1844 | 1842 | ||
1845 | free_netdev(dev); | 1843 | free_netdev(dev); |
1846 | out_disable_device: | 1844 | out_disable_device: |