projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
88ffef1
)
mtd: rawnand: ams-delta: Stop using nand_release()
author
Miquel Raynal
<miquel.raynal@bootlin.com>
Tue, 19 May 2020 12:59:34 +0000
(14:59 +0200)
committer
Miquel Raynal
<miquel.raynal@bootlin.com>
Tue, 26 May 2020 19:37:51 +0000
(21:37 +0200)
This helper is not very useful and very often people get confused:
they use nand_release() instead of nand_cleanup().
Let's stop using nand_release() by calling mtd_device_unregister() and
nand_cleanup() directly.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link:
https://lore.kernel.org/linux-mtd/20200519130035.1883-2-miquel.raynal@bootlin.com
drivers/mtd/nand/raw/ams-delta.c
patch
|
blob
|
history
diff --git
a/drivers/mtd/nand/raw/ams-delta.c
b/drivers/mtd/nand/raw/ams-delta.c
index
d66dab2
..
3711e7a
100644
(file)
--- a/
drivers/mtd/nand/raw/ams-delta.c
+++ b/
drivers/mtd/nand/raw/ams-delta.c
@@
-387,12
+387,15
@@
static int gpio_nand_remove(struct platform_device *pdev)
{
struct gpio_nand *priv = platform_get_drvdata(pdev);
struct mtd_info *mtd = nand_to_mtd(&priv->nand_chip);
+ int ret;
/* Apply write protection */
gpiod_set_value(priv->gpiod_nwp, 1);
/* Unregister device */
- nand_release(mtd_to_nand(mtd));
+ ret = mtd_device_unregister(mtd);
+ WARN_ON(ret);
+ nand_cleanup(mtd_to_nand(mtd));
return 0;
}