diff options
| author | Markus Pargmann <mpa@pengutronix.de> | 2015-08-12 16:50:07 +0200 | 
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2015-08-16 10:51:23 +0100 | 
| commit | 5d889abbe5c39c0c9ebe468dbcab2688324a33fc (patch) | |
| tree | ba8a7d542e4e5aa07194afd6c51ffa48a00ac896 /drivers/iio/gyro | |
| parent | ebc6eb5907acecdebbd0e180a754d98f21045fbc (diff) | |
iio: bmg160: Remove remaining uses of i2c_client
Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio/gyro')
| -rw-r--r-- | drivers/iio/gyro/bmg160.c | 59 | 
1 files changed, 29 insertions, 30 deletions
| diff --git a/drivers/iio/gyro/bmg160.c b/drivers/iio/gyro/bmg160.c index 7492532cd402..ca12402ffd93 100644 --- a/drivers/iio/gyro/bmg160.c +++ b/drivers/iio/gyro/bmg160.c @@ -110,6 +110,7 @@ struct bmg160_data {  	int slope_thres;  	bool dready_trigger_on;  	bool motion_trigger_on; +	int irq;  };  enum bmg160_axis { @@ -961,18 +962,13 @@ static const struct iio_buffer_setup_ops bmg160_buffer_setup_ops = {  	.postdisable = bmg160_buffer_postdisable,  }; -static int bmg160_gpio_probe(struct i2c_client *client, -			     struct bmg160_data *data) +static int bmg160_gpio_probe(struct bmg160_data *data)  {  	struct device *dev;  	struct gpio_desc *gpio; -	int ret; - -	if (!client) -		return -EINVAL; -	dev = &client->dev; +	dev = data->dev;  	/* data ready gpio interrupt pin */  	gpio = devm_gpiod_get_index(dev, BMG160_GPIO_NAME, 0, GPIOD_IN); @@ -981,11 +977,12 @@ static int bmg160_gpio_probe(struct i2c_client *client,  		return PTR_ERR(gpio);  	} -	ret = gpiod_to_irq(gpio); +	data->irq = gpiod_to_irq(gpio); -	dev_dbg(dev, "GPIO resource, no:%d irq:%d\n", desc_to_gpio(gpio), ret); +	dev_dbg(dev, "GPIO resource, no:%d irq:%d\n", desc_to_gpio(gpio), +		data->irq); -	return ret; +	return 0;  }  static const char *bmg160_match_acpi_device(struct device *dev) @@ -1007,6 +1004,7 @@ static int bmg160_probe(struct i2c_client *client,  	int ret;  	const char *name = NULL;  	struct regmap *regmap; +	struct device *dev = &client->dev;  	regmap = devm_regmap_init_i2c(client, &bmg160_regmap_i2c_conf);  	if (IS_ERR(regmap)) { @@ -1020,8 +1018,9 @@ static int bmg160_probe(struct i2c_client *client,  		return -ENOMEM;  	data = iio_priv(indio_dev); -	dev_set_drvdata(&client->dev, indio_dev); -	data->dev = &client->dev; +	dev_set_drvdata(dev, indio_dev); +	data->dev = dev; +	data->irq = client->irq;  	ret = bmg160_chip_init(data);  	if (ret < 0) @@ -1032,22 +1031,22 @@ static int bmg160_probe(struct i2c_client *client,  	if (id)  		name = id->name; -	if (ACPI_HANDLE(&client->dev)) -		name = bmg160_match_acpi_device(&client->dev); +	if (ACPI_HANDLE(dev)) +		name = bmg160_match_acpi_device(dev); -	indio_dev->dev.parent = &client->dev; +	indio_dev->dev.parent = dev;  	indio_dev->channels = bmg160_channels;  	indio_dev->num_channels = ARRAY_SIZE(bmg160_channels);  	indio_dev->name = name;  	indio_dev->modes = INDIO_DIRECT_MODE;  	indio_dev->info = &bmg160_info; -	if (client->irq <= 0) -		client->irq = bmg160_gpio_probe(client, data); +	if (data->irq <= 0) +		bmg160_gpio_probe(data); -	if (client->irq > 0) { -		ret = devm_request_threaded_irq(&client->dev, -						client->irq, +	if (data->irq > 0) { +		ret = devm_request_threaded_irq(dev, +						data->irq,  						bmg160_data_rdy_trig_poll,  						bmg160_event_handler,  						IRQF_TRIGGER_RISING, @@ -1056,28 +1055,28 @@ static int bmg160_probe(struct i2c_client *client,  		if (ret)  			return ret; -		data->dready_trig = devm_iio_trigger_alloc(&client->dev, +		data->dready_trig = devm_iio_trigger_alloc(dev,  							   "%s-dev%d",  							   indio_dev->name,  							   indio_dev->id);  		if (!data->dready_trig)  			return -ENOMEM; -		data->motion_trig = devm_iio_trigger_alloc(&client->dev, +		data->motion_trig = devm_iio_trigger_alloc(dev,  							  "%s-any-motion-dev%d",  							  indio_dev->name,  							  indio_dev->id);  		if (!data->motion_trig)  			return -ENOMEM; -		data->dready_trig->dev.parent = &client->dev; +		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 = &client->dev; +		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); @@ -1092,25 +1091,25 @@ static int bmg160_probe(struct i2c_client *client,  					 bmg160_trigger_handler,  					 &bmg160_buffer_setup_ops);  	if (ret < 0) { -		dev_err(&client->dev, +		dev_err(dev,  			"iio triggered buffer setup failed\n");  		goto err_trigger_unregister;  	}  	ret = iio_device_register(indio_dev);  	if (ret < 0) { -		dev_err(&client->dev, "unable to register iio device\n"); +		dev_err(dev, "unable to register iio device\n");  		goto err_buffer_cleanup;  	} -	ret = pm_runtime_set_active(&client->dev); +	ret = pm_runtime_set_active(dev);  	if (ret)  		goto err_iio_unregister; -	pm_runtime_enable(&client->dev); -	pm_runtime_set_autosuspend_delay(&client->dev, +	pm_runtime_enable(dev); +	pm_runtime_set_autosuspend_delay(dev,  					 BMG160_AUTO_SUSPEND_DELAY_MS); -	pm_runtime_use_autosuspend(&client->dev); +	pm_runtime_use_autosuspend(dev);  	return 0; | 
