fbdev: xilinxfb: Make xilinxfb_release() return void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 19 Oct 2022 13:24:22 +0000 (15:24 +0200)
committerHelge Deller <deller@gmx.de>
Thu, 20 Oct 2022 06:36:41 +0000 (08:36 +0200)
The function xilinxfb_release() returns zero unconditionally. Make it
return void. There is no semantic change, the only effect is that it
becomes obvious that the driver's .remove() callback always returns zero.

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: Helge Deller <deller@gmx.de>
drivers/video/fbdev/xilinxfb.c

index 438e2c78142f92bada11725dea6570f8013fd921..1ac83900a21ccef301d2a0557549b610bbe46127 100644 (file)
@@ -376,7 +376,7 @@ err_cmap:
        return rc;
 }
 
-static int xilinxfb_release(struct device *dev)
+static void xilinxfb_release(struct device *dev)
 {
        struct xilinxfb_drvdata *drvdata = dev_get_drvdata(dev);
 
@@ -402,8 +402,6 @@ static int xilinxfb_release(struct device *dev)
        if (!(drvdata->flags & BUS_ACCESS_FLAG))
                dcr_unmap(drvdata->dcr_host, drvdata->dcr_len);
 #endif
-
-       return 0;
 }
 
 /* ---------------------------------------------------------------------
@@ -480,7 +478,9 @@ static int xilinxfb_of_probe(struct platform_device *pdev)
 
 static int xilinxfb_of_remove(struct platform_device *op)
 {
-       return xilinxfb_release(&op->dev);
+       xilinxfb_release(&op->dev);
+
+       return 0;
 }
 
 /* Match table for of_platform binding */