iio: fix devm_iio_trigger_alloc with parent.cocci

Use cocci semantic patch:
@@
expression trigger, P;
@@
   trigger = devm_iio_trigger_alloc(P, ...);
   ...
-  trigger->dev.parent = P;

To remove trigger->dev.parent, since it is set by default.

Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20210309193620.2176163-3-gwendal@chromium.org
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/accel/adxl372.c b/drivers/iio/accel/adxl372.c
index 8ba1453b8dbf4432b0a3223285cdb3eb317c4acf..9c9a896a872a047cecf119c4263cf6c4359906d4 100644 (file)
--- a/drivers/iio/accel/adxl372.c
+++ b/drivers/iio/accel/adxl372.c
@@ -1236,8 +1236,6 @@ int adxl372_probe(struct device *dev, struct regmap *regmap,
 
                st->dready_trig->ops = &adxl372_trigger_ops;
                st->peak_datardy_trig->ops = &adxl372_peak_data_trigger_ops;
-               st->dready_trig->dev.parent = dev;
-               st->peak_datardy_trig->dev.parent = dev;
                iio_trigger_set_drvdata(st->dready_trig, indio_dev);
                iio_trigger_set_drvdata(st->peak_datardy_trig, indio_dev);
                ret = devm_iio_trigger_register(dev, st->dready_trig);

diff --git a/drivers/iio/accel/bmc150-accel-core.c b/drivers/iio/accel/bmc150-accel-core.c
index b0dbd12cbf4236a1c44c3665298249e858bff02c..04d85ce34e9f5703037e67f25b6f5bb3a197467c 100644 (file)
--- a/drivers/iio/accel/bmc150-accel-core.c
+++ b/drivers/iio/accel/bmc150-accel-core.c
@@ -1478,7 +1478,6 @@ static int bmc150_accel_triggers_setup(struct iio_dev *indio_dev,
                        break;
                }
 
-               t->indio_trig->dev.parent = dev;
                t->indio_trig->ops = &bmc150_accel_trigger_ops;
                t->intr = bmc150_accel_triggers[i].intr;
                t->data = data;

diff --git a/drivers/iio/accel/kxcjk-1013.c b/drivers/iio/accel/kxcjk-1013.c
index 5c056165f473f51e25960ea6eb0b67355c2a900a..ff724bc17a458a96a7e0743337645421d57b832f 100644 (file)
--- a/drivers/iio/accel/kxcjk-1013.c
+++ b/drivers/iio/accel/kxcjk-1013.c
@@ -1419,7 +1419,6 @@ static int kxcjk1013_probe(struct i2c_client *client,
                        goto err_poweroff;
                }
 
-               data->dready_trig->dev.parent = &client->dev;
                data->dready_trig->ops = &kxcjk1013_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                indio_dev->trig = data->dready_trig;
@@ -1428,7 +1427,6 @@ static int kxcjk1013_probe(struct i2c_client *client,
                if (ret)
                        goto err_poweroff;
 
-               data->motion_trig->dev.parent = &client->dev;
                data->motion_trig->ops = &kxcjk1013_trigger_ops;
                iio_trigger_set_drvdata(data->motion_trig, indio_dev);
                ret = iio_trigger_register(data->motion_trig);

diff --git a/drivers/iio/accel/mma8452.c b/drivers/iio/accel/mma8452.c
index 33f0c419d8ff4c949383e673bd456648f93c8fcc..4d307dfb9169c502e972c2d83ec6626a25e501d0 100644 (file)
--- a/drivers/iio/accel/mma8452.c
+++ b/drivers/iio/accel/mma8452.c
@@ -1465,7 +1465,6 @@ static int mma8452_trigger_setup(struct iio_dev *indio_dev)
        if (!trig)
                return -ENOMEM;
 
-       trig->dev.parent = &data->client->dev;
        trig->ops = &mma8452_trigger_ops;
        iio_trigger_set_drvdata(trig, indio_dev);
 

diff --git a/drivers/iio/accel/mxc4005.c b/drivers/iio/accel/mxc4005.c
index 0f8fd687866d4d0e086a37bb1ce20a1561f7a3e3..fb3cbaa62bd879ace716fb159b375f66f5e9ae20 100644 (file)
--- a/drivers/iio/accel/mxc4005.c
+++ b/drivers/iio/accel/mxc4005.c
@@ -450,7 +450,6 @@ static int mxc4005_probe(struct i2c_client *client,
                        return ret;
                }
 
-               data->dready_trig->dev.parent = &client->dev;
                data->dready_trig->ops = &mxc4005_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                indio_dev->trig = data->dready_trig;

diff --git a/drivers/iio/accel/stk8312.c b/drivers/iio/accel/stk8312.c
index 1a8a1a0c3ffb4319909e68eae5235761237772ba..157d8faefb9e4c19d05c1c028f70c104dc2e8a3a 100644 (file)
--- a/drivers/iio/accel/stk8312.c
+++ b/drivers/iio/accel/stk8312.c
@@ -558,7 +558,6 @@ static int stk8312_probe(struct i2c_client *client,
                        goto err_power_off;
                }
 
-               data->dready_trig->dev.parent = &client->dev;
                data->dready_trig->ops = &stk8312_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                ret = iio_trigger_register(data->dready_trig);

diff --git a/drivers/iio/accel/stk8ba50.c b/drivers/iio/accel/stk8ba50.c
index 7a06904455eb2f9ff1db3a8973d326bd2034bef4..7cf9cb7e86667187d2247c9a2844b6fd08afff8e 100644 (file)
--- a/drivers/iio/accel/stk8ba50.c
+++ b/drivers/iio/accel/stk8ba50.c
@@ -454,7 +454,6 @@ static int stk8ba50_probe(struct i2c_client *client,
                        goto err_power_off;
                }
 
-               data->dready_trig->dev.parent = &client->dev;
                data->dready_trig->ops = &stk8ba50_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                ret = iio_trigger_register(data->dready_trig);

diff --git a/drivers/iio/adc/ad7606.c b/drivers/iio/adc/ad7606.c
index ee7b108688b33537095874e988c07905c63f56a2..0af0bb4d5a7fed86dde0472b367a45c0e040839e 100644 (file)
--- a/drivers/iio/adc/ad7606.c
+++ b/drivers/iio/adc/ad7606.c
@@ -668,7 +668,6 @@ int ad7606_probe(struct device *dev, int irq, void __iomem *base_address,
                return -ENOMEM;
 
        st->trig->ops = &ad7606_trigger_ops;
-       st->trig->dev.parent = dev;
        iio_trigger_set_drvdata(st->trig, indio_dev);
        ret = devm_iio_trigger_register(dev, st->trig);
        if (ret)

diff --git a/drivers/iio/adc/ad7766.c b/drivers/iio/adc/ad7766.c
index b6b6765be7b49b25e9bea6603f73dbc62866fd9b..829a3426f2351b6e451f4a00dff7849abd0f8e19 100644 (file)
--- a/drivers/iio/adc/ad7766.c
+++ b/drivers/iio/adc/ad7766.c
@@ -253,7 +253,6 @@ static int ad7766_probe(struct spi_device *spi)
                        return -ENOMEM;
 
                ad7766->trig->ops = &ad7766_trigger_ops;
-               ad7766->trig->dev.parent = &spi->dev;
                iio_trigger_set_drvdata(ad7766->trig, ad7766);
 
                ret = devm_request_irq(&spi->dev, spi->irq, ad7766_irq,

diff --git a/drivers/iio/adc/ad7768-1.c b/drivers/iio/adc/ad7768-1.c
index 5c0cbee032308376c395c3996bfea65eb8944e64..c945f1349623f7ad7f58069710ed02565f022af5 100644 (file)
--- a/drivers/iio/adc/ad7768-1.c
+++ b/drivers/iio/adc/ad7768-1.c
@@ -631,7 +631,6 @@ static int ad7768_probe(struct spi_device *spi)
                return -ENOMEM;
 
        st->trig->ops = &ad7768_trigger_ops;
-       st->trig->dev.parent = &spi->dev;
        iio_trigger_set_drvdata(st->trig, indio_dev);
        ret = devm_iio_trigger_register(&spi->dev, st->trig);
        if (ret)

diff --git a/drivers/iio/adc/max1027.c b/drivers/iio/adc/max1027.c
index ca1dff3924ff9c8d93802297f299c280a38d922c..e3c8ec107722da41816f01c9add38e9c84d1f346 100644 (file)
--- a/drivers/iio/adc/max1027.c
+++ b/drivers/iio/adc/max1027.c
@@ -473,7 +473,6 @@ static int max1027_probe(struct spi_device *spi)
                }
 
                st->trig->ops = &max1027_trigger_ops;
-               st->trig->dev.parent = &spi->dev;
                iio_trigger_set_drvdata(st->trig, indio_dev);
                ret = devm_iio_trigger_register(&indio_dev->dev,
                                                st->trig);

diff --git a/drivers/iio/chemical/ccs811.c b/drivers/iio/chemical/ccs811.c
index 60dd87e96f5f8b0049f9b3af4e7be6e82b3b0ab8..886e96496dbf7eeb79c2fb6147f276883ea6bdff 100644 (file)
--- a/drivers/iio/chemical/ccs811.c
+++ b/drivers/iio/chemical/ccs811.c
@@ -497,7 +497,6 @@ static int ccs811_probe(struct i2c_client *client,
                        goto err_poweroff;
                }
 
-               data->drdy_trig->dev.parent = &client->dev;
                data->drdy_trig->ops = &ccs811_trigger_ops;
                iio_trigger_set_drvdata(data->drdy_trig, indio_dev);
                indio_dev->trig = data->drdy_trig;

diff --git a/drivers/iio/chemical/scd30_core.c b/drivers/iio/chemical/scd30_core.c
index 4d0d798c7cd347d97d25e6a12347b12d7896ff00..261c277ac4a5964dad3cd6a0cb3801fe4ab7a2ef 100644 (file)
--- a/drivers/iio/chemical/scd30_core.c
+++ b/drivers/iio/chemical/scd30_core.c
@@ -646,7 +646,6 @@ static int scd30_setup_trigger(struct iio_dev *indio_dev)
                return -ENOMEM;
        }
 
-       trig->dev.parent = dev;
        trig->ops = &scd30_trigger_ops;
        iio_trigger_set_drvdata(trig, indio_dev);
 

diff --git a/drivers/iio/gyro/adxrs290.c b/drivers/iio/gyro/adxrs290.c
index c45d8226cc2b165da09e345c3bc3d57d30aa1098..cec5e1f17c22a655671eec831ad1e52f9d078741 100644 (file)
--- a/drivers/iio/gyro/adxrs290.c
+++ b/drivers/iio/gyro/adxrs290.c
@@ -593,7 +593,6 @@ static int adxrs290_probe_trigger(struct iio_dev *indio_dev)
        if (!st->dready_trig)
                return -ENOMEM;
 
-       st->dready_trig->dev.parent = &st->spi->dev;
        st->dready_trig->ops = &adxrs290_trigger_ops;
        iio_trigger_set_drvdata(st->dready_trig, indio_dev);
 

diff --git a/drivers/iio/gyro/bmg160_core.c b/drivers/iio/gyro/bmg160_core.c
index 029ef4c34604662e64385e856882a3acc58697f7..b11ebd9bb7a41839e44f88083036de9cffac07bf 100644 (file)
--- a/drivers/iio/gyro/bmg160_core.c
+++ b/drivers/iio/gyro/bmg160_core.c
@@ -1148,14 +1148,12 @@ int bmg160_core_probe(struct device *dev, struct regmap *regmap, int irq,
                if (!data->motion_trig)
                        return -ENOMEM;
 
-               data->dready_trig->dev.parent = dev;
                data->dready_trig->ops = &bmg160_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                ret = iio_trigger_register(data->dready_trig);
                if (ret)
                        return ret;
 
-               data->motion_trig->dev.parent = dev;
                data->motion_trig->ops = &bmg160_trigger_ops;
                iio_trigger_set_drvdata(data->motion_trig, indio_dev);
                ret = iio_trigger_register(data->motion_trig);

diff --git a/drivers/iio/gyro/fxas21002c_core.c b/drivers/iio/gyro/fxas21002c_core.c
index 129eead8febc0ea1a3a5b161296c822d355c7106..1a20c6b88e7db5cf3b28665ab25ab17c9c270882 100644 (file)
--- a/drivers/iio/gyro/fxas21002c_core.c
+++ b/drivers/iio/gyro/fxas21002c_core.c
@@ -875,7 +875,6 @@ static int fxas21002c_trigger_probe(struct fxas21002c_data *data)
        if (ret < 0)
                return ret;
 
-       data->dready_trig->dev.parent = dev;
        data->dready_trig->ops = &fxas21002c_trigger_ops;
        iio_trigger_set_drvdata(data->dready_trig, indio_dev);
 

diff --git a/drivers/iio/health/afe4403.c b/drivers/iio/health/afe4403.c
index 38734e4ce360570215f352a69ce0e9a3bf9b6443..1fa8d51d50806d3559c3b618101cd19514b42d99 100644 (file)
--- a/drivers/iio/health/afe4403.c
+++ b/drivers/iio/health/afe4403.c
@@ -531,7 +531,6 @@ static int afe4403_probe(struct spi_device *spi)
                iio_trigger_set_drvdata(afe->trig, indio_dev);
 
                afe->trig->ops = &afe4403_trigger_ops;
-               afe->trig->dev.parent = afe->dev;
 
                ret = iio_trigger_register(afe->trig);
                if (ret) {

diff --git a/drivers/iio/health/afe4404.c b/drivers/iio/health/afe4404.c
index 61fe4932d81d02269a7b5f776a037da83724b952..e1476bf79fe2a22a00602bd65f5e5696ddc3bcc7 100644 (file)
--- a/drivers/iio/health/afe4404.c
+++ b/drivers/iio/health/afe4404.c
@@ -538,7 +538,6 @@ static int afe4404_probe(struct i2c_client *client,
                iio_trigger_set_drvdata(afe->trig, indio_dev);
 
                afe->trig->ops = &afe4404_trigger_ops;
-               afe->trig->dev.parent = afe->dev;
 
                ret = iio_trigger_register(afe->trig);
                if (ret) {

diff --git a/drivers/iio/humidity/hts221_buffer.c b/drivers/iio/humidity/hts221_buffer.c
index 95e56917677faea5ac20410418653962ad329659..f29692b9d2db0e39704991dcdcff07bae184fb25 100644 (file)
--- a/drivers/iio/humidity/hts221_buffer.c
+++ b/drivers/iio/humidity/hts221_buffer.c
@@ -135,7 +135,6 @@ int hts221_allocate_trigger(struct iio_dev *iio_dev)
 
        iio_trigger_set_drvdata(hw->trig, iio_dev);
        hw->trig->ops = &hts221_trigger_ops;
-       hw->trig->dev.parent = hw->dev;
        iio_dev->trig = iio_trigger_get(hw->trig);
 
        return devm_iio_trigger_register(hw->dev, hw->trig);

diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
index 4377047d503abe7c2d7df0c35c02606a0c7e283d..fc5a60fcfec054a4eb539517a3291ff4f9530a5d 100644 (file)
--- a/drivers/iio/imu/kmx61.c
+++ b/drivers/iio/imu/kmx61.c
@@ -1268,7 +1268,6 @@ static struct iio_trigger *kmx61_trigger_setup(struct kmx61_data *data,
        if (!trig)
                return ERR_PTR(-ENOMEM);
 
-       trig->dev.parent = &data->client->dev;
        trig->ops = &kmx61_trigger_ops;
        iio_trigger_set_drvdata(trig, indio_dev);
 

diff --git a/drivers/iio/light/rpr0521.c b/drivers/iio/light/rpr0521.c
index 31224a33bade378efc39dc9622f34a7229b9de38..033578f444e4a6f23b924a5fc5fc131a0f13100a 100644 (file)
--- a/drivers/iio/light/rpr0521.c
+++ b/drivers/iio/light/rpr0521.c
@@ -990,7 +990,6 @@ static int rpr0521_probe(struct i2c_client *client,
                        ret = -ENOMEM;
                        goto err_pm_disable;
                }
-               data->drdy_trigger0->dev.parent = indio_dev->dev.parent;
                data->drdy_trigger0->ops = &rpr0521_trigger_ops;
                indio_dev->available_scan_masks = rpr0521_available_scan_masks;
                iio_trigger_set_drvdata(data->drdy_trigger0, indio_dev);

diff --git a/drivers/iio/light/si1145.c b/drivers/iio/light/si1145.c
index b304801c791634914f24918cdc0fe78bd3ac36a3..9b5c99823943a1f050dcdfd507c8c392c82a077e 100644 (file)
--- a/drivers/iio/light/si1145.c
+++ b/drivers/iio/light/si1145.c
@@ -1247,7 +1247,6 @@ static int si1145_probe_trigger(struct iio_dev *indio_dev)
        if (!trig)
                return -ENOMEM;
 
-       trig->dev.parent = &client->dev;
        trig->ops = &si1145_trigger_ops;
        iio_trigger_set_drvdata(trig, indio_dev);
 

diff --git a/drivers/iio/light/st_uvis25_core.c b/drivers/iio/light/st_uvis25_core.c
index 1055594b22764e5ec7f7df5d0a80163a0150d300..41a2ce5a2d53882c807bde64a67d238aa3ce64e2 100644 (file)
--- a/drivers/iio/light/st_uvis25_core.c
+++ b/drivers/iio/light/st_uvis25_core.c
@@ -210,7 +210,6 @@ static int st_uvis25_allocate_trigger(struct iio_dev *iio_dev)
                return -ENOMEM;
 
        iio_trigger_set_drvdata(hw->trig, iio_dev);
-       hw->trig->dev.parent = dev;
 
        return devm_iio_trigger_register(dev, hw->trig);
 }

diff --git a/drivers/iio/light/vcnl4000.c b/drivers/iio/light/vcnl4000.c
index fff4b36b8b58dc04cc197b52a846d9737a95648a..2f7916f95689e873b21bad598138a2c5257deda4 100644 (file)
--- a/drivers/iio/light/vcnl4000.c
+++ b/drivers/iio/light/vcnl4000.c
@@ -1002,7 +1002,6 @@ static int vcnl4010_probe_trigger(struct iio_dev *indio_dev)
        if (!trigger)
                return -ENOMEM;
 
-       trigger->dev.parent = &client->dev;
        trigger->ops = &vcnl4010_trigger_ops;
        iio_trigger_set_drvdata(trigger, indio_dev);
 

diff --git a/drivers/iio/light/vcnl4035.c b/drivers/iio/light/vcnl4035.c
index 73a28e30dddccd7b26219e0017467406a75ecc46..ae87740d9cef21a4f5f51821c802d8293367a262 100644 (file)
--- a/drivers/iio/light/vcnl4035.c
+++ b/drivers/iio/light/vcnl4035.c
@@ -511,7 +511,6 @@ static int vcnl4035_probe_trigger(struct iio_dev *indio_dev)
        if (!data->drdy_trigger0)
                return -ENOMEM;
 
-       data->drdy_trigger0->dev.parent = indio_dev->dev.parent;
        data->drdy_trigger0->ops = &vcnl4035_trigger_ops;
        iio_trigger_set_drvdata(data->drdy_trigger0, indio_dev);
        ret = devm_iio_trigger_register(indio_dev->dev.parent,

diff --git a/drivers/iio/magnetometer/bmc150_magn.c b/drivers/iio/magnetometer/bmc150_magn.c
index b2f3129e1b4f3cefe371d487acc93c805b65fa16..00f9766bad5c54bd3b3de8603054356958a68e5d 100644 (file)
--- a/drivers/iio/magnetometer/bmc150_magn.c
+++ b/drivers/iio/magnetometer/bmc150_magn.c
@@ -922,7 +922,6 @@ int bmc150_magn_probe(struct device *dev, struct regmap *regmap,
                        goto err_poweroff;
                }
 
-               data->dready_trig->dev.parent = dev;
                data->dready_trig->ops = &bmc150_magn_trigger_ops;
                iio_trigger_set_drvdata(data->dready_trig, indio_dev);
                ret = iio_trigger_register(data->dready_trig);

diff --git a/drivers/iio/magnetometer/rm3100-core.c b/drivers/iio/magnetometer/rm3100-core.c
index 7242897a05e956ee853ebb30a7c8c862bf72722e..dd811da9cb6dbbae02f67e05ab114d887e5b9416 100644 (file)
--- a/drivers/iio/magnetometer/rm3100-core.c
+++ b/drivers/iio/magnetometer/rm3100-core.c
@@ -579,7 +579,6 @@ int rm3100_common_probe(struct device *dev, struct regmap *regmap, int irq)
                if (!data->drdy_trig)
                        return -ENOMEM;
 
-               data->drdy_trig->dev.parent = dev;
                ret = devm_iio_trigger_register(dev, data->drdy_trig);
                if (ret < 0)
                        return ret;

diff --git a/drivers/iio/pressure/zpa2326.c b/drivers/iio/pressure/zpa2326.c
index 2cecbe0adb3f382be11c90b1f3615824f9f28527..70adff62cd209de6aa163e544828c97192a5754a 100644 (file)
--- a/drivers/iio/pressure/zpa2326.c
+++ b/drivers/iio/pressure/zpa2326.c
@@ -1413,7 +1413,6 @@ static int zpa2326_init_managed_trigger(struct device          *parent,
                return -ENOMEM;
 
        /* Basic setup. */
-       trigger->dev.parent = parent;
        trigger->ops = &zpa2326_trigger_ops;
 
        private->trigger = trigger;

diff --git a/drivers/iio/proximity/sx9310.c b/drivers/iio/proximity/sx9310.c
index 37fd0b65a0140085a9672269bd02adf8da391d6f..394c2afe0f23396ee0aeb718117e80c82f6f00fe 100644 (file)
--- a/drivers/iio/proximity/sx9310.c
+++ b/drivers/iio/proximity/sx9310.c
@@ -1453,7 +1453,6 @@ static int sx9310_probe(struct i2c_client *client)
                if (!data->trig)
                        return -ENOMEM;
 
-               data->trig->dev.parent = dev;
                data->trig->ops = &sx9310_trigger_ops;
                iio_trigger_set_drvdata(data->trig, indio_dev);
 

diff --git a/drivers/iio/proximity/sx9500.c b/drivers/iio/proximity/sx9500.c
index 67353d33af94ba0d49d73f1637138d1c30aa1f97..a87f4a8e43273ac158c071b8f95beb95796c886e 100644 (file)
--- a/drivers/iio/proximity/sx9500.c
+++ b/drivers/iio/proximity/sx9500.c
@@ -950,7 +950,6 @@ static int sx9500_probe(struct i2c_client *client,
                if (!data->trig)
                        return -ENOMEM;
 
-               data->trig->dev.parent = &client->dev;
                data->trig->ops = &sx9500_trigger_ops;
                iio_trigger_set_drvdata(data->trig, indio_dev);