On 23/09/16 16:19, Lars-Peter Clausen wrote:
> Use the new iio_trigger_validate_own_device() to verify that the trigger
> can only be attached to the matching IIO device rather than using a custom
> variant.
>
> While the implementation of iio_trigger_validate_own_device() and the
> custom variant and are not identical their behaviour is.
>
> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
Applied to the togreg branch of iio.git - initially pushed out as testing
for the autobuilders to play with.
Thanks,
Jonathan
> ---
> New in v3
> ---
> drivers/iio/adc/max1027.c | 13 +------------
> 1 file changed, 1 insertion(+), 12 deletions(-)
>
> diff --git a/drivers/iio/adc/max1027.c b/drivers/iio/adc/max1027.c
> index 712fbd2..8ea3271 100644
> --- a/drivers/iio/adc/max1027.c
> +++ b/drivers/iio/adc/max1027.c
> @@ -360,17 +360,6 @@ static int max1027_set_trigger_state(struct iio_trigger *trig, bool state)
> return 0;
> }
>
> -static int max1027_validate_device(struct iio_trigger *trig,
> - struct iio_dev *indio_dev)
> -{
> - struct iio_dev *indio = iio_trigger_get_drvdata(trig);
> -
> - if (indio != indio_dev)
> - return -EINVAL;
> -
> - return 0;
> -}
> -
> static irqreturn_t max1027_trigger_handler(int irq, void *private)
> {
> struct iio_poll_func *pf = (struct iio_poll_func *)private;
> @@ -391,7 +380,7 @@ static irqreturn_t max1027_trigger_handler(int irq, void *private)
>
> static const struct iio_trigger_ops max1027_trigger_ops = {
> .owner = THIS_MODULE,
> - .validate_device = &max1027_validate_device,
> + .validate_device = &iio_trigger_validate_own_device,
> .set_trigger_state = &max1027_set_trigger_state,
> };
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
