Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

mtd: powernv_flash: Warn about failure to unregister mtd device

mtd_device_unregister() shouldn't fail. Wail loudly if it does anyhow.

This matches how other drivers (e.g. nand/raw/nandsim.c) use
mtd_device_unregister().

By returning 0 in the platform remove callback a generic error message
by the device core is suppressed, nothing else changes.

This is a preparation for making platform remove callbacks return void.

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-4-u.kleine-koenig@pengutronix.de

authored by

Uwe Kleine-König and committed by
Miquel Raynal
90914b4e 0c90466a

+3 -1
+3 -1
drivers/mtd/devices/powernv_flash.c
··· 270 270 struct powernv_flash *data = dev_get_drvdata(&(pdev->dev)); 271 271 272 272 /* All resources should be freed automatically */ 273 - return mtd_device_unregister(&(data->mtd)); 273 + WARN_ON(mtd_device_unregister(&data->mtd)); 274 + 275 + return 0; 274 276 } 275 277 276 278 static const struct of_device_id powernv_flash_match[] = {