linux-kernel - Re: [PATCH 5/5] docs: iio: ad7380: add support for single-ended parts

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240728173725.45043f11@jic23-huawei>
Date: Sun, 28 Jul 2024 17:37:25 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Julien Stephan <jstephan@...libre.com>
Cc: Michael Hennerich <michael.hennerich@...log.com>, Nuno Sá
 <nuno.sa@...log.com>, David Lechner <dlechner@...libre.com>, Lars-Peter
 Clausen <lars@...afoo.de>, Rob Herring <robh@...nel.org>, Krzysztof
 Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
 Jonathan Corbet <corbet@....net>, linux-iio@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-doc@...r.kernel.org
Subject: Re: [PATCH 5/5] docs: iio: ad7380: add support for single-ended
 parts

On Fri, 26 Jul 2024 17:20:10 +0200
Julien Stephan <jstephan@...libre.com> wrote:

> The AD7380 family has some compatible single-ended chips: AD7386/7/8(-4).
> These single-ended chips have a  2:1 multiplexer in front of each ADC.
> They also include additional configuration registers that allow for either
> manual selection or automatic switching (sequencer mode), of the
> multiplexer inputs. Add a section to describe this.
> 
> Signed-off-by: Julien Stephan <jstephan@...libre.com>
Just one trivial missing space.

Nice patch set. Thanks,

Jonathan


> ---
>  Documentation/iio/ad7380.rst | 42 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 42 insertions(+)
> 
> diff --git a/Documentation/iio/ad7380.rst b/Documentation/iio/ad7380.rst
> index 061cd632b5df..81dfa39519fb 100644
> --- a/Documentation/iio/ad7380.rst
> +++ b/Documentation/iio/ad7380.rst
> @@ -17,10 +17,16 @@ The following chips are supported by this driver:
>  * `AD7381 <https://www.analog.com/en/products/ad7381.html>`_
>  * `AD7383 <https://www.analog.com/en/products/ad7383.html>`_
>  * `AD7384 <https://www.analog.com/en/products/ad7384.html>`_
> +* `AD7386 <https://www.analog.com/en/products/ad7386.html>`_
> +* `AD7387 <https://www.analog.com/en/products/ad7387.html>`_
> +* `AD7388 <https://www.analog.com/en/products/ad7388.html>`_
>  * `AD7380-4 <https://www.analog.com/en/products/ad7380-4.html>`_
>  * `AD7381-4 <https://www.analog.com/en/products/ad7381-4.html>`_
>  * `AD7383-4 <https://www.analog.com/en/products/ad7383-4.html>`_
>  * `AD7384-4 <https://www.analog.com/en/products/ad7384-4.html>`_
> +* `AD7386-4 <https://www.analog.com/en/products/ad7386-4.html>`_
> +* `AD7387-4 <https://www.analog.com/en/products/ad7387-4.html>`_
> +* `AD7388-4 <https://www.analog.com/en/products/ad7388-4.html>`_
>  
>  
>  Supported features
> @@ -69,6 +75,42 @@ must restart iiod using the following command:
>  
>  	root:~# systemctl restart iiod
>  
> +Channel selection and sequencer (single-end chips only)
> +-------------------------------------------------------
> +
> +Single-ended chips of this family (ad7386/7/8(-4)) have a 2:1 multiplexer in
> +front of each ADC. They also include additional configuration registers that
allow for either manual selection or automatic switching (sequencer mode),of the
space after ,
plus adjust the wrap as that'll make it 81 chars I think.

> +multiplexer inputs.
> +
> +From an IIO point of view, all inputs are exported, i.e ad7386/7/8
> +export 4 channels and ad7386-4/7-4/8-4 export 8 channels.
> +
> +Inputs ``AinX0`` of multiplexers correspond to the first half of IIO channels (i.e
> +0-1 or 0-3) and inputs ``AinX1`` correspond to second half (i.e 2-3 or 4-7).
> +Example for AD7386/7/8 (2 channels parts):
> +
> +.. code-block::
> +
> +	   IIO   | AD7386/7/8
> +	         |         +----------------------------
> +	         |         |     _____        ______
> +	         |         |    |     |      |      |
> +	voltage0 | AinA0 --|--->|     |      |      |
> +	         |         |    | mux |----->| ADCA |---
> +	voltage2 | AinA1 --|--->|     |      |      |
> +	         |         |    |_____|      |_____ |
> +	         |         |     _____        ______
> +	         |         |    |     |      |      |
> +	voltage1 | AinB0 --|--->|     |      |      |
> +	         |         |    | mux |----->| ADCB |---
> +	voltage3 | AinB1 --|--->|     |      |      |
> +	         |         |    |_____|      |______|
> +	         |         |
> +	         |         +----------------------------
> +
> +
> +When enabling sequencer mode, the effective sampling rate is divided by two.
>  
>  Unimplemented features
>  ----------------------
>