diff options
author | Miquel Raynal <miquel.raynal@bootlin.com> | 2018-03-21 09:01:42 -0400 |
---|---|---|
committer | Boris Brezillon <boris.brezillon@bootlin.com> | 2018-03-29 03:38:18 -0400 |
commit | 5826b8802a970d2b0aaf097951de42040ee32480 (patch) | |
tree | 1e72dca6c76548a953d1c253e92d5e4b031835cf | |
parent | 15d6f118285f2ffc73fc8de75ecf7c36c3c6fe18 (diff) |
mtd: rawnand: brcmnand: fix probe function error path
An error after nand_scan_tail() should trigger a nand_cleanup().
The helper mtd_device_register() returns an error code that should
be checked and nand_cleanup() called accordingly.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
-rw-r--r-- | drivers/mtd/nand/raw/brcmnand/brcmnand.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c b/drivers/mtd/nand/raw/brcmnand/brcmnand.c index c28fd2bc1a84..1306aaa7a8bf 100644 --- a/drivers/mtd/nand/raw/brcmnand/brcmnand.c +++ b/drivers/mtd/nand/raw/brcmnand/brcmnand.c | |||
@@ -2297,7 +2297,11 @@ static int brcmnand_init_cs(struct brcmnand_host *host, struct device_node *dn) | |||
2297 | if (ret) | 2297 | if (ret) |
2298 | return ret; | 2298 | return ret; |
2299 | 2299 | ||
2300 | return mtd_device_register(mtd, NULL, 0); | 2300 | ret = mtd_device_register(mtd, NULL, 0); |
2301 | if (ret) | ||
2302 | nand_cleanup(chip); | ||
2303 | |||
2304 | return ret; | ||
2301 | } | 2305 | } |
2302 | 2306 | ||
2303 | static void brcmnand_save_restore_cs_config(struct brcmnand_host *host, | 2307 | static void brcmnand_save_restore_cs_config(struct brcmnand_host *host, |