iio:bma180: Enable use of device without IRQ
authorPeter Meerwald <pmeerw@pmeerw.net>
Tue, 19 Aug 2014 22:43:00 +0000 (23:43 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sun, 14 Sep 2014 17:46:49 +0000 (18:46 +0100)
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Cc: Oleksandr Kravchenko <o.v.kravchenko@globallogic.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/accel/bma180.c

index a077cc86421b5eb24519ab25a61ebb41105af5e8..f5e26fb697a333de5b4205f25d631e75a9a06896 100644 (file)
@@ -529,7 +529,6 @@ static int bma180_probe(struct i2c_client *client,
 {
        struct bma180_data *data;
        struct iio_dev *indio_dev;
-       struct iio_trigger *trig;
        int ret;
 
        indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
@@ -553,29 +552,31 @@ static int bma180_probe(struct i2c_client *client,
        indio_dev->modes = INDIO_DIRECT_MODE;
        indio_dev->info = &bma180_info;
 
-       trig = iio_trigger_alloc("%s-dev%d", indio_dev->name, indio_dev->id);
-       if (!trig) {
-               ret = -ENOMEM;
-               goto err_chip_disable;
-       }
+       if (client->irq > 0) {
+               data->trig = iio_trigger_alloc("%s-dev%d", indio_dev->name,
+                       indio_dev->id);
+               if (!data->trig) {
+                       ret = -ENOMEM;
+                       goto err_chip_disable;
+               }
 
-       ret = devm_request_irq(&client->dev, client->irq,
-                       iio_trigger_generic_data_rdy_poll,
-                       IRQF_TRIGGER_RISING, BMA180_IRQ_NAME, trig);
-       if (ret) {
-               dev_err(&client->dev, "unable to request IRQ\n");
-               goto err_trigger_free;
-       }
+               ret = devm_request_irq(&client->dev, client->irq,
+                       iio_trigger_generic_data_rdy_poll, IRQF_TRIGGER_RISING,
+                       BMA180_IRQ_NAME, data->trig);
+               if (ret) {
+                       dev_err(&client->dev, "unable to request IRQ\n");
+                       goto err_trigger_free;
+               }
 
-       trig->dev.parent = &client->dev;
-       trig->ops = &bma180_trigger_ops;
-       iio_trigger_set_drvdata(trig, indio_dev);
-       data->trig = trig;
-       indio_dev->trig = trig;
+               data->trig->dev.parent = &client->dev;
+               data->trig->ops = &bma180_trigger_ops;
+               iio_trigger_set_drvdata(data->trig, indio_dev);
+               indio_dev->trig = data->trig;
 
-       ret = iio_trigger_register(trig);
-       if (ret)
-               goto err_trigger_free;
+               ret = iio_trigger_register(data->trig);
+               if (ret)
+                       goto err_trigger_free;
+       }
 
        ret = iio_triggered_buffer_setup(indio_dev, NULL,
                        bma180_trigger_handler, NULL);
@@ -595,9 +596,10 @@ static int bma180_probe(struct i2c_client *client,
 err_buffer_cleanup:
        iio_triggered_buffer_cleanup(indio_dev);
 err_trigger_unregister:
-       iio_trigger_unregister(trig);
+       if (data->trig)
+               iio_trigger_unregister(data->trig);
 err_trigger_free:
-       iio_trigger_free(trig);
+       iio_trigger_free(data->trig);
 err_chip_disable:
        bma180_chip_disable(data);
 
@@ -611,8 +613,10 @@ static int bma180_remove(struct i2c_client *client)
 
        iio_device_unregister(indio_dev);
        iio_triggered_buffer_cleanup(indio_dev);
-       iio_trigger_unregister(data->trig);
-       iio_trigger_free(data->trig);
+       if (data->trig) {
+               iio_trigger_unregister(data->trig);
+               iio_trigger_free(data->trig);
+       }
 
        mutex_lock(&data->mutex);
        bma180_chip_disable(data);