diff mbox series

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

Message ID 20230716144612.32132-4-miquel.raynal@bootlin.com (mailing list archive)
State New, archived
Headers show
Series mtd: rawnand: qcom: Misc fixes | expand

Commit Message

Miquel Raynal July 16, 2023, 2:46 p.m. UTC
Fix the following checkpatch warning:
    	"CHECK: Alignment should match open parenthesis"

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
 drivers/mtd/nand/raw/qcom_nandc.c | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

Comments

Manivannan Sadhasivam July 17, 2023, 2:27 a.m. UTC | #1
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
>
Tudor Ambarus July 28, 2023, 2:17 a.m. UTC | #2
On 7/16/23 15:46, Miquel Raynal wrote:
> Fix the following checkpatch warning:
>     	"CHECK: Alignment should match open parenthesis"
> 
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

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

Patch

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;