Commit 9197fe19 authored by Stephen M. Cameron's avatar Stephen M. Cameron Committed by james toy

hpsa: fix typo that causes scsi status to be lost

Signed-off-by: default avatarStephen M. Cameron <scameron@beardog.cce.hp.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Jens Axboe <jens.axboe@oracle.com>
Cc: Mike Miller <mikem@beardog.cce.hp.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent b5b6a8bd
...@@ -987,7 +987,7 @@ static void complete_scsi_command(struct CommandList *cp, ...@@ -987,7 +987,7 @@ static void complete_scsi_command(struct CommandList *cp,
/* Must be some other type of check condition */ /* Must be some other type of check condition */
cmd->result |= (ei->ScsiStatus < 1); cmd->result |= (ei->ScsiStatus << 1);
dev_warn(&h->pdev->dev, "cp %p has check condition: " dev_warn(&h->pdev->dev, "cp %p has check condition: "
"unknown type: " "unknown type: "
"Sense: 0x%x, ASC: 0x%x, ASCQ: 0x%x, " "Sense: 0x%x, ASC: 0x%x, ASCQ: 0x%x, "
...@@ -1010,7 +1010,7 @@ static void complete_scsi_command(struct CommandList *cp, ...@@ -1010,7 +1010,7 @@ static void complete_scsi_command(struct CommandList *cp,
*/ */
if (ei->ScsiStatus) { if (ei->ScsiStatus) {
cmd->result |= (ei->ScsiStatus < 1); cmd->result |= (ei->ScsiStatus << 1);
dev_warn(&h->pdev->dev, "cp %p has status 0x%x " dev_warn(&h->pdev->dev, "cp %p has status 0x%x "
"Sense: 0x%x, ASC: 0x%x, ASCQ: 0x%x, " "Sense: 0x%x, ASC: 0x%x, ASCQ: 0x%x, "
"Returning result: 0x%x\n", "Returning result: 0x%x\n",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment