diff mbox series

[v11,4/4] mtd: rawnand: qcom: Add missing nand_cleanup() in error path

Message ID 20210402150128.29128-5-manivannan.sadhasivam@linaro.org
State Superseded
Headers show
Series Add support for secure regions in NAND | expand

Commit Message

Manivannan Sadhasivam April 2, 2021, 3:01 p.m. UTC 
Add missing nand_cleanup() in the alloc_bam_transaction() error path
to cleanup the resources properly.

Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
 drivers/mtd/nand/raw/qcom_nandc.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Miquel Raynal April 7, 2021, 9:06 a.m. UTC | #1 
On Fri, 2021-04-02 at 15:01:28 UTC, Manivannan Sadhasivam wrote:
> Add missing nand_cleanup() in the alloc_bam_transaction() error path

> to cleanup the resources properly.

> 

> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>


Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel
diff mbox series

Patch

diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c
index 87c23bb320bf..fe74cf3aece5 100644
--- a/drivers/mtd/nand/raw/qcom_nandc.c
+++ b/drivers/mtd/nand/raw/qcom_nandc.c
@@ -2882,6 +2882,7 @@  static int qcom_nand_host_init_and_register(struct qcom_nand_controller *nandc,
 		if (!nandc->bam_txn) {
 			dev_err(nandc->dev,
 				"failed to allocate bam transaction\n");
+			nand_cleanup(chip);
 			return -ENOMEM;
 		}
 	}