Commit 4d031666 authored by Gwendal Grignou's avatar Gwendal Grignou Committed by Jonathan Cameron
Browse files

iio: lmp91000: Remove code to set trigger parent 



iio_trigger_set_drvdata() sets the trigger device parent to first
argument of viio_trigger_alloc(), no need to do it again in the driver
code.

Given data->dev is dev, and we call devm_iio_trigger_alloc with
dev instead of data->dev, we do not have to set data->trig->dev.parent to
dev anymore.

Signed-off-by: default avatarGwendal Grignou <gwendal@chromium.org>
Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: default avatarMatt Ranostay <matt.ranostay@konsulko.com>
Link: https://lore.kernel.org/r/20210309193620.2176163-6-gwendal@chromium.org


Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent cd214139

drivers/iio/potentiostat/lmp91000.c

+1 −2
Original line number Diff line number Diff line
@@ -322,7 +322,7 @@ static int lmp91000_probe(struct i2c_client *client, 
		return PTR_ERR(data->regmap);

	}



	data->trig = devm_iio_trigger_alloc(data->dev, "%s-mux%d",

	data->trig = devm_iio_trigger_alloc(dev, "%s-mux%d",

					    indio_dev->name, indio_dev->id);

	if (!data->trig) {

		dev_err(dev, "cannot allocate iio trigger.\n");
@@ -330,7 +330,6 @@ static int lmp91000_probe(struct i2c_client *client, 
	}



	data->trig->ops = &lmp91000_trigger_ops;

	data->trig->dev.parent = dev;

	init_completion(&data->completion);



	ret = lmp91000_read_config(data);