Re: [PATCH 5/8] mtd: rawnand: qcom: Fix wrong indentation

Tudor Ambarus
2023-07-28T02:21:03Z

On 7/16/23 15:46, Miquel Raynal wrote:
> The main "for" loop in qcom_read_status_exec() does guard the following
> to if's which are badly indented. Fix the indentation.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

Reviewed-by: Tudor Ambarus <tudor.ambarus@linaro.org>

> ---
> drivers/mtd/nand/raw/qcom_nandc.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c
> index 554cca3801da..3adc40ef5cf6 100644
> --- a/drivers/mtd/nand/raw/qcom_nandc.c
> +++ b/drivers/mtd/nand/raw/qcom_nandc.c
> @@ -2722,12 +2722,12 @@ static int qcom_read_status_exec(struct nand_chip *chip,
> for (i = 0; i < num_cw; i++) {
> flash_status = le32_to_cpu(nandc->reg_read_buf[i]);
>
> - if (flash_status & FS_MPU_ERR)
> - host->status &= ~NAND_STATUS_WP;
> + if (flash_status & FS_MPU_ERR)
> + host->status &= ~NAND_STATUS_WP;
>
> - if (flash_status & FS_OP_ERR ||
> - (i == (num_cw - 1) && (flash_status & FS_DEVICE_STS_ERR)))
> - host->status |= NAND_STATUS_FAIL;
> + if (flash_status & FS_OP_ERR ||
> + (i == (num_cw - 1) && (flash_status & FS_DEVICE_STS_ERR)))
> + host->status |= NAND_STATUS_FAIL;
> }
>
> flash_status = host->status;