mmc: dw_mmc: Make dw_mci_pltfm_remove() return void
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 5 May 2023 22:15:06 +0000 (00:15 +0200)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 12 Jun 2023 13:16:18 +0000 (15:16 +0200)
dw_mci_pltfm_remove() returned zero unconditionally. Make it return void
instead which makes its semantics a bit clearer.

Convert the drivers that use this function as .remove() callback to
.remove_new() which has the right prototype. This helps getting rid of
the platform_driver's remove callback that returns an int (which is
error prone). The other users didn't check the return value anyhow.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20230505221506.1247424-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/dw_mmc-bluefield.c
drivers/mmc/host/dw_mmc-k3.c
drivers/mmc/host/dw_mmc-pltfm.c
drivers/mmc/host/dw_mmc-pltfm.h
drivers/mmc/host/dw_mmc-starfive.c

index 10baf12..4747e56 100644 (file)
@@ -52,7 +52,7 @@ static int dw_mci_bluefield_probe(struct platform_device *pdev)
 
 static struct platform_driver dw_mci_bluefield_pltfm_driver = {
        .probe          = dw_mci_bluefield_probe,
-       .remove         = dw_mci_pltfm_remove,
+       .remove_new     = dw_mci_pltfm_remove,
        .driver         = {
                .name           = "dwmmc_bluefield",
                .probe_type     = PROBE_PREFER_ASYNCHRONOUS,
index 0311a37..e8ee7c4 100644 (file)
@@ -470,7 +470,7 @@ static const struct dev_pm_ops dw_mci_k3_dev_pm_ops = {
 
 static struct platform_driver dw_mci_k3_pltfm_driver = {
        .probe          = dw_mci_k3_probe,
-       .remove         = dw_mci_pltfm_remove,
+       .remove_new     = dw_mci_pltfm_remove,
        .driver         = {
                .name           = "dwmmc_k3",
                .probe_type     = PROBE_PREFER_ASYNCHRONOUS,
index 48b7da2..2353fad 100644 (file)
@@ -121,18 +121,17 @@ static int dw_mci_pltfm_probe(struct platform_device *pdev)
        return dw_mci_pltfm_register(pdev, drv_data);
 }
 
-int dw_mci_pltfm_remove(struct platform_device *pdev)
+void dw_mci_pltfm_remove(struct platform_device *pdev)
 {
        struct dw_mci *host = platform_get_drvdata(pdev);
 
        dw_mci_remove(host);
-       return 0;
 }
 EXPORT_SYMBOL_GPL(dw_mci_pltfm_remove);
 
 static struct platform_driver dw_mci_pltfm_driver = {
        .probe          = dw_mci_pltfm_probe,
-       .remove         = dw_mci_pltfm_remove,
+       .remove_new     = dw_mci_pltfm_remove,
        .driver         = {
                .name           = "dw_mmc",
                .probe_type     = PROBE_PREFER_ASYNCHRONOUS,
index 2d50d7d..64cf752 100644 (file)
@@ -10,7 +10,7 @@
 
 extern int dw_mci_pltfm_register(struct platform_device *pdev,
                                const struct dw_mci_drv_data *drv_data);
-extern int dw_mci_pltfm_remove(struct platform_device *pdev);
+extern void dw_mci_pltfm_remove(struct platform_device *pdev);
 extern const struct dev_pm_ops dw_mci_pltfm_pmops;
 
 #endif /* _DW_MMC_PLTFM_H_ */
index dab1508..fd05a64 100644 (file)
@@ -172,7 +172,7 @@ static int dw_mci_starfive_probe(struct platform_device *pdev)
 
 static struct platform_driver dw_mci_starfive_driver = {
        .probe = dw_mci_starfive_probe,
-       .remove = dw_mci_pltfm_remove,
+       .remove_new = dw_mci_pltfm_remove,
        .driver = {
                .name = "dwmmc_starfive",
                .probe_type = PROBE_PREFER_ASYNCHRONOUS,