Re: [PATCH v3 08/32] scsi: hisi_sas: add hisi_sas_remove

From	John Garry <john.garry@huawei.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 08/32] scsi: hisi_sas: add hisi_sas_remove
Date	2015-11-10 12:00 +0100
Message-ID	<qteOB-3Za-9@gated-at.bofh.it> (permalink)
References	<qsXkJ-i7-11@gated-at.bofh.it> <qsXkK-i7-25@gated-at.bofh.it> <qt86t-87T-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

>> +int hisi_sas_remove(struct platform_device *pdev)
>> +{
>> +	struct sas_ha_struct *sha = platform_get_drvdata(pdev);
>> +	struct hisi_hba *hisi_hba = sha->lldd_ha;
>> +
>> +	sas_unregister_ha(sha);
>> +	sas_remove_host(sha->core.shost);
>> +	scsi_remove_host(sha->core.shost);
>
> scsi host should be removed before detaching SAS transport.
>
> See more information:
> http://www.spinics.net/lists/linux-scsi/msg90088.html
>
> Thanks,
> Wei
>
OK, I can make this minor change.

Thanks,
John

>> +	hisi_sas_free(hisi_hba);
>> +	return 0;
>> +}
>> +EXPORT_SYMBOL_GPL(hisi_sas_remove);
>> +
>>   static __init int hisi_sas_init(void)
>>   {
>>   	pr_info("hisi_sas: driver version %s\n", DRV_VERSION);
>>
>
>
> .
>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v3 08/32] scsi: hisi_sas: add hisi_sas_remove John Garry <john.garry@huawei.com> - 2015-11-09 17:20 +0100
  Re: [PATCH v3 08/32] scsi: hisi_sas: add hisi_sas_remove Wei Fang <fangwei1@huawei.com> - 2015-11-10 04:50 +0100
    Re: [PATCH v3 08/32] scsi: hisi_sas: add hisi_sas_remove John Garry <john.garry@huawei.com> - 2015-11-10 12:00 +0100

csiph-web