diff options
| author | Vignesh Raghavendra <vigneshr@ti.com> | 2024-07-31 20:28:55 +0530 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2024-08-27 14:27:44 -0600 |
| commit | 8a5642fa840308875ffff388dd87636f68725883 (patch) | |
| tree | 4c5a2d6f643dbcb3e2a8b3755ccd80eafec844e8 /drivers/mtd/nand/raw | |
| parent | f85dafa9affb280990bdb9ceaa0200a7f534f881 (diff) | |
mtd: nand: raw: omap_gpmc: Check return value of gpmc_nand_init
If the function is called with no NAND device attached, then this
function can return error value, proceeding further ignoring the same
can cause system crash. This is seen when "mtd list" is run with no NAND
addon cards connected.
Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
Acked-by: Roger Quadros <rogerq@kernel.org>
Diffstat (limited to 'drivers/mtd/nand/raw')
| -rw-r--r-- | drivers/mtd/nand/raw/omap_gpmc.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/mtd/nand/raw/omap_gpmc.c b/drivers/mtd/nand/raw/omap_gpmc.c index 92a92ad63a0..a36e2a148cc 100644 --- a/drivers/mtd/nand/raw/omap_gpmc.c +++ b/drivers/mtd/nand/raw/omap_gpmc.c @@ -1188,7 +1188,10 @@ static int gpmc_nand_probe(struct udevice *dev) return ret; base = devm_ioremap(dev, res.start, resource_size(&res)); - gpmc_nand_init(nand, base); + ret = gpmc_nand_init(nand, base); + if (ret) + return ret; + mtd->dev = dev; nand_set_flash_node(nand, dev_ofnode(dev)); |