2017-12-01 13:33:33

by Colin King

Subject: [PATCH] scsi: ipr: fix incorrect indentation of assignment statement

From: Colin Ian King <[email protected]>

Remove one extraneous level of indentation on an assignment statement.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/scsi/ipr.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
index cc0187965eee..e07dd990e585 100644
--- a/drivers/scsi/ipr.c
+++ b/drivers/scsi/ipr.c
@@ -9653,8 +9653,8 @@ static int ipr_alloc_cmd_blks(struct ipr_ioa_cfg *ioa_cfg)
if (i == 0) {
entries_each_hrrq = IPR_NUM_INTERNAL_CMD_BLKS;
ioa_cfg->hrrq[i].min_cmd_id = 0;
- ioa_cfg->hrrq[i].max_cmd_id =
- (entries_each_hrrq - 1);
+ ioa_cfg->hrrq[i].max_cmd_id =
+ (entries_each_hrrq - 1);
} else {
entries_each_hrrq =
IPR_NUM_BASE_CMD_BLKS/
--
2.14.1

2017-12-01 16:30:54

by Brian King

Subject: Re: [PATCH] scsi: ipr: fix incorrect indentation of assignment statement

On 12/01/2017 07:33 AM, Colin King wrote:
> From: Colin Ian King <[email protected]>
>
> Remove one extraneous level of indentation on an assignment statement.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/scsi/ipr.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
> index cc0187965eee..e07dd990e585 100644
> --- a/drivers/scsi/ipr.c
> +++ b/drivers/scsi/ipr.c
> @@ -9653,8 +9653,8 @@ static int ipr_alloc_cmd_blks(struct ipr_ioa_cfg *ioa_cfg)
> if (i == 0) {
> entries_each_hrrq = IPR_NUM_INTERNAL_CMD_BLKS;
> ioa_cfg->hrrq[i].min_cmd_id = 0;
> - ioa_cfg->hrrq[i].max_cmd_id =
> - (entries_each_hrrq - 1);
> + ioa_cfg->hrrq[i].max_cmd_id =
> + (entries_each_hrrq - 1);
> } else {
> entries_each_hrrq =
> IPR_NUM_BASE_CMD_BLKS/
>

Acked-by: Brian King <[email protected]>

--
Brian King
Power Linux I/O
IBM Linux Technology Center

2017-12-05 03:30:47

by Martin K. Petersen

Subject: Re: [PATCH] scsi: ipr: fix incorrect indentation of assignment statement

Colin,

> Remove one extraneous level of indentation on an assignment statement.

Applied to 4.16/scsi-queue, thanks!

--
Martin K. Petersen Oracle Linux Engineering