Commit ddb99b1d authored by Hannes Reinecke's avatar Hannes Reinecke Committed by Martin K. Petersen
Browse files

scsi: mac53c94: Do not set invalid command result 

CMD_ACCEPT_MSG is an internal definition and most certainly not a SCSI
status. As the latter gets set during command completion we can drop the
assignment here.

Link: https://lore.kernel.org/r/20210113090500.129644-26-hare@suse.de


Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarHannes Reinecke <hare@suse.de>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent f3272258

drivers/scsi/mac53c94.c

+0 −1
Original line number Diff line number Diff line
@@ -326,7 +326,6 @@ static void mac53c94_interrupt(int irq, void *dev_id) 
		}

		cmd->SCp.Status = readb(&regs->fifo);

		cmd->SCp.Message = readb(&regs->fifo);

		cmd->result = CMD_ACCEPT_MSG;

		writeb(CMD_ACCEPT_MSG, &regs->command);

		state->phase = busfreeing;

		break;