Commit 242b4a39 authored Oct 12, 2018 by YueHaibing Committed by Martin K. Petersen Oct 16, 2018

scsi: arcmsr: Remove set but not used variables 'id, lun'

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/scsi/arcmsr/arcmsr_hba.c: In function 'arcmsr_drain_donequeue':
drivers/scsi/arcmsr/arcmsr_hba.c:1320:10: warning:
variable 'lun' set but not used [-Wunused-but-set-variable]

drivers/scsi/arcmsr/arcmsr_hba.c:1320:6: warning:
variable 'id' set but not used [-Wunused-but-set-variable]

Never used since introduction in commit ae52e7f0 ("arcmsr: Support 1024 scatter-gather list entries and improve AP while FW trapped and behaviors of EHs").

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

parent 47db7873

drivers/scsi/arcmsr/arcmsr_hba.c

+0 −3

Original line number	Diff line number	Diff line
		@@ -1317,13 +1317,10 @@ static void arcmsr_report_ccb_state(struct AdapterControlBlock *acb,

		static void arcmsr_drain_donequeue(struct AdapterControlBlock acb, struct CommandControlBlock pCCB, bool error)
		{
		int id, lun;
		if ((pCCB->acb != acb) \|\| (pCCB->startdone != ARCMSR_CCB_START)) {
		if (pCCB->startdone == ARCMSR_CCB_ABORTED) {
		struct scsi_cmnd *abortcmd = pCCB->pcmd;
		if (abortcmd) {
		id = abortcmd->device->id;
		lun = abortcmd->device->lun;
		abortcmd->result \|= DID_ABORT << 16;
		arcmsr_ccb_complete(pCCB);
		printk(KERN_NOTICE "arcmsr%d: pCCB ='0x%p' isr got aborted command \n",

Admin message