RE: [PATCH 2/3] scsi: aacraid: get rid of one level of indentation

From: Raghava Aditya Renukunta
Date: Tue Aug 29 2017 - 22:11:07 EST




> -----Original Message-----
> From: Nikola Pajkovsky [mailto:npajkovsky@xxxxxxx]
> Sent: Tuesday, August 29, 2017 4:59 AM
> To: dl-esc-Aacraid Linux Driver <aacraid@xxxxxxxxxxxxx>
> Cc: Nikola Pajkovsky <npajkovsky@xxxxxxx>; James E.J. Bottomley
> <jejb@xxxxxxxxxxxxxxxxxx>; Martin K. Petersen
> <martin.petersen@xxxxxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx
> 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@xxxxxxx>
> ---
[.....]
Reviewed-by: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@xxxxxxxxxxxxx>