From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751991AbcAEL4w (ORCPT ); Tue, 5 Jan 2016 06:56:52 -0500 Received: from youngberry.canonical.com ([91.189.89.112]:40008 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751188AbcAEL4s (ORCPT ); Tue, 5 Jan 2016 06:56:48 -0500 From: Colin King To: James Bottomley , "Martin K . Petersen" , Tomas Henzl , Johannes Thumshirn , linux-scsi@vger.kernel.org Cc: linux-kernel@vger.kernel.org Subject: [PATCH] scsi: mvsas: fix indenting on return error code Date: Tue, 5 Jan 2016 11:56:40 +0000 Message-Id: <1451995000-9210-1-git-send-email-colin.king@canonical.com> X-Mailer: git-send-email 2.6.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Colin Ian King 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 --- 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