spi: mt65xx: Properly handle failures in .remove()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 30 May 2023 08:16:46 +0000 (10:16 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 30 May 2023 12:46:56 +0000 (13:46 +0100)
commit22f407278ea43df46f90cece6595e5e8a0d5447c
treec121ad79cb47c721701663e51aa3df8194203caa
parentac9a78681b921877518763ba0e89202254349d1b
spi: mt65xx: Properly handle failures in .remove()

Returning an error code in a platform driver's remove function is wrong
most of the time and there is an effort to make the callback return
void. To prepare this rework the function not to exit early.

There wasn't a real problem because if pm runtime resume failed the only
step missing was pm_runtime_disable() which isn't an issue.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Link: https://lore.kernel.org/r/20230530081648.2199419-2-u.kleine-koenig@pengutronix.de
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-mt65xx.c