diff options
| author | Bernhard Walle <bwalle@suse.de> | 2007-03-27 18:49:49 -0400 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2007-04-01 11:27:29 -0400 |
| commit | 665db93b8a5fc6a11e2ace6b20569420a353d037 (patch) | |
| tree | 156c0ddf6081aa31e2c16163621f389afa6b993d | |
| parent | 4dfb7cbef856689caebd0f498dbd140d1b79954f (diff) | |
[SCSI] qla2xxx: Remove duplicate pci_disable_device() call
On the path qla2x00_probe_one() -> probe_failed -> qla2x00_free_device(),
pci_disable_device() is executed twice, once in qla2x00_free_device()
and once in qla2x00_probe_one().
This patch removes the unnecessary call.
Signed-off-by: Bernhard Walle <bwalle@suse.de>
Acked-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
| -rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index b6c96a8e0337..b78919a318e2 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
| @@ -1707,6 +1707,7 @@ qla2x00_remove_one(struct pci_dev *pdev) | |||
| 1707 | 1707 | ||
| 1708 | scsi_host_put(ha->host); | 1708 | scsi_host_put(ha->host); |
| 1709 | 1709 | ||
| 1710 | pci_disable_device(pdev); | ||
| 1710 | pci_set_drvdata(pdev, NULL); | 1711 | pci_set_drvdata(pdev, NULL); |
| 1711 | } | 1712 | } |
| 1712 | 1713 | ||
| @@ -1749,8 +1750,6 @@ qla2x00_free_device(scsi_qla_host_t *ha) | |||
| 1749 | if (ha->iobase) | 1750 | if (ha->iobase) |
| 1750 | iounmap(ha->iobase); | 1751 | iounmap(ha->iobase); |
| 1751 | pci_release_regions(ha->pdev); | 1752 | pci_release_regions(ha->pdev); |
| 1752 | |||
| 1753 | pci_disable_device(ha->pdev); | ||
| 1754 | } | 1753 | } |
| 1755 | 1754 | ||
| 1756 | static inline void | 1755 | static inline void |
