diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-19 16:00:13 +0100 |
|---|---|---|
| committer | Borislav Petkov <bp@suse.de> | 2014-11-19 16:33:48 +0100 |
| commit | 0a98babd85b2e69d64ef6f2f8a5d029ddaa702e1 (patch) | |
| tree | ac47547d6f75491f4cd75e9f0d2e1f27482bd937 /drivers/edac/i3200_edac.c | |
| parent | EDAC, pci_sysfs: remove unneccessary ifdef around entire file (diff) | |
| download | linux-0a98babd85b2e69d64ef6f2f8a5d029ddaa702e1.tar.xz linux-0a98babd85b2e69d64ef6f2f8a5d029ddaa702e1.zip | |
EDAC: Delete unnecessary check before calling pci_dev_put()
The pci_dev_put() function tests whether its argument is NULL and then
returns immediately. Thus the test before the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Link: http://lkml.kernel.org/r/546CB20D.4070808@users.sourceforge.net
[ Boris: commit message. ]
Signed-off-by: Borislav Petkov <bp@suse.de>
Diffstat (limited to 'drivers/edac/i3200_edac.c')
| -rw-r--r-- | drivers/edac/i3200_edac.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/edac/i3200_edac.c b/drivers/edac/i3200_edac.c index 022a70273ada..d1ff0ea2d3f3 100644 --- a/drivers/edac/i3200_edac.c +++ b/drivers/edac/i3200_edac.c @@ -523,8 +523,7 @@ fail1: pci_unregister_driver(&i3200_driver); fail0: - if (mci_pdev) - pci_dev_put(mci_pdev); + pci_dev_put(mci_pdev); return pci_rc; } |