[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250531170724.12373532@jic23-huawei>
Date: Sat, 31 May 2025 17:07:24 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Angelo Dureghello <adureghello@...libre.com>
Cc: David Lechner <dlechner@...libre.com>, Nuno Sá
<nuno.sa@...log.com>, Andy Shevchenko <andy@...nel.org>, Lars-Peter Clausen
<lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>, Rob
Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor
Dooley <conor+dt@...nel.org>, linux-iio@...r.kernel.org,
linux-kernel@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH v7 1/6] Documentation: ABI: IIO: add calibconv_delay
documentation
On Mon, 26 May 2025 12:03:16 +0200
Angelo Dureghello <adureghello@...libre.com> wrote:
> From: Angelo Dureghello <adureghello@...libre.com>
>
> Add new IIO "convdelay" documentation.
That's not what the patch title says... It's called calibconv_delay still
there.
I'd fix that and version number, but looks like you are going to be doing
a v8 anyway based on David's feedback on patch 6.
Jonathan
>
> The ad7606 implements a phase calibation feature, in nanoseconds.
> Being this a time delay, using the convdelay suffix.
>
> Signed-off-by: Angelo Dureghello <adureghello@...libre.com>
> ---
> Documentation/ABI/testing/sysfs-bus-iio | 24 ++++++++++++++++++++++++
> 1 file changed, 24 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/testing/sysfs-bus-iio
> index ef52c427a015cf47bb9847782e13afbee01e9f31..7e59cbd5acb85fd0909c1d56f9d76a84933d418a 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio
> +++ b/Documentation/ABI/testing/sysfs-bus-iio
> @@ -559,6 +559,30 @@ Description:
> - a small discrete set of values like "0 2 4 6 8"
> - a range specified as "[min step max]"
>
> +What: /sys/bus/iio/devices/iio:deviceX/in_voltageY_convdelay
> +KernelVersion: 6.16
> +Contact: linux-iio@...r.kernel.org
> +Description:
> + Delay of start of conversion from common reference point shared
> + by all channels. Can be writable when used to compensate for
> + delay variation introduced by external filters feeding a
> + simultaneous sampling ADC.
> +
> + E.g., for the ad7606 ADC series, this value is intended as a
> + configurable time delay in seconds, to correct delay introduced
> + by an optional external filtering circuit.
> +
> +What: /sys/bus/iio/devices/iio:deviceX/in_voltageY_convdelay_available
> +KernelVersion: 6.16
> +Contact: linux-iio@...r.kernel.org
> +Description:
> + Available values of convdelay. Maybe expressed as:
> +
> + - a range specified as "[min step max]"
> +
> + If shared across all channels, <type>_convdelay_available
> + is used.
> +
> What: /sys/bus/iio/devices/iio:deviceX/in_accel_x_calibscale
> What: /sys/bus/iio/devices/iio:deviceX/in_accel_y_calibscale
> What: /sys/bus/iio/devices/iio:deviceX/in_accel_z_calibscale
>
Powered by blists - more mailing lists