summaryrefslogtreecommitdiff
path: root/drivers/mmc/host/dw_mmc-pltfm.h
diff options
context:
space:
mode:
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>2023-05-06 01:15:06 +0300
committerUlf Hansson <ulf.hansson@linaro.org>2023-06-12 16:16:18 +0300
commit982fe2e093c62dbe144bd86c8b88cee65e5ad345 (patch)
tree5f0fdf74f48bf8b08c7fe00d1f6a6f425b626ba1 /drivers/mmc/host/dw_mmc-pltfm.h
parent2fba941d3423ee3084721ac7e64c042fefa9cdab (diff)
downloadlinux-982fe2e093c62dbe144bd86c8b88cee65e5ad345.tar.xz
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-pltfm.h')
-rw-r--r--drivers/mmc/host/dw_mmc-pltfm.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/dw_mmc-pltfm.h b/drivers/mmc/host/dw_mmc-pltfm.h
index 2d50d7da2e36..64cf7522a3d4 100644
--- a/drivers/mmc/host/dw_mmc-pltfm.h
+++ b/drivers/mmc/host/dw_mmc-pltfm.h
@@ -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_ */