diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-02 22:08:29 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2015-02-03 07:04:44 -0500 |
| commit | 214f1aff3f33bc86cc85ece4080d9518af688fb1 (patch) | |
| tree | 61c59413055899a1c5760ca90eac9f44939659ab /drivers/ata/pata_cs5530.c | |
| parent | 17263905399471016cda6c1975044d14291c5ba5 (diff) | |
| download | lwn-214f1aff3f33bc86cc85ece4080d9518af688fb1.tar.gz lwn-214f1aff3f33bc86cc85ece4080d9518af688fb1.zip | |
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/pata_cs5530.c')
| -rw-r--r-- | drivers/ata/pata_cs5530.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/ata/pata_cs5530.c b/drivers/ata/pata_cs5530.c index 48ae4b434474..f9ca72e937ee 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; } |