[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210313151518.2681a59a@archlinux>
Date: Sat, 13 Mar 2021 15:15:18 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Oleksij Rempel <o.rempel@...gutronix.de>
Cc: Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org,
Pengutronix Kernel Team <kernel@...gutronix.de>,
David Jander <david@...tonic.nl>,
Robin van der Gracht <robin@...tonic.nl>,
linux-iio@...r.kernel.org, Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald-Stadler <pmeerw@...erw.net>,
Dmitry Torokhov <dmitry.torokhov@...il.com>
Subject: Re: [PATCH v2 1/3] dt-bindings:iio:adc: add generic
settling-time-us and average-samples channel properties
On Fri, 12 Mar 2021 11:55:13 +0100
Oleksij Rempel <o.rempel@...gutronix.de> wrote:
> Settling time and over sampling is a typical challenge for different IIO ADC
> devices. So, introduce channel specific settling-time-us and average-samples
> properties to cover this use case.
>
> Signed-off-by: Oleksij Rempel <o.rempel@...gutronix.de>
> ---
> Documentation/devicetree/bindings/iio/adc/adc.yaml | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/adc/adc.yaml b/Documentation/devicetree/bindings/iio/adc/adc.yaml
> index 912a7635edc4..c748f6573027 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adc.yaml
> @@ -39,4 +39,13 @@ properties:
> The first value specifies the positive input pin, the second
> specifies the negative input pin.
>
> + settling-time-us:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description:
> + Time between enabling the channel and firs stable readings.
first
> +
> + average-samples:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Number of data samples which are averaged for each read.
So in IIO at least, we tend to refer to this as oversampling. Perhaps we
should use that term here as well? It would also be good to give some hint
as to why this might be a DT property rather than a userspace control.
Thanks,
Jonathan
> +
> additionalProperties: true
Powered by blists - more mailing lists