linux-kernel - Re: [PATCH v4 1/1] Documentation: iio: Add ADC documentation

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: <Z7n-s5yOD7D9t1Oi@debian-BULLSEYE-live-builder-AMD64>
Date: Sat, 22 Feb 2025 13:43:31 -0300
From: Marcelo Schmitt <marcelo.schmitt1@...il.com>
To: Bagas Sanjaya <bagasdotme@...il.com>
Cc: Marcelo Schmitt <marcelo.schmitt@...log.com>, linux-iio@...r.kernel.org,
	linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
	jic23@...nel.org, lars@...afoo.de, corbet@....net,
	dlechner@...libre.com
Subject: Re: [PATCH v4 1/1] Documentation: iio: Add ADC documentation

On 02/20, Bagas Sanjaya wrote:
> On Wed, Feb 19, 2025 at 06:00:56PM -0300, Marcelo Schmitt wrote:
> > +For a **single-ended bipolar** channel, the analog voltage input can go from
> > +-VREF to +VREF (where -VREF is the voltage reference that has the lower
> > +electrical potential while +VREF is the reference with the higher one). Some ADC
> > +chips derive the lower reference from +VREF, others get it from a separate
>                                                   "... obtain it ..."
Ack. Maybe, if there are no additional suggestions to the docs, Jonathan can
tweak this while applying the patch? Otherwise I'll change accordingly on v5.

> > +input. Often, +VREF and -VREF are symmetric but they don't need to be so. When
> > +-VREF is lower than system ground, these inputs are also called single-ended
> > +true bipolar. Also, while there is a relevant difference between bipolar and
> > +true bipolar from the electrical perspective, IIO makes no explicit distinction
> > +between them.
> > +
> > <snipped>...
> > +In the ADC driver, ``differential = 1`` is set into ``struct iio_chan_spec`` for
> > +the channel. Even though, there are three general input types, ``differential``
> > +is only used to distinguish between differential and not differential (either
>                                                    "... non-differential ..."
Ack. Same as the other suggestion.

> > +single-ended or pseudo-differential) input types. See
> > +``include/linux/iio/iio.h`` for more information.
> 
> Thanks.
> 
> -- 
> An old man doll... just what I always wanted! - Clara


Thanks,

Marcelo