From: Colin Ian King <colin.king@xxxxxxxxxxxxx> The return code and error return is incorrectly indented which may cause some confusion as it appears at first sight to be associated with a device not ready error (with missing { } braces) rather than a DEV_IS_GONE() failure. The incorrect indenting was introduced by commit 0b15fb1fdfd403726542cb6111bc916b7a9f7fad ("[SCSI] mvsas: add support for Task collector mode and fixed relative bugs") No functional change, just a minor code reformatting clean up. Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> --- drivers/scsi/mvsas/mv_sas.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c index 9c78074..e712fe7 100644 --- a/drivers/scsi/mvsas/mv_sas.c +++ b/drivers/scsi/mvsas/mv_sas.c @@ -737,8 +737,8 @@ static int mvs_task_prep(struct sas_task *task, struct mvs_info *mvi, int is_tmf mv_dprintk("device %016llx not ready.\n", SAS_ADDR(dev->sas_addr)); - rc = SAS_PHY_DOWN; - return rc; + rc = SAS_PHY_DOWN; + return rc; } tei.port = dev->port->lldd_port; if (tei.port && !tei.port->port_attached && !tmf) { -- 2.6.4 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html