diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2023-05-06 00:15:06 +0200 |
|---|---|---|
| committer | Ulf Hansson <ulf.hansson@linaro.org> | 2023-06-12 15:16:18 +0200 |
| commit | 982fe2e093c62dbe144bd86c8b88cee65e5ad345 (patch) | |
| tree | 5f0fdf74f48bf8b08c7fe00d1f6a6f425b626ba1 /drivers/mmc/host/dw_mmc-starfive.c | |
| parent | 2fba941d3423ee3084721ac7e64c042fefa9cdab (diff) | |
| download | linux-982fe2e093c62dbe144bd86c8b88cee65e5ad345.tar.gz linux-982fe2e093c62dbe144bd86c8b88cee65e5ad345.tar.bz2 linux-982fe2e093c62dbe144bd86c8b88cee65e5ad345.zip | |
mmc: dw_mmc: Make dw_mci_pltfm_remove() return void
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>
Diffstat (limited to 'drivers/mmc/host/dw_mmc-starfive.c')
| -rw-r--r-- | drivers/mmc/host/dw_mmc-starfive.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/dw_mmc-starfive.c b/drivers/mmc/host/dw_mmc-starfive.c index dab1508bf83c..fd05a648a8bb 100644 --- a/drivers/mmc/host/dw_mmc-starfive.c +++ b/drivers/mmc/host/dw_mmc-starfive.c @@ -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, |