[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20230805174146.57006-3-manivannan.sadhasivam@linaro.org>
Date: Sat, 5 Aug 2023 23:11:38 +0530
From: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
To: miquel.raynal@...tlin.com, vigneshr@...com
Cc: richard@....at, linux-mtd@...ts.infradead.org,
linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
quic_mdalam@...cinc.com, quic_srichara@...cinc.com,
dan.carpenter@...aro.org,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Subject: [PATCH 02/10] mtd: rawnand: qcom: Rename variables in qcom_op_cmd_mapping()
qcom_op_cmd_mapping() function accepts opcode and returns the corresponding
command register. So let's rename the local variables and parameters to
reflect the same.
Reported-by: Miquel Raynal <miquel.raynal@...tlin.com>
Closes: https://lore.kernel.org/all/20230804190750.3367a044@xps-13/
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
---
drivers/mtd/nand/raw/qcom_nandc.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c
index b6751fb17587..be428b80a39b 100644
--- a/drivers/mtd/nand/raw/qcom_nandc.c
+++ b/drivers/mtd/nand/raw/qcom_nandc.c
@@ -2555,39 +2555,39 @@ static int qcom_nand_attach_chip(struct nand_chip *chip)
return 0;
}
-static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 cmd,
+static int qcom_op_cmd_mapping(struct qcom_nand_controller *nandc, u8 opcode,
struct qcom_op *q_op)
{
- int ret;
+ int cmd;
- switch (cmd) {
+ switch (opcode) {
case NAND_CMD_RESET:
- ret = OP_RESET_DEVICE;
+ cmd = OP_RESET_DEVICE;
break;
case NAND_CMD_READID:
- ret = OP_FETCH_ID;
+ cmd = OP_FETCH_ID;
break;
case NAND_CMD_PARAM:
if (nandc->props->qpic_v2)
- ret = OP_PAGE_READ_ONFI_READ;
+ cmd = OP_PAGE_READ_ONFI_READ;
else
- ret = OP_PAGE_READ;
+ cmd = OP_PAGE_READ;
break;
case NAND_CMD_ERASE1:
case NAND_CMD_ERASE2:
- ret = OP_BLOCK_ERASE;
+ cmd = OP_BLOCK_ERASE;
break;
case NAND_CMD_STATUS:
- ret = OP_CHECK_STATUS;
+ cmd = OP_CHECK_STATUS;
break;
case NAND_CMD_PAGEPROG:
- ret = OP_PROGRAM_PAGE;
+ cmd = OP_PROGRAM_PAGE;
q_op->flag = OP_PROGRAM_PAGE;
nandc->exec_opwrite = true;
break;
}
- return ret;
+ return cmd;
}
/* NAND framework ->exec_op() hooks and related helpers */
--
2.25.1
Powered by blists - more mailing lists