summaryrefslogtreecommitdiff
path: root/drivers/staging/iio/accel/adis16209_trigger.c
diff options
context:
space:
mode:
authorJonathan Cameron <jic23@cam.ac.uk>2011-05-18 14:41:35 +0100
committerGreg Kroah-Hartman <gregkh@suse.de>2011-05-19 16:14:52 -0700
commitd7e044f146ade0336d431f1c15d4915565731963 (patch)
tree07737035a2e351dba0b8caa126f5343dbb8e4f18 /drivers/staging/iio/accel/adis16209_trigger.c
parent80142f0de68a3d68f114f25f56e736c4a63e1c18 (diff)
staging:iio:accel:adis16209 move to irqchip based trigger handling.
Untested. Signed-off-by: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/iio/accel/adis16209_trigger.c')
-rw-r--r--drivers/staging/iio/accel/adis16209_trigger.c40
1 files changed, 15 insertions, 25 deletions
diff --git a/drivers/staging/iio/accel/adis16209_trigger.c b/drivers/staging/iio/accel/adis16209_trigger.c
index 488e4821f54a..86e83c356cb2 100644
--- a/drivers/staging/iio/accel/adis16209_trigger.c
+++ b/drivers/staging/iio/accel/adis16209_trigger.c
@@ -17,7 +17,6 @@
**/
static irqreturn_t adis16209_data_rdy_trig_poll(int irq, void *trig)
{
- disable_irq_nosync(irq);
iio_trigger_poll(trig, iio_get_time_ns());
return IRQ_HANDLED;
}
@@ -46,27 +45,26 @@ static int adis16209_data_rdy_trigger_set_state(struct iio_trigger *trig,
return adis16209_set_irq(&st->indio_dev->dev, state);
}
-/**
- * adis16209_trig_try_reen() try renabling irq for data rdy trigger
- * @trig: the datardy trigger
- **/
-static int adis16209_trig_try_reen(struct iio_trigger *trig)
-{
- struct adis16209_state *st = trig->private_data;
- enable_irq(st->us->irq);
- return 0;
-}
-
int adis16209_probe_trigger(struct iio_dev *indio_dev)
{
int ret;
struct adis16209_state *st = indio_dev->dev_data;
+ char *name;
- st->trig = iio_allocate_trigger();
- if (st->trig == NULL) {
+ name = kasprintf(GFP_KERNEL,
+ "adis16209-dev%d",
+ indio_dev->id);
+ if (name == NULL) {
ret = -ENOMEM;
goto error_ret;
}
+
+ st->trig = iio_allocate_trigger_named(name);
+ if (st->trig == NULL) {
+ ret = -ENOMEM;
+ goto error_free_name;
+ }
+
ret = request_irq(st->us->irq,
adis16209_data_rdy_trig_poll,
IRQF_TRIGGER_RISING,
@@ -74,34 +72,26 @@ int adis16209_probe_trigger(struct iio_dev *indio_dev)
st->trig);
if (ret)
goto error_free_trig;
- st->trig->name = kasprintf(GFP_KERNEL,
- "adis16209-dev%d",
- indio_dev->id);
- if (!st->trig->name) {
- ret = -ENOMEM;
- goto error_free_irq;
- }
st->trig->dev.parent = &st->us->dev;
st->trig->owner = THIS_MODULE;
st->trig->private_data = st;
st->trig->set_trigger_state = &adis16209_data_rdy_trigger_set_state;
- st->trig->try_reenable = &adis16209_trig_try_reen;
st->trig->control_attrs = &adis16209_trigger_attr_group;
ret = iio_trigger_register(st->trig);
/* select default trigger */
indio_dev->trig = st->trig;
if (ret)
- goto error_free_trig_name;
+ goto error_free_irq;
return 0;
-error_free_trig_name:
- kfree(st->trig->name);
error_free_irq:
free_irq(st->us->irq, st->trig);
error_free_trig:
iio_free_trigger(st->trig);
+error_free_name:
+ kfree(name);
error_ret:
return ret;
}