]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
fbdev: amifb: Convert to platform remove callback returning void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thu, 9 Nov 2023 22:01:54 +0000 (23:01 +0100)
committerHelge Deller <deller@gmx.de>
Fri, 10 Nov 2023 08:16:02 +0000 (09:16 +0100)
commitdce217780270300db7931d0fd73796aa64fea237
tree7f8ca1b3957b1a9e206bd899800a9dd8a9a0bc91
parent67e1ab5bb58a787809772eba14e7e758c765e2fd
fbdev: amifb: 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 ignored (apart
from emitting a warning) 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. Eventually after all drivers
are converted, .remove_new() will be renamed to .remove().

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>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Helge Deller <deller@gmx.de>
drivers/video/fbdev/amifb.c