All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging iio: use iio_trigger_generic_data_rdy_poll() in accel/adis16209 driver
@ 2012-09-29  0:07 Peter Meerwald
  2012-10-03  9:18 ` Lars-Peter Clausen
  0 siblings, 1 reply; 2+ messages in thread
From: Peter Meerwald @ 2012-09-29  0:07 UTC (permalink / raw)
  To: linux-iio; +Cc: Peter Meerwald, 21cnbao

Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Cc: 21cnbao@gmail.com
---
 drivers/staging/iio/accel/adis16209_trigger.c |   11 +----------
 drivers/staging/iio/accel/adis16240_trigger.c |   11 +----------
 2 files changed, 2 insertions(+), 20 deletions(-)

diff --git a/drivers/staging/iio/accel/adis16209_trigger.c b/drivers/staging/iio/accel/adis16209_trigger.c
index 2ad93dc..1122803 100644
--- a/drivers/staging/iio/accel/adis16209_trigger.c
+++ b/drivers/staging/iio/accel/adis16209_trigger.c
@@ -8,15 +8,6 @@
 #include "adis16209.h"
 
 /**
- * adis16209_data_rdy_trig_poll() the event handler for the data rdy trig
- **/
-static irqreturn_t adis16209_data_rdy_trig_poll(int irq, void *trig)
-{
-	iio_trigger_poll(trig, iio_get_time_ns());
-	return IRQ_HANDLED;
-}
-
-/**
  * adis16209_data_rdy_trigger_set_state() set datardy interrupt state
  **/
 static int adis16209_data_rdy_trigger_set_state(struct iio_trigger *trig,
@@ -45,7 +36,7 @@ int adis16209_probe_trigger(struct iio_dev *indio_dev)
 	}
 
 	ret = request_irq(st->us->irq,
-			  adis16209_data_rdy_trig_poll,
+			  iio_trigger_generic_data_rdy_poll,
 			  IRQF_TRIGGER_RISING,
 			  "adis16209",
 			  st->trig);
diff --git a/drivers/staging/iio/accel/adis16240_trigger.c b/drivers/staging/iio/accel/adis16240_trigger.c
index fa90a22..f3caf09 100644
--- a/drivers/staging/iio/accel/adis16240_trigger.c
+++ b/drivers/staging/iio/accel/adis16240_trigger.c
@@ -8,15 +8,6 @@
 #include "adis16240.h"
 
 /**
- * adis16240_data_rdy_trig_poll() the event handler for the data rdy trig
- **/
-static irqreturn_t adis16240_data_rdy_trig_poll(int irq, void *trig)
-{
-	iio_trigger_poll(trig, iio_get_time_ns());
-	return IRQ_HANDLED;
-}
-
-/**
  * adis16240_data_rdy_trigger_set_state() set datardy interrupt state
  **/
 static int adis16240_data_rdy_trigger_set_state(struct iio_trigger *trig,
@@ -45,7 +36,7 @@ int adis16240_probe_trigger(struct iio_dev *indio_dev)
 	}
 
 	ret = request_irq(st->us->irq,
-			  adis16240_data_rdy_trig_poll,
+			  iio_trigger_generic_data_rdy_poll,
 			  IRQF_TRIGGER_RISING,
 			  "adis16240",
 			  st->trig);
-- 
1.7.9.5


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] staging iio: use iio_trigger_generic_data_rdy_poll() in accel/adis16209 driver
  2012-09-29  0:07 [PATCH] staging iio: use iio_trigger_generic_data_rdy_poll() in accel/adis16209 driver Peter Meerwald
@ 2012-10-03  9:18 ` Lars-Peter Clausen
  0 siblings, 0 replies; 2+ messages in thread
From: Lars-Peter Clausen @ 2012-10-03  9:18 UTC (permalink / raw)
  To: Peter Meerwald; +Cc: linux-iio, 21cnbao

On 09/29/2012 02:07 AM, Peter Meerwald wrote:
> Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
> Cc: 21cnbao@gmail.com

Looks good,

Acked-by: Lars-Peter Clausen <lars@metafoo.de>

> ---
>  drivers/staging/iio/accel/adis16209_trigger.c |   11 +----------
>  drivers/staging/iio/accel/adis16240_trigger.c |   11 +----------
>  2 files changed, 2 insertions(+), 20 deletions(-)
> 
> diff --git a/drivers/staging/iio/accel/adis16209_trigger.c b/drivers/staging/iio/accel/adis16209_trigger.c
> index 2ad93dc..1122803 100644
> --- a/drivers/staging/iio/accel/adis16209_trigger.c
> +++ b/drivers/staging/iio/accel/adis16209_trigger.c
> @@ -8,15 +8,6 @@
>  #include "adis16209.h"
>  
>  /**
> - * adis16209_data_rdy_trig_poll() the event handler for the data rdy trig
> - **/
> -static irqreturn_t adis16209_data_rdy_trig_poll(int irq, void *trig)
> -{
> -	iio_trigger_poll(trig, iio_get_time_ns());
> -	return IRQ_HANDLED;
> -}
> -
> -/**
>   * adis16209_data_rdy_trigger_set_state() set datardy interrupt state
>   **/
>  static int adis16209_data_rdy_trigger_set_state(struct iio_trigger *trig,
> @@ -45,7 +36,7 @@ int adis16209_probe_trigger(struct iio_dev *indio_dev)
>  	}
>  
>  	ret = request_irq(st->us->irq,
> -			  adis16209_data_rdy_trig_poll,
> +			  iio_trigger_generic_data_rdy_poll,
>  			  IRQF_TRIGGER_RISING,
>  			  "adis16209",
>  			  st->trig);
> diff --git a/drivers/staging/iio/accel/adis16240_trigger.c b/drivers/staging/iio/accel/adis16240_trigger.c
> index fa90a22..f3caf09 100644
> --- a/drivers/staging/iio/accel/adis16240_trigger.c
> +++ b/drivers/staging/iio/accel/adis16240_trigger.c
> @@ -8,15 +8,6 @@
>  #include "adis16240.h"
>  
>  /**
> - * adis16240_data_rdy_trig_poll() the event handler for the data rdy trig
> - **/
> -static irqreturn_t adis16240_data_rdy_trig_poll(int irq, void *trig)
> -{
> -	iio_trigger_poll(trig, iio_get_time_ns());
> -	return IRQ_HANDLED;
> -}
> -
> -/**
>   * adis16240_data_rdy_trigger_set_state() set datardy interrupt state
>   **/
>  static int adis16240_data_rdy_trigger_set_state(struct iio_trigger *trig,
> @@ -45,7 +36,7 @@ int adis16240_probe_trigger(struct iio_dev *indio_dev)
>  	}
>  
>  	ret = request_irq(st->us->irq,
> -			  adis16240_data_rdy_trig_poll,
> +			  iio_trigger_generic_data_rdy_poll,
>  			  IRQF_TRIGGER_RISING,
>  			  "adis16240",
>  			  st->trig);


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-03  9:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-29  0:07 [PATCH] staging iio: use iio_trigger_generic_data_rdy_poll() in accel/adis16209 driver Peter Meerwald
2012-10-03  9:18 ` Lars-Peter Clausen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.