From: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com>
To: Nikola Pajkovsky <npajkovsky@suse.cz>,
        dl-esc-Aacraid Linux Driver <aacraid@microsemi.com>
CC: "James E.J. Bottomley" <jejb@linux.vnet.ibm.com>,
        "Martin K. Petersen" <martin.petersen@oracle.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 2/3] scsi: aacraid: get rid of one level of indentation
Thread-Topic: [PATCH 2/3] scsi: aacraid: get rid of one level of indentation
Thread-Index: AQHTIL476E4/LN/vc0GZ615APZpcbqKcKK4Q
Date: Wed, 30 Aug 2017 02:10:35 +0000
Message-ID: <4D8E82A446BF54499747901DBDEB737A8D0C198B@avsrvexchmbx1.microsemi.net>
References: <d039d8dcc35ced04bf27f20a653532429486d190.1504007944.git.npajkovsky@suse.cz>
 <bc42b0e33f50825a6227c49459f253557d862b24.1504007944.git.npajkovsky@suse.cz>
In-Reply-To: <bc42b0e33f50825a6227c49459f253557d862b24.1504007944.git.npajkovsky@suse.cz>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="Windows-1252"
MIME-Version: 1.0
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Aug 2017 02:10:59.1900
 (UTC)
X-MS-Exchange-CrossTenant-Id: f267a5c8-86d8-4cc9-af71-1fd2c67c8fad
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f267a5c8-86d8-4cc9-af71-1fd2c67c8fad;Ip=[208.19.100.20];Helo=[avsrvexchhts2.microsemi.net]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR02MB2197
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v7U2B8Lk001045
Content-Length: 991
Lines: 39


> -----Original Message-----
> From: Nikola Pajkovsky [mailto:npajkovsky@suse.cz]
> Sent: Tuesday, August 29, 2017 4:59 AM
> To: dl-esc-Aacraid Linux Driver <aacraid@microsemi.com>
> Cc: Nikola Pajkovsky <npajkovsky@suse.cz>; James E.J. Bottomley
> <jejb@linux.vnet.ibm.com>; Martin K. Petersen
> <martin.petersen@oracle.com>; linux-scsi@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: [PATCH 2/3] scsi: aacraid: get rid of one level of indentation
> 
> EXTERNAL EMAIL
> 
> 
>   unsigned long byte_count = 0;
>   nseg = scsi_dma_map(scsicmd);
>   if (nseg < 0)
>      return nseg;
>   if (nseg) {
>      ...
>   }
>   return byte_count;
> 
> is equal to
> 
>   unsigned long byte_count = 0;
>   nseg = scsi_dma_map(scsicmd);
>   if (nseg <= 0)
>      return nseg;
>   ...
>   return byte_count;
> 
> No other code has changed.
> 
> Signed-off-by: Nikola Pajkovsky <npajkovsky@suse.cz>
> ---
[.....]
Reviewed-by: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@microsemi.com>