raspi/linux
1
0
Fork 1
mirror of https://github.com/raspberrypi/linux.git synced 2025-12-06 01:49:46 +00:00
Code Issues Projects Releases Wiki Activity

scsi: qla1280: Remove set but not used variable in qla1280_status_entry()

Browse Source 
This addresses the following gcc warning with "make W=1":

drivers/scsi/qla1280.c: In function ‘qla1280_status_entry’:
drivers/scsi/qla1280.c:3607:28: warning: variable ‘lun’ set but not used
[-Wunused-but-set-variable]
 3607 |  unsigned int bus, target, lun;
      |                            ^~~
drivers/scsi/qla1280.c:3607:20: warning: variable ‘target’ set but not
used [-Wunused-but-set-variable]
 3607 |  unsigned int bus, target, lun;
      |                    ^~~~~~
drivers/scsi/qla1280.c:3607:15: warning: variable ‘bus’ set but not used
[-Wunused-but-set-variable]
 3607 |  unsigned int bus, target, lun;
      |               ^~~

Link: https://lore.kernel.org/r/20200907074518.2326360-5-yanaijie@huawei.com
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Jason Yan <yanaijie@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Jason Yan
2020-09-07 15:45:18 +08:00
committed by Martin K. Petersen
parent bf70bf28bf
commit c8d67fbb60
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
drivers/scsi/qla1280.c
View File

@@ -3601,7 +3601,6 @@ static void
qla1280_status_entry(struct scsi_qla_host *ha, struct response *pkt, qla1280_status_entry(struct scsi_qla_host *ha, struct response *pkt,
struct list_head *done_q) struct list_head *done_q)
{ {
unsigned int bus, target, lun;
int sense_sz; int sense_sz;
struct srb *sp; struct srb *sp;
struct scsi_cmnd *cmd; struct scsi_cmnd *cmd;
@@ -3627,11 +3626,6 @@ qla1280_status_entry(struct scsi_qla_host *ha, struct response *pkt,
cmd = sp->cmd; cmd = sp->cmd;
/* Generate LU queue on cntrl, target, LUN */
bus = SCSI_BUS_32(cmd);
target = SCSI_TCN_32(cmd);
lun = SCSI_LUN_32(cmd);
if (comp_status || scsi_status) { if (comp_status || scsi_status) {
dprintk(3, "scsi: comp_status = 0x%x, scsi_status = " dprintk(3, "scsi: comp_status = 0x%x, scsi_status = "
"0x%x, handle = 0x%x\n", comp_status, "0x%x, handle = 0x%x\n", comp_status,
@@ -3670,7 +3664,8 @@ qla1280_status_entry(struct scsi_qla_host *ha, struct response *pkt,
dprintk(2, "qla1280_status_entry: Check " dprintk(2, "qla1280_status_entry: Check "
"condition Sense data, b %i, t %i, " "condition Sense data, b %i, t %i, "
"l %i\n", bus, target, lun); "l %i\n", SCSI_BUS_32(cmd), SCSI_TCN_32(cmd),
SCSI_LUN_32(cmd));
if (sense_sz) if (sense_sz)
qla1280_dump_buffer(2, qla1280_dump_buffer(2,
(char *)cmd->sense_buffer, (char *)cmd->sense_buffer,