Re: [PATCH 3/8] mtd: rawnand: qcom: Fix alignment with open parenthesis

From: Manivannan Sadhasivam <mani@kernel.org>
To: Miquel Raynal <miquel.raynal@bootlin.com>
Cc: Md Sadre Alam <quic_mdalam@quicinc.com>,
	Sricharan Ramabadhran <quic_srichara@quicinc.com>,
	Richard Weinberger <richard@nod.at>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Tudor Ambarus <tudor.ambarus@linaro.org>,
	Pratyush Yadav <pratyush@kernel.org>,
	Michael Walle <michael@walle.cc>,
	linux-mtd@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 3/8] mtd: rawnand: qcom: Fix alignment with open parenthesis
Date: Mon, 17 Jul 2023 07:57:59 +0530	[thread overview]
Message-ID: <20230717022759.GC5996@thinkpad> (raw)
In-Reply-To: <20230716144612.32132-4-miquel.raynal@bootlin.com>

On Sun, Jul 16, 2023 at 04:46:07PM +0200, Miquel Raynal wrote:
> Fix the following checkpatch warning:
>     	"CHECK: Alignment should match open parenthesis"
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

Acked-by: Manivannan Sadhasivam <mani@kernel.org>

- Mani

> ---
>  drivers/mtd/nand/raw/qcom_nandc.c | 16 +++++++---------
>  1 file changed, 7 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c
> index 0136df01738e..54a7b49bda87 100644
> --- a/drivers/mtd/nand/raw/qcom_nandc.c
> +++ b/drivers/mtd/nand/raw/qcom_nandc.c
> @@ -2348,7 +2348,7 @@ static int qcom_nand_ooblayout_ecc(struct mtd_info *mtd, int section,
>  }
>  
>  static int qcom_nand_ooblayout_free(struct mtd_info *mtd, int section,
> -				     struct mtd_oob_region *oobregion)
> +				    struct mtd_oob_region *oobregion)
>  {
>  	struct nand_chip *chip = mtd_to_nand(mtd);
>  	struct qcom_nand_host *host = to_qcom_nand_host(chip);
> @@ -2593,7 +2593,7 @@ static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 cmd,
>  /* NAND framework ->exec_op() hooks and related helpers */
>  static void qcom_parse_instructions(struct nand_chip *chip,
>  				    const struct nand_subop *subop,
> -					struct qcom_op *q_op)
> +				    struct qcom_op *q_op)
>  {
>  	struct qcom_nand_controller *nandc = get_qcom_nand_controller(chip);
>  	const struct nand_op_instr *instr = NULL;
> @@ -3089,19 +3089,17 @@ static int qcom_nandc_alloc(struct qcom_nand_controller *nandc)
>  	 */
>  	nandc->buf_size = 532;
>  
> -	nandc->data_buffer = devm_kzalloc(nandc->dev, nandc->buf_size,
> -					GFP_KERNEL);
> +	nandc->data_buffer = devm_kzalloc(nandc->dev, nandc->buf_size, GFP_KERNEL);
>  	if (!nandc->data_buffer)
>  		return -ENOMEM;
>  
> -	nandc->regs = devm_kzalloc(nandc->dev, sizeof(*nandc->regs),
> -					GFP_KERNEL);
> +	nandc->regs = devm_kzalloc(nandc->dev, sizeof(*nandc->regs), GFP_KERNEL);
>  	if (!nandc->regs)
>  		return -ENOMEM;
>  
> -	nandc->reg_read_buf = devm_kcalloc(nandc->dev,
> -				MAX_REG_RD, sizeof(*nandc->reg_read_buf),
> -				GFP_KERNEL);
> +	nandc->reg_read_buf = devm_kcalloc(nandc->dev, MAX_REG_RD,
> +					   sizeof(*nandc->reg_read_buf),
> +					   GFP_KERNEL);
>  	if (!nandc->reg_read_buf)
>  		return -ENOMEM;
>  
> -- 
> 2.34.1
> 

-- 
மணிவண்ணன் சதாசிவம்

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/