Commit c03f2422 authored by Luo Jiaxing's avatar Luo Jiaxing Committed by Martin K. Petersen

scsi: libsas: Correctly indent statements in sas_to_ata_err()

checkpatch reported an error in sas_to_ata_err(). switch and case
statements are incorrectly indented.

Link: https://lore.kernel.org/r/1616675396-6108-2-git-send-email-luojiaxing@huawei.comSigned-off-by: default avatarLuo Jiaxing <luojiaxing@huawei.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent a1e9981b
...@@ -35,46 +35,40 @@ static enum ata_completion_errors sas_to_ata_err(struct task_status_struct *ts) ...@@ -35,46 +35,40 @@ static enum ata_completion_errors sas_to_ata_err(struct task_status_struct *ts)
/* ts->resp == SAS_TASK_COMPLETE */ /* ts->resp == SAS_TASK_COMPLETE */
/* task delivered, what happened afterwards? */ /* task delivered, what happened afterwards? */
switch (ts->stat) { switch (ts->stat) {
case SAS_DEV_NO_RESPONSE: case SAS_DEV_NO_RESPONSE:
return AC_ERR_TIMEOUT; return AC_ERR_TIMEOUT;
case SAS_INTERRUPTED:
case SAS_INTERRUPTED: case SAS_PHY_DOWN:
case SAS_PHY_DOWN: case SAS_NAK_R_ERR:
case SAS_NAK_R_ERR: return AC_ERR_ATA_BUS;
return AC_ERR_ATA_BUS; case SAS_DATA_UNDERRUN:
/*
* Some programs that use the taskfile interface
case SAS_DATA_UNDERRUN: * (smartctl in particular) can cause underrun
/* * problems. Ignore these errors, perhaps at our
* Some programs that use the taskfile interface * peril.
* (smartctl in particular) can cause underrun */
* problems. Ignore these errors, perhaps at our return 0;
* peril. case SAS_DATA_OVERRUN:
*/ case SAS_QUEUE_FULL:
return 0; case SAS_DEVICE_UNKNOWN:
case SAS_SG_ERR:
case SAS_DATA_OVERRUN: return AC_ERR_INVALID;
case SAS_QUEUE_FULL: case SAS_OPEN_TO:
case SAS_DEVICE_UNKNOWN: case SAS_OPEN_REJECT:
case SAS_SG_ERR: pr_warn("%s: Saw error %d. What to do?\n",
return AC_ERR_INVALID; __func__, ts->stat);
return AC_ERR_OTHER;
case SAS_OPEN_TO: case SAM_STAT_CHECK_CONDITION:
case SAS_OPEN_REJECT: case SAS_ABORTED_TASK:
pr_warn("%s: Saw error %d. What to do?\n", return AC_ERR_DEV;
__func__, ts->stat); case SAS_PROTO_RESPONSE:
return AC_ERR_OTHER; /* This means the ending_fis has the error
* value; return 0 here to collect it
case SAM_STAT_CHECK_CONDITION: */
case SAS_ABORTED_TASK: return 0;
return AC_ERR_DEV; default:
return 0;
case SAS_PROTO_RESPONSE:
/* This means the ending_fis has the error
* value; return 0 here to collect it */
return 0;
default:
return 0;
} }
} }
......
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