Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1592750 > unrolled thread
| Started by | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| First post | 2017-03-05 13:20 +0100 |
| Last post | 2017-03-05 13:20 +0100 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v3 4/6] iio: iio-interrupt-trigger: device-tree support Jonathan Cameron <jic23@kernel.org> - 2017-03-05 13:20 +0100
| From | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| Date | 2017-03-05 13:20 +0100 |
| Subject | Re: [PATCH v3 4/6] iio: iio-interrupt-trigger: device-tree support |
| Message-ID | <thDiO-7Q-9@gated-at.bofh.it> |
On 28/02/17 16:51, Fabrice Gasnier wrote:
> From: Grégor Boirie <gregor.boirie@parrot.com>
>
> Add compatible string that should be used in DT.
> Also cascade of_node to newly allocated trigger device.
>
> Signed-off-by: Gregor Boirie <gregor.boirie@parrot.com>
I'd normally not keep Gregor's sign off in a patch that has been
changed like this. If Gregor is happy it's a different matter,
but I suspect he hasn't replied yet as he is always very busy!
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
Otherwise, all good, subject to convincing the device tree
guys about the binding.
Jonathan
> ---
> Original PATCH from Gregor Boirie:
> https://marc.info/?l=linux-iio&m=145590953607905&w=4
>
> Updates:
> - compatible string changed to "interrupt-trigger"
> - cascade of_node down to iio trigger device
> ---
> drivers/iio/trigger/iio-trig-interrupt.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/drivers/iio/trigger/iio-trig-interrupt.c b/drivers/iio/trigger/iio-trig-interrupt.c
> index e18f12b..6ee1f7e 100644
> --- a/drivers/iio/trigger/iio-trig-interrupt.c
> +++ b/drivers/iio/trigger/iio-trig-interrupt.c
> @@ -62,6 +62,7 @@ static int iio_interrupt_trigger_probe(struct platform_device *pdev)
> }
> iio_trigger_set_drvdata(trig, trig_info);
> trig_info->irq = irq;
> + trig->dev.of_node = pdev->dev.of_node;
> trig->ops = &iio_interrupt_trigger_ops;
> ret = request_irq(irq, iio_interrupt_trigger_poll,
> irqflags, trig->name, trig);
> @@ -104,11 +105,20 @@ static int iio_interrupt_trigger_remove(struct platform_device *pdev)
> return 0;
> }
>
> +#ifdef CONFIG_OF
> +static const struct of_device_id iio_interrupt_trigger_of_match[] = {
> + { .compatible = "interrupt-trigger" },
> + {},
> +};
> +MODULE_DEVICE_TABLE(of, iio_interrupt_trigger_of_match);
> +#endif
> +
> static struct platform_driver iio_interrupt_trigger_driver = {
> .probe = iio_interrupt_trigger_probe,
> .remove = iio_interrupt_trigger_remove,
> .driver = {
> .name = "iio_interrupt_trigger",
> + .of_match_table = of_match_ptr(iio_interrupt_trigger_of_match),
> },
> };
>
>
Back to top | Article view | linux.kernel
csiph-web