SCSI: remove unnecessary pci_set_drvdata()
Since commit 0998d06310
(device-core: Ensure drvdata = NULL when no driver is bound),
the driver core clears the driver data to NULL after device_release
or on probe failure. Thus, it is not needed to manually clear the
device driver data to NULL.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Cc: James Bottomley <JBottomley@parallels.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
parent
6d3be300c6
commit
08b7e10716
|
@ -1035,7 +1035,6 @@ static void arcmsr_remove(struct pci_dev *pdev)
|
|||
pci_release_regions(pdev);
|
||||
scsi_host_put(host);
|
||||
pci_disable_device(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
|
||||
static void arcmsr_shutdown(struct pci_dev *pdev)
|
||||
|
|
|
@ -3144,8 +3144,6 @@ static void atp870u_remove (struct pci_dev *pdev)
|
|||
atp870u_free_tables(pshost);
|
||||
printk(KERN_INFO "scsi_host_put : %p\n",pshost);
|
||||
scsi_host_put(pshost);
|
||||
printk(KERN_INFO "pci_set_drvdata : %p\n",pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
MODULE_LICENSE("GPL");
|
||||
|
||||
|
|
|
@ -833,7 +833,6 @@ bfad_pci_uninit(struct pci_dev *pdev, struct bfad_s *bfad)
|
|||
/* Disable PCIE Advanced Error Recovery (AER) */
|
||||
pci_disable_pcie_error_reporting(pdev);
|
||||
pci_disable_device(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
|
||||
bfa_status_t
|
||||
|
|
|
@ -1010,7 +1010,6 @@ err_lnode_exit:
|
|||
csio_hw_stop(hw);
|
||||
spin_unlock_irq(&hw->lock);
|
||||
csio_lnodes_unblock_request(hw);
|
||||
pci_set_drvdata(hw->pdev, NULL);
|
||||
csio_lnodes_exit(hw, 0);
|
||||
csio_hw_free(hw);
|
||||
err_pci_exit:
|
||||
|
@ -1044,7 +1043,6 @@ static void csio_remove_one(struct pci_dev *pdev)
|
|||
|
||||
csio_lnodes_exit(hw, 0);
|
||||
csio_hw_free(hw);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
csio_pci_exit(pdev, &bars);
|
||||
}
|
||||
|
||||
|
|
|
@ -4859,7 +4859,6 @@ static void dc395x_remove_one(struct pci_dev *dev)
|
|||
adapter_uninit(acb);
|
||||
pci_disable_device(dev);
|
||||
scsi_host_put(scsi_host);
|
||||
pci_set_drvdata(dev, NULL);
|
||||
}
|
||||
|
||||
|
||||
|
|
|
@ -877,7 +877,6 @@ static void fnic_remove(struct pci_dev *pdev)
|
|||
fnic_iounmap(fnic);
|
||||
pci_release_regions(pdev);
|
||||
pci_disable_device(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
scsi_host_put(lp->host);
|
||||
}
|
||||
|
||||
|
|
|
@ -594,8 +594,6 @@ static void gdth_pci_remove_one(struct pci_dev *pdev)
|
|||
{
|
||||
gdth_ha_str *ha = pci_get_drvdata(pdev);
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
|
||||
list_del(&ha->list);
|
||||
gdth_remove_one(ha);
|
||||
|
||||
|
|
|
@ -4994,7 +4994,6 @@ static void hpsa_remove_one(struct pci_dev *pdev)
|
|||
kfree(h->hba_inquiry_data);
|
||||
pci_disable_device(pdev);
|
||||
pci_release_regions(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
kfree(h);
|
||||
}
|
||||
|
||||
|
|
|
@ -4581,8 +4581,6 @@ lpfc_disable_pci_dev(struct lpfc_hba *phba)
|
|||
/* Release PCI resource and disable PCI device */
|
||||
pci_release_selected_regions(pdev, bars);
|
||||
pci_disable_device(pdev);
|
||||
/* Null out PCI private reference to driver */
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
|
||||
return;
|
||||
}
|
||||
|
@ -9421,7 +9419,6 @@ lpfc_pci_remove_one_s3(struct pci_dev *pdev)
|
|||
/* Disable interrupt */
|
||||
lpfc_sli_disable_intr(phba);
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
scsi_host_put(shost);
|
||||
|
||||
/*
|
||||
|
|
|
@ -534,7 +534,6 @@ megaraid_probe_one(struct pci_dev *pdev, const struct pci_device_id *id)
|
|||
return 0;
|
||||
|
||||
out_cmm_unreg:
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
megaraid_cmm_unregister(adapter);
|
||||
out_fini_mbox:
|
||||
megaraid_fini_mbox(adapter);
|
||||
|
@ -594,11 +593,6 @@ megaraid_detach_one(struct pci_dev *pdev)
|
|||
// detach from the IO sub-system
|
||||
megaraid_io_detach(adapter);
|
||||
|
||||
// reset the device state in the PCI structure. We check this
|
||||
// condition when we enter here. If the device state is NULL,
|
||||
// that would mean the device has already been removed
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
|
||||
// Unregister from common management module
|
||||
//
|
||||
// FIXME: this must return success or failure for conditions if there
|
||||
|
|
|
@ -4323,7 +4323,6 @@ retry_irq_register:
|
|||
megasas_mgmt_info.instance[megasas_mgmt_info.max_index] = NULL;
|
||||
megasas_mgmt_info.max_index--;
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
instance->instancet->disable_intr(instance);
|
||||
if (instance->msix_vectors)
|
||||
for (i = 0 ; i < instance->msix_vectors; i++)
|
||||
|
@ -4679,8 +4678,6 @@ static void megasas_detach_one(struct pci_dev *pdev)
|
|||
}
|
||||
}
|
||||
|
||||
pci_set_drvdata(instance->pdev, NULL);
|
||||
|
||||
instance->instancet->disable_intr(instance);
|
||||
|
||||
if (instance->msix_vectors)
|
||||
|
@ -4722,8 +4719,6 @@ static void megasas_detach_one(struct pci_dev *pdev)
|
|||
instance->evt_detail, instance->evt_detail_h);
|
||||
scsi_host_put(host);
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
|
||||
pci_disable_device(pdev);
|
||||
|
||||
return;
|
||||
|
|
|
@ -657,7 +657,6 @@ static void mvs_pci_remove(struct pci_dev *pdev)
|
|||
tasklet_kill(&((struct mvs_prv_info *)sha->lldd_ha)->mv_tasklet);
|
||||
#endif
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
sas_unregister_ha(sha);
|
||||
sas_remove_host(mvi->shost);
|
||||
scsi_remove_host(mvi->shost);
|
||||
|
|
|
@ -2583,7 +2583,6 @@ static int mvumi_probe_one(struct pci_dev *pdev, const struct pci_device_id *id)
|
|||
return 0;
|
||||
|
||||
fail_io_attach:
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
mhba->instancet->disable_intr(mhba);
|
||||
free_irq(mhba->pdev->irq, mhba);
|
||||
fail_init_irq:
|
||||
|
@ -2618,7 +2617,6 @@ static void mvumi_detach_one(struct pci_dev *pdev)
|
|||
free_irq(mhba->pdev->irq, mhba);
|
||||
mvumi_release_fw(mhba);
|
||||
scsi_host_put(host);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
pci_disable_device(pdev);
|
||||
dev_dbg(&pdev->dev, "driver is removed!\n");
|
||||
}
|
||||
|
|
|
@ -873,7 +873,6 @@ static void pm8001_pci_remove(struct pci_dev *pdev)
|
|||
struct pm8001_hba_info *pm8001_ha;
|
||||
int i;
|
||||
pm8001_ha = sha->lldd_ha;
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
sas_unregister_ha(sha);
|
||||
sas_remove_host(pm8001_ha->shost);
|
||||
list_del(&pm8001_ha->list);
|
||||
|
|
|
@ -6049,7 +6049,6 @@ out_release_regions:
|
|||
|
||||
out_disable_device:
|
||||
atomic_dec(&pmcraid_adapter_count);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
pci_disable_device(pdev);
|
||||
return -ENODEV;
|
||||
}
|
||||
|
|
|
@ -3183,7 +3183,6 @@ qla2x00_remove_one(struct pci_dev *pdev)
|
|||
pci_disable_pcie_error_reporting(pdev);
|
||||
|
||||
pci_disable_device(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
|
||||
static void
|
||||
|
|
|
@ -7400,7 +7400,6 @@ static void qla4xxx_remove_adapter(struct pci_dev *pdev)
|
|||
|
||||
pci_disable_pcie_error_reporting(pdev);
|
||||
pci_disable_device(pdev);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -1790,8 +1790,6 @@ static void stex_remove(struct pci_dev *pdev)
|
|||
|
||||
scsi_remove_host(hba->host);
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
|
||||
stex_hba_stop(hba);
|
||||
|
||||
stex_hba_free(hba);
|
||||
|
|
|
@ -2553,7 +2553,6 @@ static void dc390_remove_one(struct pci_dev *dev)
|
|||
|
||||
pci_disable_device(dev);
|
||||
scsi_host_put(scsi_host);
|
||||
pci_set_drvdata(dev, NULL);
|
||||
}
|
||||
|
||||
static struct pci_device_id tmscsim_pci_tbl[] = {
|
||||
|
|
|
@ -132,7 +132,6 @@ static void ufshcd_pci_remove(struct pci_dev *pdev)
|
|||
pm_runtime_forbid(&pdev->dev);
|
||||
pm_runtime_get_noresume(&pdev->dev);
|
||||
ufshcd_remove(hba);
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -1405,7 +1405,6 @@ out_release_resources:
|
|||
out_free_host:
|
||||
scsi_host_put(host);
|
||||
out_disable_device:
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
pci_disable_device(pdev);
|
||||
|
||||
return error;
|
||||
|
@ -1445,7 +1444,6 @@ static void pvscsi_remove(struct pci_dev *pdev)
|
|||
|
||||
scsi_host_put(host);
|
||||
|
||||
pci_set_drvdata(pdev, NULL);
|
||||
pci_disable_device(pdev);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue