mtd: rawnand: omap2: Suppress error message after WARN in .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 3 Jun 2022 21:07:53 +0000 (23:07 +0200)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Thu, 9 Jun 2022 13:06:21 +0000 (15:06 +0200)
Returning an error value in a platform remove callback results in an error
message being emitted by the platform core, but otherwise it doesn't make
a difference. After the WARN splat this generic error message doesn't add
any value, so return 0 unconditionally

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20220603210758.148493-10-u.kleine-koenig@pengutronix.de
drivers/mtd/nand/raw/omap2.c

index 58c32a1..4a9f2b6 100644 (file)
@@ -2278,16 +2278,14 @@ static int omap_nand_remove(struct platform_device *pdev)
        struct mtd_info *mtd = platform_get_drvdata(pdev);
        struct nand_chip *nand_chip = mtd_to_nand(mtd);
        struct omap_nand_info *info = mtd_to_omap(mtd);
-       int ret;
 
        rawnand_sw_bch_cleanup(nand_chip);
 
        if (info->dma)
                dma_release_channel(info->dma);
-       ret = mtd_device_unregister(mtd);
-       WARN_ON(ret);
+       WARN_ON(mtd_device_unregister(mtd));
        nand_cleanup(nand_chip);
-       return ret;
+       return 0;
 }
 
 /* omap_nand_ids defined in linux/platform_data/mtd-nand-omap2.h */