All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ncr53c8xx: Fixup typos
@ 2021-01-25  8:54 Hannes Reinecke
  2021-01-27  3:11 ` Martin K. Petersen
  2021-01-29 19:01 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: Hannes Reinecke @ 2021-01-25  8:54 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Christoph Hellwig, James Bottomley, linux-scsi, Hannes Reinecke,
	kernel test robot

The patch to switch using SAM status values had some typos;
fix them up.

Fixes: 491152c7c3b5 ("scsi: ncr53c8xx: Use SAM status values")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Hannes Reinecke <hare@suse.de>
---
 drivers/scsi/ncr53c8xx.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/scsi/ncr53c8xx.c b/drivers/scsi/ncr53c8xx.c
index 71e97384102a..c76e9f05d042 100644
--- a/drivers/scsi/ncr53c8xx.c
+++ b/drivers/scsi/ncr53c8xx.c
@@ -4999,7 +4999,7 @@ void ncr_complete (struct ncb *np, struct ccb *cp)
 		/*
 		**   SCSI bus reset
 		*/
-		set_status_byte(cmd, sp->scsi_status);
+		set_status_byte(cmd, cp->scsi_status);
 		set_host_byte(cmd, DID_RESET);
 
 	} else if (cp->host_status == HS_ABORTED) {
@@ -6605,7 +6605,7 @@ static void ncr_sir_to_redo(struct ncb *np, int num, struct ccb *cp)
 		OUTL_DSP (NCB_SCRIPT_PHYS (np, reselect));
 		return;
 	case SAM_STAT_COMMAND_TERMINATED:
-	case SAM_STAT_CHECK_CONDIION:
+	case SAM_STAT_CHECK_CONDITION:
 		/*
 		**	If we were requesting sense, give up.
 		*/
@@ -8044,7 +8044,7 @@ printk("ncr53c8xx_queue_command\n");
      spin_lock_irqsave(&np->smp_lock, flags);
 
      if ((sts = ncr_queue_command(np, cmd)) != DID_OK) {
-	     set_host_byte(cmd, sts;
+	     set_host_byte(cmd, sts);
 #ifdef DEBUG_NCR53C8XX
 printk("ncr53c8xx : command not queued - result=%d\n", sts);
 #endif
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ncr53c8xx: Fixup typos
  2021-01-25  8:54 [PATCH] ncr53c8xx: Fixup typos Hannes Reinecke
@ 2021-01-27  3:11 ` Martin K. Petersen
  2021-01-29 19:01 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2021-01-27  3:11 UTC (permalink / raw)
  To: Hannes Reinecke
  Cc: Martin K. Petersen, Christoph Hellwig, James Bottomley,
	linux-scsi, kernel test robot


Hannes,

> The patch to switch using SAM status values had some typos; fix them
> up.

Applied to 5.12/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ncr53c8xx: Fixup typos
  2021-01-25  8:54 [PATCH] ncr53c8xx: Fixup typos Hannes Reinecke
  2021-01-27  3:11 ` Martin K. Petersen
@ 2021-01-29 19:01 ` Martin K. Petersen
  1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2021-01-29 19:01 UTC (permalink / raw)
  To: Hannes Reinecke
  Cc: Martin K . Petersen, Christoph Hellwig, kernel test robot,
	James Bottomley, linux-scsi

On Mon, 25 Jan 2021 09:54:15 +0100, Hannes Reinecke wrote:

> The patch to switch using SAM status values had some typos;
> fix them up.

Applied to 5.12/scsi-queue, thanks!

[1/1] ncr53c8xx: Fixup typos
      https://git.kernel.org/mkp/scsi/c/3ae0819079f2

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-01-29 19:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-25  8:54 [PATCH] ncr53c8xx: Fixup typos Hannes Reinecke
2021-01-27  3:11 ` Martin K. Petersen
2021-01-29 19:01 ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.