Re: [PATCH v10 10/11] docs: iio: add documentation for adf41513 driver

Jonathan Cameron
2026-04-25T16:40:33Z

On Wed, 15 Apr 2026 10:51:53 +0100
Rodrigo Alencar via B4 Relay <devnull+rodrigo.alencar.analog.com@kernel.org> wrote:

> From: Rodrigo Alencar <rodrigo.alencar@analog.com>
>
> Add documentation for ADF41513 driver, which describes the device
> driver files and shows how userspace may consume the ABI for various
> tasks.
>
> Signed-off-by: Rodrigo Alencar <rodrigo.alencar@analog.com>
> ---
> Documentation/iio/adf41513.rst | 199 +++++++++++++++++++++++++++++++++++++++++
> Documentation/iio/index.rst | 1 +
> MAINTAINERS | 1 +
> 3 files changed, 201 insertions(+)
>
> diff --git a/Documentation/iio/adf41513.rst b/Documentation/iio/adf41513.rst
> new file mode 100644
> index 000000000000..4193c825b532
> --- /dev/null
> +++ b/Documentation/iio/adf41513.rst
> @@ -0,0 +1,199 @@

> +2. Device attributes
> +====================
> +
> +The ADF41513 driver provides the following IIO extended attributes for
> +frequency control and monitoring:
> +
> +Each IIO device has a device folder under ``/sys/bus/iio/devices/iio:deviceX``,
> +where X is the IIO index of the device. Under these folders reside a set of
> +device files that provide access to the synthesizer's functionality.
> +
> +The following table shows the ADF41513 related device files:
> +
> ++----------------------+-------------------------------------------------------+
> +| Device file | Description |
> ++======================+=======================================================+
> +| frequency | RF output frequency control and readback (Hz) |
> ++----------------------+-------------------------------------------------------+
> +| frequency_resolution | Target frequency resolution control (Hz) |
> ++----------------------+-------------------------------------------------------+
> +| powerdown | Power management control (0=active, 1=power down) |
> ++----------------------+-------------------------------------------------------+
> +| phase | RF output phase adjustment and readback (radians) |
> ++----------------------+-------------------------------------------------------+

These seem to be truncated as the out_altvoltage0_ prefix isn't mentioned.
Fine to do that but add a note to say that more clearly.
I briefly read this as 'new ABI' though the examples a at the end of the file
make it clear that we are simply missing the prefix.