A remove callback just returning 0 is equivalent to no remove callback
at all. So drop the useless function.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/soc/tegra/cbb/tegra234-cbb.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/soc/tegra/cbb/tegra234-cbb.c b/drivers/soc/tegra/cbb/tegra234-cbb.c
index 3528f9e15d5c..3b26dcf9a70c 100644
--- a/drivers/soc/tegra/cbb/tegra234-cbb.c
+++ b/drivers/soc/tegra/cbb/tegra234-cbb.c
@@ -1066,11 +1066,6 @@ static int tegra234_cbb_probe(struct platform_device *pdev)
return tegra_cbb_register(&cbb->base);
}
-static int tegra234_cbb_remove(struct platform_device *pdev)
-{
- return 0;
-}
-
static int __maybe_unused tegra234_cbb_resume_noirq(struct device *dev)
{
struct tegra234_cbb *cbb = dev_get_drvdata(dev);
@@ -1088,7 +1083,6 @@ static const struct dev_pm_ops tegra234_cbb_pm = {
static struct platform_driver tegra234_cbb_driver = {
.probe = tegra234_cbb_probe,
- .remove = tegra234_cbb_remove,
.driver = {
.name = "tegra234-cbb",
.of_match_table = tegra234_cbb_dt_ids,
--
2.38.1
On 13/12/22 03:55, Uwe Kleine-König wrote: > External email: Use caution opening links or attachments > > > A remove callback just returning 0 is equivalent to no remove callback > at all. So drop the useless function. > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Reviewed-by: Sumit Gupta <sumitg@nvidia.com> > --- > drivers/soc/tegra/cbb/tegra234-cbb.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/soc/tegra/cbb/tegra234-cbb.c b/drivers/soc/tegra/cbb/tegra234-cbb.c > index 3528f9e15d5c..3b26dcf9a70c 100644 > --- a/drivers/soc/tegra/cbb/tegra234-cbb.c > +++ b/drivers/soc/tegra/cbb/tegra234-cbb.c > @@ -1066,11 +1066,6 @@ static int tegra234_cbb_probe(struct platform_device *pdev) > return tegra_cbb_register(&cbb->base); > } > > -static int tegra234_cbb_remove(struct platform_device *pdev) > -{ > - return 0; > -} > - > static int __maybe_unused tegra234_cbb_resume_noirq(struct device *dev) > { > struct tegra234_cbb *cbb = dev_get_drvdata(dev); > @@ -1088,7 +1083,6 @@ static const struct dev_pm_ops tegra234_cbb_pm = { > > static struct platform_driver tegra234_cbb_driver = { > .probe = tegra234_cbb_probe, > - .remove = tegra234_cbb_remove, > .driver = { > .name = "tegra234-cbb", > .of_match_table = tegra234_cbb_dt_ids, > -- > 2.38.1 >
Hello, On Mon, Dec 12, 2022 at 11:25:49PM +0100, Uwe Kleine-König wrote: > A remove callback just returning 0 is equivalent to no remove callback > at all. So drop the useless function. the other two patches in this series were applied individually to their matching trees. For this one I didn't get feedback yet. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
Hello, [adding Sumit Gupta to Cc who authored all commits to this driver so far, context available at https://lore.kernel.org/r/20221212222549.3779846-4-u.kleine-koenig@pengutronix.de] On Thu, Jan 12, 2023 at 10:55:11PM +0100, Uwe Kleine-König wrote: > On Mon, Dec 12, 2022 at 11:25:49PM +0100, Uwe Kleine-König wrote: > > A remove callback just returning 0 is equivalent to no remove callback > > at all. So drop the useless function. > > the other two patches in this series were applied individually to their > matching trees. For this one I didn't get feedback yet. Did this patch fell through the cracks? Is it just -ENOMAINTAINERTIME? Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
From: Thierry Reding <treding@nvidia.com> On Mon, 12 Dec 2022 23:25:49 +0100, Uwe Kleine-König wrote: > A remove callback just returning 0 is equivalent to no remove callback > at all. So drop the useless function. > > Applied, thanks! [3/3] soc: tegra: cbb: Drop empty platform remove function commit: c299a2e6bf944b4218acc194a1cdf500b34e80aa Best regards, -- Thierry Reding <treding@nvidia.com>
© 2016 - 2024 Red Hat, Inc.