Commit 659d36cc authored by Nilesh Javali's avatar Nilesh Javali Committed by Martin K. Petersen
Browse files

scsi: qla2xxx: Remove unused variables in qla24xx_build_scsi_type_6_iocbs() 



Sparse warning reported,

drivers/scsi/qla2xxx/qla_iocb.c: In function 'qla24xx_build_scsi_type_6_iocbs':
>> drivers/scsi/qla2xxx/qla_iocb.c:594:29: warning: variable 'ha' set but not used [-Wunused-but-set-variable]
     594 |         struct qla_hw_data *ha;
         |                             ^~

Remove unused variables 'vha' and 'ha'.

Reported-by: default avatarkernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202308230757.VKMIztAB-lkp@intel.com/


Signed-off-by: default avatarNilesh Javali <njavali@marvell.com>
Link: https://lore.kernel.org/r/20230825070017.46066-1-njavali@marvell.com


Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 27177862

drivers/scsi/qla2xxx/qla_iocb.c

+0 −5
Original line number Diff line number Diff line
@@ -591,8 +591,6 @@ qla24xx_build_scsi_type_6_iocbs(srb_t *sp, struct cmd_type_6 *cmd_pkt, 
	uint16_t tot_dsds)

{

	struct dsd64 *cur_dsd = NULL, *next_dsd;

	scsi_qla_host_t	*vha;

	struct qla_hw_data *ha;

	struct scsi_cmnd *cmd;

	struct	scatterlist *cur_seg;

	uint8_t avail_dsds;
@@ -614,9 +612,6 @@ qla24xx_build_scsi_type_6_iocbs(srb_t *sp, struct cmd_type_6 *cmd_pkt, 
		return 0;

	}



	vha = sp->vha;

	ha = vha->hw;



	/* Set transfer direction */

	if (cmd->sc_data_direction == DMA_TO_DEVICE) {

		cmd_pkt->control_flags = cpu_to_le16(CF_WRITE_DATA);