mfd: cs42l43: Use devres for remove as well

Currently the device is powered down in the remove callback, however
all other clean up is done through devres. The problem here is the
MFD children are cleaned up through devres. As this happens after
the remove callback has run, this leads to the incorrect ordering
where the child remove functions run after the device has been powered
down. Put the power down into devres as well such that everything runs
in the expected order.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20241205115822.2371719-4-ckeepax@opensource.cirrus.com
Signed-off-by: Lee Jones <lee@kernel.org>

diff --git a/drivers/mfd/cs42l43-i2c.c b/drivers/mfd/cs42l43-i2c.c
index c9e4ea76149a8880dde2212b0790958e3ecb89a1..1e6422cdf01234cb99e89ffea3f4772944d0af72 100644 (file)
--- a/drivers/mfd/cs42l43-i2c.c
+++ b/drivers/mfd/cs42l43-i2c.c
@@ -56,13 +56,6 @@ static int cs42l43_i2c_probe(struct i2c_client *i2c)
        return cs42l43_dev_probe(cs42l43);
 }
 
-static void cs42l43_i2c_remove(struct i2c_client *i2c)
-{
-       struct cs42l43 *cs42l43 = dev_get_drvdata(&i2c->dev);
-
-       cs42l43_dev_remove(cs42l43);
-}
-
 #if IS_ENABLED(CONFIG_OF)
 static const struct of_device_id cs42l43_of_match[] = {
        { .compatible = "cirrus,cs42l43", },
@@ -88,7 +81,6 @@ static struct i2c_driver cs42l43_i2c_driver = {
        },
 
        .probe          = cs42l43_i2c_probe,
-       .remove         = cs42l43_i2c_remove,
 };
 module_i2c_driver(cs42l43_i2c_driver);
 

diff --git a/drivers/mfd/cs42l43-sdw.c b/drivers/mfd/cs42l43-sdw.c
index 65f7b1d7824861041899c0f6bddda4776edc8cf0..6af8465b2099da78bf970d4ddd9bf3d8127c8175 100644 (file)
--- a/drivers/mfd/cs42l43-sdw.c
+++ b/drivers/mfd/cs42l43-sdw.c
@@ -187,15 +187,6 @@ static int cs42l43_sdw_probe(struct sdw_slave *sdw, const struct sdw_device_id *
        return cs42l43_dev_probe(cs42l43);
 }
 
-static int cs42l43_sdw_remove(struct sdw_slave *sdw)
-{
-       struct cs42l43 *cs42l43 = dev_get_drvdata(&sdw->dev);
-
-       cs42l43_dev_remove(cs42l43);
-
-       return 0;
-}
-
 static const struct sdw_device_id cs42l43_sdw_id[] = {
        SDW_SLAVE_ENTRY(0x01FA, 0x4243, 0),
        {}
@@ -209,7 +200,6 @@ static struct sdw_driver cs42l43_sdw_driver = {
        },
 
        .probe          = cs42l43_sdw_probe,
-       .remove         = cs42l43_sdw_remove,
        .id_table       = cs42l43_sdw_id,
        .ops            = &cs42l43_sdw_ops,
 };

diff --git a/drivers/mfd/cs42l43.c b/drivers/mfd/cs42l43.c
index 9572c7fd419a74ebc5329fd6295f7c70d948062b..beb63c4efd2153c6acef8021717a1c69e37b0f1a 100644 (file)
--- a/drivers/mfd/cs42l43.c
+++ b/drivers/mfd/cs42l43.c
@@ -1038,6 +1038,15 @@ static int cs42l43_power_down(struct cs42l43 *cs42l43)
        return 0;
 }
 
+static void cs42l43_dev_remove(void *data)
+{
+       struct cs42l43 *cs42l43 = data;
+
+       cancel_work_sync(&cs42l43->boot_work);
+
+       cs42l43_power_down(cs42l43);
+}
+
 int cs42l43_dev_probe(struct cs42l43 *cs42l43)
 {
        int i, ret;
@@ -1084,6 +1093,10 @@ int cs42l43_dev_probe(struct cs42l43 *cs42l43)
        if (ret)
                return ret;
 
+       ret = devm_add_action_or_reset(cs42l43->dev, cs42l43_dev_remove, cs42l43);
+       if (ret)
+               return ret;
+
        pm_runtime_set_autosuspend_delay(cs42l43->dev, CS42L43_AUTOSUSPEND_TIME_MS);
        pm_runtime_use_autosuspend(cs42l43->dev);
        pm_runtime_set_active(cs42l43->dev);
@@ -1102,14 +1115,6 @@ int cs42l43_dev_probe(struct cs42l43 *cs42l43)
 }
 EXPORT_SYMBOL_NS_GPL(cs42l43_dev_probe, MFD_CS42L43);
 
-void cs42l43_dev_remove(struct cs42l43 *cs42l43)
-{
-       cancel_work_sync(&cs42l43->boot_work);
-
-       cs42l43_power_down(cs42l43);
-}
-EXPORT_SYMBOL_NS_GPL(cs42l43_dev_remove, MFD_CS42L43);
-
 static int cs42l43_suspend(struct device *dev)
 {
        struct cs42l43 *cs42l43 = dev_get_drvdata(dev);

diff --git a/drivers/mfd/cs42l43.h b/drivers/mfd/cs42l43.h
index 8d1b1b0f5a4732e15dc8c8c4b22f0c300ca202c6..f3da783930f53404d7bc7731eb03f582a9ee8710 100644 (file)
--- a/drivers/mfd/cs42l43.h
+++ b/drivers/mfd/cs42l43.h
@@ -25,6 +25,5 @@ bool cs42l43_precious_register(struct device *dev, unsigned int reg);
 bool cs42l43_volatile_register(struct device *dev, unsigned int reg);
 
 int cs42l43_dev_probe(struct cs42l43 *cs42l43);
-void cs42l43_dev_remove(struct cs42l43 *cs42l43);
 
 #endif /* CS42L43_CORE_INT_H */