Received-SPF: pass (google.com: domain of linux-kernel+bounces-30628-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
IronPort-PHdr: A9a23:nwBeoxKF2nRq87iMLtmcua0yDhhOgF28FgcR7pxijKpBbeH/uZ/jJ
 0fYo/5qiQyBUYba7qdcgvHN++D7WGMG6Iqcqn1KbpFWVhEEhMlX1wwtCcKIEwv6edbhbjcxG
 4JJU1o2t2qjPx1tEd3lL0bXvmX06DcTHhvlMg8gJe3vBo/Whsef3OGp8JqVaAJN13KxZLpoJ
 0CupB7K/okO1JFvKKs61lPFo2AdfeNQyCIgKQeYng334YG7+5sLzg==
IronPort-Data: A9a23:JiJpsq3h7Z//g3b9A/bD5fVxkn2cJEfYwER7XKvMYLTBsI5bpzBWz
 GcXDT2OO/iIZGekeNtwOovkpkpQu5KDyNBjQVQ53Hw8FHgiRegpqji6wuYcGwvIc6UvmWo+t
 512huHodZ1yFjmE4E71btANlFEkvYmQXL3wFeXYDS54QA5gWU8JhAlq8wIDqtYAbeORXUXV5
 Lsen+WFYAX5g2ctYjpOg06+gEoHUMra6WtwUmMWPZinjHeG/1EJAZQWI72GLneQauG4ycbjG
 o4vZJnglo/o109F5uGNy94XQWVWKlLmBjViv1INM0SUbreukQRpukozHKJ0hU66EFxllfgpo
 DlGncTYpQvEosQglcxFOyS0HR2SMoVs/qbLDEKDg/bP4E78VWnO5vJVAhAPaNhwFuZfWQmi9
 NQCIzwLKxuEne/znPSwS/JngYIoK8yD0IE34y47i2qGS6d9B8meHs0m5vcAtNs0rsNHB+rfY
 8MaQTFudx/HJRZIPz/7Dbpnw7331yekI2cwRFS9hoop42fRxT5K4KXcD/aLIsaBadRckRPNz
 o7B1z+kWk5BboP3JSC+2natgPLf2CD2QoQfEJWm+fNwxl6e3GoeDFsRT1TTif24jFOuHslUM
 E085CUjt+4x+VatQ927WAe3yENopTYGUNZWVuY98gzIluzf4h2SAS4PSTsphMEaWNEebzoz2
 nKzo+HTWRNsipuHa1eazbnFlGbnUcQKFlMqaSgBRAoDxtDspoAvkx7CJuqP9obo3rUZ/hmtk
 1i3QDgCulkFsSIcO0yGEb3vmTmgoN3CSRQ4o1mRVWO+5QQ/b4mgD2BJ1bQ5xagdRGp6ZgDd1
 JThpyR4xLteZX1qvHfdKNjh5Jnzu5643MT02DaD5aUJ+TW34GKEdotN+jx4L0oBGp9bIWK3O
 BWN6VwMucY70J6WgUlfPtPZ5yMCkPmIKDgZfqC8gidmO8EuJFHdoEmCm2bJgz6z+KTTrU3PE
 czGKZn3Vyly5VVPxzutTOBVyq4w2i073ivSQ5u9pylLIpLADEN5vYwtaQPUBshgtfvsiFyMr
 753aZDQoz0BC7KWX8Ui2dNJRbz8BSJlVcmeRg0+XrPrHzeK70l/UKGAnu99ININcmY8vr6gw
 0xRk3RwkTLXrXbGMg6NLHtkbdvSsVxX9BrX4QRE0Y6U5kUe
IronPort-HdrOrdr: A9a23:OKvjVaPynSqwksBcT47255DYdb4zR+YMi2TDiHoBKiC9I/b5qy
 nxppUmPEfP+UgssREb9expOMG7MBXhHO1OkPgs1NCZLUbbUQqTXc1fBOTZskfd8kHFh4pgPO
 JbAtdD4b7LfBZHZKTBkXSF+r8bqbHtntHL9ILjJjVWPH1Xgspbnn5E43OgYzZLrX59dOIE/f
 Snl616jgvlU046Ku68AX4IVfXCodrkqLLKCCRtOzcXrCO1oXeN8rDVLzi0ty1yb9pI+9gf2F
 mAtza8yrSosvm9xBOZ/XTU9Y5qlNzozcYGLNCQi+AOQw+cyjqAVcBEYfmvrTo1qOag5BIBi9
 /XuSotOMx19jf4Yny1mx3wwAPtuQxeqEMKiGXow0cLk/aJAA7SOPAxwr6xtSGprXbIiesMlZ
 6jGVjp7qa/QymwxBgVrOK4Jy2C3nDE0kbK19RjzkC2leAlGeVsRUt1xjIPLL4QWC3984wpC+
 9oEYXV4+tXa0qTazTDsnBo28HEZAV5Iv6qeDlKhiWu6UkfoFlpi08DgMAPlHYJ85wwD5FC+u
 TfK6xt0LVDVNUfY65xDPoIBZLfMB2BfTvcdGaJZVj3HqAOPHzA75bx/bUu/emvPJgF1oE7lp
 jNWE5R8WQyZ0XtA8uT24AjyGGGfEytGTD2js1O7ZlwvbPxALLtLC2YUVgr19Ctpv0Oa/erLc
 pb+KgmdMMLAVGebbqhhTeOKaW6AUNuJfEohg==
From: "Karan Tilak Kumar (kartilak)" <kartilak@cisco.com>
To: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>,
        "Satish Kharat
 (satishkh)" <satishkh@cisco.com>
CC: "Sesidhar Baddela (sebaddel)" <sebaddel@cisco.com>,
        "jejb@linux.ibm.com"
	<jejb@linux.ibm.com>,
        "martin.petersen@oracle.com"
	<martin.petersen@oracle.com>,
        "linux-scsi@vger.kernel.org"
	<linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>,
        Abaci Robot <abaci@linux.alibaba.com>
Subject: RE: [PATCH] scsi: fnic: clean up some inconsistent indenting
Thread-Topic: [PATCH] scsi: fnic: clean up some inconsistent indenting
Thread-Index: AQHaSbJMF9q6HN+o9kOEpYOIx9JFibDgNafw
Date: Thu, 18 Jan 2024 23:21:43 +0000
Message-ID:
 <SJ0PR11MB58969F9C068CDD09DB63728CC3712@SJ0PR11MB5896.namprd11.prod.outlook.com>
References: <20240118020128.24432-1-jiapeng.chong@linux.alibaba.com>
In-Reply-To: <20240118020128.24432-1-jiapeng.chong@linux.alibaba.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0:
 =?us-ascii?Q?P9hAqg2xUhBRAwyAT39dCO6eYj4CG6lpqW7L8bvDNMs9EuJe619NQyG9xgt5?=
 =?us-ascii?Q?NajgELxRElTAnh0oSWGcD9dlZENtLeED3/FpRaK7UvRsJBtBA1NVkqsZQzIs?=
 =?us-ascii?Q?noIMVFfbclrmHayKoea1p0HatadtKo+oR0Q9hO8zBMGSYNhFu+khw+U1IS73?=
 =?us-ascii?Q?wlEh5apxy+9L8oQIM5Gy+66crupE9MtTks8Kzt4iT7aDdi0Akd6nfmSNmOrP?=
 =?us-ascii?Q?9XfKAXDOT61MQ5banYQi/0RTPmb6Ur0pMRkMWJTrP+YpgQOnkBmNwTYcdRUa?=
 =?us-ascii?Q?TXxK3gVZ2G/8RPY0WsZomxxKJlNKRDDD6TNy9zO/WxQrjrcwpnOqGM1VKzei?=
 =?us-ascii?Q?etIQWmyNnVDLYCP2TUV/NcUQGhRARjnxG/+P9zynACajZ5IjprLVWl0+Hdv3?=
 =?us-ascii?Q?7zfOXkcgIgOzfrZ9Rywmw54KhU2efgXzRpLN9DgLsoUVY+CD+3VgXwnwnqg8?=
 =?us-ascii?Q?4ZWx8Kwet1db87H5n0ZG0SiWvaQDxd56cQhUUGFE+KVRC6gnInOkhN/k2qtE?=
 =?us-ascii?Q?gVdFwvrp8kBxxwQssKdFMndwHPGB4ZtbvUrHPNZWzPz8uLNQsf1i9gtQ0EZH?=
 =?us-ascii?Q?FPKjCTpZ8Wv/vkYzw1NQzEW0wSpCTM54NldTbTgPI8diklXPM9jGvdWhG15y?=
 =?us-ascii?Q?y39KjwfN3hxx5FaAdCR9GahqvnnZw+i2hOSLmwbci0gaXKTxwQejBPKkeS/2?=
 =?us-ascii?Q?rzqvvhnO67t4Xr6eReQPgqnvIN3aU6/PFVmGM2+LgoU3gm3UjG2Qw5UXFsUN?=
 =?us-ascii?Q?eMYCT1Vmh8UFwxB8FsnNGuKGmpck7zYGsW+x8IDplHJnoMeu8ahtlfzKyAqQ?=
 =?us-ascii?Q?FwnJxq46ZyuftGhSxB5rWzYXGiLagDqZ1yMqbYPea4WaLL7ePNdfvHPMqV6+?=
 =?us-ascii?Q?/fbcZOxiHA8/OwCUwmS5H8a3hK795YG5Bbh9wqrULA7v+iVh1vNsHAxORvHj?=
 =?us-ascii?Q?Yup7l9zZZ9BKDMZvSrn8mJcaOLOrp7twDNz/+1cLFYdT4L4TggbnT4ugr8IH?=
 =?us-ascii?Q?JRV2F0nZoLcilcStvuZjGAtqMVFd0iITmlbuPOPNAXOZJ0gNHcXSsGCGH1Ym?=
 =?us-ascii?Q?4wm+1i4KgCK3M7xpauPR7TMqn/Q9q8fWYd1Kqcz8QPX4s5xPgcwY1utKUmxl?=
 =?us-ascii?Q?JtbKKXGbv0d+ugtbDVjquLPnPx6OwyMWjiMULQKkh9VknQFjvRx7aOf/noEm?=
 =?us-ascii?Q?+DQX9NkFaBlZHrfnYnu8QkmFhuea/qOwGPcTZSlMrVOMFQ+vIIzUmtBcikd4?=
 =?us-ascii?Q?nOT/4hX2X3LFIvvzcAuriYmkqz34uqLRE20B7kV9RB+E9OyRoyPszy3lbt2Z?=
 =?us-ascii?Q?osYTVz5CxOVjfos9WI+ugt0lol8zufZ0WNzKGwn4BNAh36i6LO5uUAfB00R+?=
 =?us-ascii?Q?+F7h6jhDTZEnfsGW9p3Silqjdi8rZrkuTqC1EF6aLnx7cBvOVWIgSSsz2U3E?=
 =?us-ascii?Q?IgPtWcP5UdVqprKjebP92yMnn4P7ez8RnqhdBOvBe0/GCdy5hdoWFFXkLrM+?=
 =?us-ascii?Q?JNOxIR9ZrPCH8GnySuidtcJ/rAGhO0/FW573gJl6Os79K6F2EZ6WCvx43o1z?=
 =?us-ascii?Q?m7XVxgM6ZOVUl9uP2/bZZ9WD/5u3DiHkGxFAwO6bPKoYMcCQ0Oou3mvmMZpY?=
 =?us-ascii?Q?jT/FoYww7r13YyxIG0Ku7H8=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: cisco.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR11MB5896.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0a3e4212-bad8-45aa-c793-08dc187c3b70
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Jan 2024 23:21:43.1676
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 1YLgsXGuXFu3zCG08A4N/S4sD6frTBWrSBiulse1BCbvPc1KfCK8Yq0Dc/6K7iEjtOUKvcM9WH0Zri7pjz158Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB4904
X-Outbound-SMTP-Client: 173.37.147.252, alln-opgw-4.cisco.com
X-Outbound-Node: rcdn-core-4.cisco.com

On Wednesday, January 17, 2024 6:01 PM, Jiapeng Chong <jiapeng.chong@linux.=
alibaba.com> wrote:
>
> No functional modification involved.
>
> drivers/scsi/fnic/fnic_scsi.c:1964 fnic_abort_cmd() warn: inconsistent in=
denting.
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=3D7930
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
> ---
> drivers/scsi/fnic/fnic_scsi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/fnic/fnic_scsi.c b/drivers/scsi/fnic/fnic_scsi.=
c index 8d7fc5284293..5b4768e669f0 100644
> --- a/drivers/scsi/fnic/fnic_scsi.c
> +++ b/drivers/scsi/fnic/fnic_scsi.c
> @@ -1961,8 +1961,8 @@ int fnic_abort_cmd(struct scsi_cmnd *sc)
>
> if (!(fnic_priv(sc)->flags & (FNIC_IO_ABORTED | FNIC_IO_DONE))) {
>               spin_unlock_irqrestore(&fnic->wq_copy_lock[hwq], flags);
> -         FNIC_SCSI_DBG(KERN_ERR, fnic->lport->host, fnic->fnic_num,
> -                     "Issuing host reset due to out of order IO\n");
> +             FNIC_SCSI_DBG(KERN_ERR, fnic->lport->host, fnic->fnic_num,
> +                           "Issuing host reset due to out of order IO\n"=
);
>
> ret =3D FAILED;
> goto fnic_abort_cmd_end;
> --
> 2.20.1.7.g153144c
>

Looks good to me.

Reviewed-by: Karan Tilak Kumar <kartilak@cisco.com>

Regards,
Karan