[PATCH] mtd: nand: gpmi: Release DMA channels on failure
Fabio Estevam
festevam at gmail.com
Thu Dec 28 10:07:00 PST 2017
From: Fabio Estevam <fabio.estevam at nxp.com>
release_dma_channels() should be called in the case of error in
gpmi_init() or bch_set_geometry(), because acquire_dma_channels() has
been called previously.
Handle the error cases correctly.
Signed-off-by: Fabio Estevam <fabio.estevam at nxp.com>
---
drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
index 2ef8979..acfa5f5 100644
--- a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
+++ b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
@@ -2137,20 +2137,25 @@ static int gpmi_pm_resume(struct device *dev)
ret = gpmi_init(this);
if (ret) {
dev_err(this->dev, "Error setting GPMI : %d\n", ret);
- return ret;
+ goto release_dma_channels;
}
/* re-init the BCH registers */
ret = bch_set_geometry(this);
if (ret) {
dev_err(this->dev, "Error setting BCH : %d\n", ret);
- return ret;
+ goto release_dma_channels;
}
/* re-init others */
gpmi_extra_init(this);
return 0;
+
+release_dma_channels:
+ release_dma_channels(this);
+ return ret;
+
}
#endif /* CONFIG_PM_SLEEP */
--
2.7.4
More information about the linux-mtd
mailing list