clk: s2mps11: Convert to platform remove callback returning void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Sun, 12 Mar 2023 16:14:56 +0000 (17:14 +0100)
committerStephen Boyd <sboyd@kernel.org>
Wed, 29 Mar 2023 02:23:37 +0000 (19:23 -0700)
commit34014ff811c6e6423a0215e90fb5a02cf80d4e3c
tree3e35ae3b4a81689bfdebc5c8e7b1b70ffa821d99
parente72cdad50cdd1401525b37ac30519361d73fd139
clk: s2mps11: Convert to platform remove callback returning void

The .remove() callback for a platform driver returns an int which makes
many driver authors wrongly assume it's possible to do error handling by
returning an error code. However the value returned is (mostly) ignored
and this typically results in resource leaks. To improve here there is a
quest to make the remove callback return void. In the first step of this
quest all drivers are converted to .remove_new() which already returns
void.

Trivially convert this driver from always returning zero in the remove
callback to the void returning variant.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20230312161512.2715500-15-u.kleine-koenig@pengutronix.de
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk-s2mps11.c