[PATCH v2 -next] scsi: dpt_i2o: remove set but not used variable 'pHba'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/dpt_i2o.c: In function 'adpt_slave_configure': drivers/scsi/dpt_i2o.c:414:12: warning: variable 'pHba' set but not used [-Wunused-but-set-variable] It never used since git history start. Signed-off-by: YueHaibing --- v2:fix patch

Re: [PATCH -next] scsi: libfc: Remove set but not used variable 'disc'

On 2018/11/22 11:15, Martin K. Petersen wrote: > > YueHaibing, > >> From: Yue Haibing >> >> Fixes gcc '-Wunused-but-set-variable' warning: >> >> drivers/scsi/libfc/fc_rport.c: In function 'fc_rport_recv_flogi_req': >> drivers/scsi/libfc/fc_rpo

[PATCH -next] scsi: dpt_i2o: remove set but not used variable ''

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/dpt_i2o.c: In function 'adpt_slave_configure': drivers/scsi/dpt_i2o.c:414:12: warning: variable 'pHba' set but not used [-Wunused-but-set-variable] It never used since git history start. Signed-off-by: YueHaibing --- drivers/scsi

[PATCH -next] [SCSI] initio: remove set but not used variable 'prev'

nvert into a real Linux driver and update to modern style") Signed-off-by: YueHaibing --- drivers/scsi/initio.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/initio.c b/drivers/scsi/initio.c index 0a8d786c..acef1f2 100644 --- a/drivers/scsi/initio.c +++ b/dr

[PATCH -next] scsi: ufs: Use PTR_ERR_OR_ZERO in ufs_hisi_get_resource()

Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Signed-off-by: YueHaibing --- drivers/scsi/ufs/ufs-hisi.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/scsi/ufs/ufs-hisi.c b/drivers/scsi/ufs/ufs-hisi.c index 452e19f..6fdf059 100644 --- a/drivers/scsi

[PATCH -next] scsi: pmcraid: remove set but not used variables 'res, access'

i: pmcraid: use normal copy_from_user") Signed-off-by: YueHaibing --- drivers/scsi/pmcraid.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/pmcraid.c b/drivers/scsi/pmcraid.c index 707d766..555e4a3 100644 --- a/drivers/scsi/pmcraid.c +++ b/dr

[PATCH -next] scsi: ufs: Fix platform_no_drv_owner.cocci warnings

Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci Signed-off-by: YueHaibing --- drivers/scsi/ufs/cdns-pltfrm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/ufs/cdns-pltfrm.c b/drivers/scsi/ufs

[PATCH -next] scsi: bnx2i: remove set but not used variable 'cid_num'

: Add bnx2i iSCSI driver.") Signed-off-by: YueHaibing --- drivers/scsi/bnx2i/bnx2i_hwi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/bnx2i/bnx2i_hwi.c b/drivers/scsi/bnx2i/bnx2i_hwi.c index 6bad268..91f5316 100644 --- a/drivers/scsi/bnx2i/bnx2i_hwi.c +++ b/drivers/

[PATCH -next] scsi: qedi: remove set but not used variable 'conn'

: Add QLogic FastLinQ offload iSCSI driver framework.") Signed-off-by: YueHaibing --- drivers/scsi/qedi/qedi_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/qedi/qedi_main.c b/drivers/scsi/qedi/qedi_main.c index 105b0e4..a2eb611 100644 --- a/drivers/scsi/qedi/qedi_ma

[PATCH -next] scsi: a100u2w: remove set but not used variable 'bios_phys'

nvert into Linux style") Signed-off-by: YueHaibing --- drivers/scsi/a100u2w.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/scsi/a100u2w.c b/drivers/scsi/a100u2w.c index 00072ed..012aa25 100644 --- a/drivers/scsi/a100u2w.c +++ b/drivers/scsi/a100u2w.c @@ -1089,8 +1089,6 @@

[PATCH -next] scsi: isci: remove set but not used variable 'index'

uests") Signed-off-by: YueHaibing --- drivers/scsi/isci/host.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/isci/host.c b/drivers/scsi/isci/host.c index 7b5deae..7ebfa3c 100644 --- a/drivers/scsi/isci/host.c +++ b/drivers/scsi/isci/host.c @@ -2671,7 +2671,6 @@ enum

[PATCH -next] mvsas: Remove set but not used variable 'id'

pport for 94xx; layout change; bug fixes") Signed-off-by: YueHaibing --- drivers/scsi/mvsas/mv_sas.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c index 3df1428..ab50798 100644 --- a/drivers/scsi/mvsas/mv_sas.c +++ b/drivers/

[PATCH -next] scsi: qedf: remove set but not used variable 'fcport'

61d8658b4a43 ("scsi: qedf: Add QLogic FastLinQ offload FCoE driver framework.") Signed-off-by: YueHaibing --- drivers/scsi/qedf/qedf_main.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/scsi/qedf/qedf_main.c b/drivers/scsi/qedf/qedf_main.c index d5a4f17..04f50a3 100644 ---

[PATCH -next] scsi: arcmsr: Remove set but not used variables 'id, lun'

' set but not used [-Wunused-but-set-variable] It never used since introduction in commit ae52e7f09ff5 Signed-off-by: YueHaibing --- drivers/scsi/arcmsr/arcmsr_hba.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c index

[PATCH -next] scsi: lpfc: Remove set but not used variables 'tgtp'

arget: Add debugfs support") Signed-off-by: YueHaibing --- drivers/scsi/lpfc/lpfc_debugfs.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_debugfs.c b/drivers/scsi/lpfc/lpfc_debugfs.c index aec5b10..0c8005b 100644 --- a/drivers/scsi/lpfc/lpfc_debugfs.c +++ b/dr

[PATCH -next] scsi: lpfc: Remove set but not used variable 'psli'

tions") 'psli' is not used any more. Signed-off-by: YueHaibing --- drivers/scsi/lpfc/lpfc_hbadisc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_hbadisc.c b/drivers/scsi/lpfc/lpfc_hbadisc.c index f9a038ec..f4deb86 100644 --- a/drivers/scsi/lpfc/lpfc_hbadi

[PATCH -next] scsi: ufs-qcom: Remove set but not used variable 'val'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/ufs/ufs-qcom.c: In function 'ufs_qcom_pwr_change_notify': drivers/scsi/ufs/ufs-qcom.c:919:6: warning: variable 'val' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/ufs/ufs-qcom.c | 3 --- 1

[PATCH -next] scsi: lpfc: Remove set but not used variables 'fc_hdr' and 'hw_page_size'

/scsi/lpfc/lpfc_sli.c:14852:11: warning: variable 'hw_page_size' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/lpfc/lpfc_sli.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_sli.c b/drivers/scsi/lpfc/lpfc_sli.c index

[PATCH -next] [SCSI] aic79xx: Use kmemdup rather than duplicating its implementation in ahd_loadseq

Use kmemdup rather than duplicating its implementation Signed-off-by: YueHaibing --- drivers/scsi/aic7xxx/aic79xx_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/aic7xxx/aic79xx_core.c b/drivers/scsi/aic7xxx/aic79xx_core.c index 9ee75c9..0970bb5

[PATCH -next] [SCSI] aic7xxx: Use kmemdup rather than duplicating its implementation in ahc_loadseq

Use kmemdup rather than duplicating its implementation Signed-off-by: YueHaibing --- drivers/scsi/aic7xxx/aic7xxx_core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/aic7xxx/aic7xxx_core.c b/drivers/scsi/aic7xxx/aic7xxx_core.c index f3362f4..8549f1e

[PATCH -next] scsi: mptfusion: Use kmemdup rather than duplicating its implementation in mpt_remove_dead_ioc_func

Use kmemdup rather than duplicating its implementation Signed-off-by: YueHaibing --- drivers/message/fusion/mptbase.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/message/fusion/mptbase.c b/drivers/message/fusion/mptbase.c index dc1e43a..a32d3d0 100644

[PATCH -next] scsi: pm80xx: Remove set but not used variable 'page_code'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/pm8001/pm80xx_hwi.c: In function 'pm8001_set_phy_profile': drivers/scsi/pm8001/pm80xx_hwi.c:4679:6: warning: variable 'page_code' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/pm8001

[PATCH -next] scsi: qla2xxx: Remove set but not used variable 'ptr_dma'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/qla2xxx/qla_iocb.c: In function 'qla24xx_els_dcmd2_iocb': drivers/scsi/qla2xxx/qla_iocb.c:2644:13: warning: variable 'ptr_dma' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/qla2xxx/qla_iocb.c

[PATCH -next] scsi: lpfc: Remove set but not used variable 'sgl_size'

-by: YueHaibing --- drivers/scsi/lpfc/lpfc_nvme.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_nvme.c b/drivers/scsi/lpfc/lpfc_nvme.c index 5438732..d86eb51 100644 --- a/drivers/scsi/lpfc/lpfc_nvme.c +++ b/drivers/scsi/lpfc/lpfc_nvme.c @@ -2235,13

[PATCH -next] scsi: pm80xx: Remove set but not used variable 'device_id'

'pm8001_abort_task': drivers/scsi/pm8001/pm8001_sas.c:1191:6: warning: variable 'device_id' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/pm8001/pm8001_sas.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/scsi/pm8001/pm8001_sas.c b/drivers/scsi

[PATCH -next] scsi: qla2xxx: Use PTR_ERR_OR_ZERO in tcm_qla2xxx_check_initiator_node_acl()

Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Signed-off-by: YueHaibing --- drivers/scsi/qla2xxx/tcm_qla2xxx.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c b/drivers/scsi/qla2xxx/tcm_qla2xxx.c index 731a094..70356d2

[PATCH -next] scsi: bnx2fc: Remove set but not used variable 'oxid'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/bnx2fc/bnx2fc_fcoe.c: In function 'bnx2fc_rcv': drivers/scsi/bnx2fc/bnx2fc_fcoe.c:435:17: warning: variable 'oxid' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 3

[PATCH -next] scsi: aic7xxx: remove set but not used variable 'shared_scb_data'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/aic7xxx/aic79xx_pci.c: In function 'ahd_pci_config': drivers/scsi/aic7xxx/aic79xx_pci.c:291:19: warning: variable 'shared_scb_data' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/aic7xxx

[PATCH -next] scsi: libfc: remove set but not used variable 'rpriv'

Fixes gcc '-Wunused-but-set-variable' warning: drivers/scsi/libfc/fc_fcp.c: In function 'fc_queuecommand': drivers/scsi/libfc/fc_fcp.c:1875:30: warning: variable 'rpriv' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- drivers/scsi/libfc/fc_fcp.c | 3 --- 1 file

[PATCH 3/3] scsi: a100u2w: Using module_pci_driver.

Remove boilerplate code by using macro module_pci_driver. Signed-off-by: YueHaibing <yuehaib...@huawei.com> --- drivers/scsi/a100u2w.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/a100u2w.c b/drivers/scsi/a100u2w.c index 8086bd0..b2942ec

[PATCH 1/3] am53c974: Using module_pci_driver.

Remove boilerplate code by using macro module_pci_driver. Signed-off-by: YueHaibing <yuehaib...@huawei.com> --- drivers/scsi/am53c974.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/am53c974.c b/drivers/scsi/am53c974.c index beea30e..d

[PATCH 2/3] scsi: wd719x: Using module_pci_driver.

Remove boilerplate code by using macro module_pci_driver. Signed-off-by: YueHaibing <yuehaib...@huawei.com> --- drivers/scsi/wd719x.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/wd719x.c b/drivers/scsi/wd719x.c index 2ba2b7b..974bfb3

[PATCH scsi-next] scsi: mvumi: Using module_pci_driver.

Remove boilerplate code by using macro module_pci_driver. Signed-off-by: YueHaibing <yuehaib...@huawei.com> --- drivers/scsi/mvumi.c | 20 +--- 1 file changed, 1 insertion(+), 19 deletions(-) diff --git a/drivers/scsi/mvumi.c b/drivers/scsi/mvumi.c index fe97401..2e6fd86