summaryrefslogtreecommitdiffstats
path: root/drivers/ata
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-02-02 22:08:29 +0100
committerTejun Heo <tj@kernel.org>2015-02-03 07:04:44 -0500
commit214f1aff3f33bc86cc85ece4080d9518af688fb1 (patch)
tree61c59413055899a1c5760ca90eac9f44939659ab /drivers/ata
parent17263905399471016cda6c1975044d14291c5ba5 (diff)
downloadop-kernel-dev-214f1aff3f33bc86cc85ece4080d9518af688fb1.zip
op-kernel-dev-214f1aff3f33bc86cc85ece4080d9518af688fb1.tar.gz
ata: Delete 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> Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/ata')
-rw-r--r--drivers/ata/pata_cs5530.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/ata/pata_cs5530.c b/drivers/ata/pata_cs5530.c
index 48ae4b4..f9ca72e 100644
--- a/drivers/ata/pata_cs5530.c
+++ b/drivers/ata/pata_cs5530.c
@@ -276,10 +276,8 @@ static int cs5530_init_chip(void)
pci_dev_put(cs5530_0);
return 0;
fail_put:
- if (master_0)
- pci_dev_put(master_0);
- if (cs5530_0)
- pci_dev_put(cs5530_0);
+ pci_dev_put(master_0);
+ pci_dev_put(cs5530_0);
return -ENODEV;
}
OpenPOWER on IntegriCloud