On 1/14/25 7:53 AM, Marcelo Schmitt wrote: > ADCs can have different input configurations such that developers can get > confused when trying to model some of them into IIO channels. > > For example, some differential ADCs can have their channels configured as > pseudo-differential channels. In that configuration, only one input > connects to the signal of interest as opposed to using two inputs of a > differential input configuration. Datasheets sometimes also refer to > pseudo-differential inputs as single-ended inputs even though they have > distinct physical configuration and measurement procedure. There has been > some previous discussion in the mailing list about pseudo-differential and > single-ended channels [1]. > > Documenting the many possible ADC channel configurations should provide two > benefits: > A) Consolidate the knowledge from [2] and from [1], and hopefully reduce > the reviewing time of forthcoming ADC drivers. > B) Help Linux developers figure out quicker how to better support > differential ADCs, specially those that can have channels configured as > pseudo-differential inputs. > > Add documentation about common ADC characteristics and IIO support for them. > > [1]: https://lore.kernel.org/linux-iio/0fef36f8-a7db-40cc-86bd-9449cb4ab46e@xxxxxxxxx/ > [2]: https://www.analog.com/en/resources/technical-articles/sar-adc-input-types.html. > > Signed-off-by: Marcelo Schmitt <marcelo.schmitt@xxxxxxxxxx> > --- > Change log v1 -> v2 > - Split apart from AD4170 patch set. > - Added disclaimer paragraph about complex ADCs. > - Made the input type general description a bit more general, by adding > `general` to the text. > - Changed `::` placement to be consistet in all sections. > - Improved Differential channels section with more precise explanation. > - Improved Differential Unipolar Channels section with comment about common > mode voltage being described in dt as a voltage regulator. > - Removed -VREF from Pseudo-differential Unipolar diagram. > - Added comment about omitting `_offset` when IN- is at GND. > - Improved .rst formating with ``. > > Link to v1: https://lore.kernel.org/linux-iio/48876e204590c47c532fc5f71e02ca3a00028cb7.1734530280.git.marcelo.schmitt@xxxxxxxxxx/ > > > Documentation/iio/iio_adc.rst | 290 ++++++++++++++++++++++++++++++++++ > Documentation/iio/index.rst | 1 + > 2 files changed, 291 insertions(+) > create mode 100644 Documentation/iio/iio_adc.rst > > diff --git a/Documentation/iio/iio_adc.rst b/Documentation/iio/iio_adc.rst > new file mode 100644 > index 000000000000..b9d30e732a61 > --- /dev/null > +++ b/Documentation/iio/iio_adc.rst > @@ -0,0 +1,290 @@ > +.. SPDX-License-Identifier: GPL-2.0 > + > +========================= > +IIO Abstractions for ADCs > +========================= > + > +1. Overview > +=========== > + > +The IIO subsystem supports many Analog to Digital Converters (ADCs). Some ADCs > +have features and characteristics that are supported in specific ways by IIO > +device drivers. This documentation describes common ADC features and explains > +how they are (should be?) supported by the IIO subsystem. > + > +1. ADC Channel Types > +==================== > + > +ADCs can have distinct types of inputs, each of them measuring analog voltages > +in a slightly different way. An ADC digitizes the analog input voltage over a > +span that is often given by the provided voltage reference, the input type, and > +the input polarity. The input range allowed to an ADC channel is needed to > +determine the scale factor and offset needed to obtain the measured value in > +real-world units (millivolts for voltage measurement, milliamps for current > +measurement, etc.). > + > +Elaborated designs may have nonlinear characteristics or integrated components > +(such as amplifiers and reference buffers) that might also have to be considered > +to derive the allowed input range for an ADC. For clarity, the sections below > +assume the input range only depends on the provided voltage references, input > +type, and input polarity. > + > +There are three general types of ADC inputs (single-ended, differential, > +pseudo-differential) and two possible polarities (unipolar, bipolar). I think we actually need to say there are 3 polarities as well. As you will see, my comments below in the psudo-diffential sections was assuming the definition of "bipolar" is "true bipolar" where it means that the input volage can be negative. However, it is now clear to me that you have been writing this with the perspective of bipolar only being "pseduo bipolar" where uniploar would be limited to between a positive common mode voltage and the max voltage (VREF) an bipolar would still be limited to a positive voltage but range from 0V to VREF. For ease of understanding, I think we should start with giving diagrams and examples of true bipolar, then move on to the more complex and less common examples of pseudo bipolar. So I will leave my commemts below (mostly with regards to psudo-differential) as I first wrote them, but take it to mean that we are both right and these are two diffent cases that need to be considered. > The input > +type (single-ended, differential, pseudo-differential) is one channel > +characteristic, and is completely independent of the polarity (unipolar, > +bipolar) aspect. A comprehensive article about ADC input types (on which this > +doc is heavily based on) can be found at > +https://www.analog.com/en/resources/technical-articles/sar-adc-input-types.html. > + > +1.1 Single-ended channels > +------------------------- > + > +Single-ended channels digitize the analog input voltage relative to ground and > +can be either unipolar or bipolar. > + > +1.1.1 Single-ended Unipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +:: > + > + ---------- VREF ------------- > + ´ ` ´ ` _____________ > + / \ / \ / | > + / \ / \ --- < IN ADC | > + \ / \ / \ | > + `-´ `-´ \ VREF | > + -------- GND (0V) ----------- +-----------+ > + ^ > + | > + External VREF > + > +The input voltage to a **single-ended unipolar** channel is allowed to swing > +from GND to VREF (where VREF is a voltage reference with electrical potential > +higher than system ground). The maximum input voltage is also called VFS > +(full-scale input voltage), with VFS being determined by VREF. The voltage > +reference may be provided from an external supply or derived from the chip power > +source. > + > +A single-ended unipolar channel could be described in device tree like the > +following example:: We should probably mention somewhere that channel@ nodes are only needed for chips that don't have uniform inputs. > + > + adc@0 { > + ... > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0>; If a chip has mixed differential and single-ended, single-channel could also be needed here in the case where reg is an arbitrary number and doesn't match the input pin number. > + }; > + }; > + > +See ``Documentation/devicetree/bindings/iio/adc/adc.yaml`` for the complete > +documentation of ADC specific device tree properties. > + > + > +1.1.2 Single-ended Bipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +:: > + > + ---------- +VREF ------------ > + ´ ` ´ ` _____________________ > + / \ / \ / | > + / \ / \ --- < IN ADC | > + \ / \ / \ | > + `-´ `-´ \ +VREF -VREF | > + ---------- -VREF ------------ +-------------------+ > + ^ ^ > + | | > + External +VREF ------+ External -VREF > + > +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 > +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. > + > +Here's an example device tree description of a single-ended bipolar channel:: > + > + adc@0 { > + ... > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0>; > + bipolar; > + }; > + }; > + > +1.2 Differential channels > +------------------------- > + > +A differential voltage measurement digitizes the voltage level at the positive > +input (IN+) relative to the negative input (IN-) over the -VREF to +VREF span. > +In other words, a differential channel measures the potential difference between > +IN+ and IN-, which is often denoted by the IN+ - IN- formula. > + > +1.2.1 Differential Bipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +:: > + > + -------- +VREF ------ > + ´ ` ´ ` +-------------------+ > + / \ / \ / / | > + `-´ `-´ --- < IN+ | > + -------- -VREF ------ | | > + | ADC | > + -------- +VREF ------ | | > + ´ ` ´ ` --- < IN- | > + \ / \ / \ \ +VREF -VREF | > + `-´ `-´ +-------------------+ > + -------- -VREF ------ ^ ^ > + | +---- External -VREF > + External +VREF > + > +The analog signals to **differential bipolar** inputs are also allowed to swing > +from -VREF to +VREF. If -VREF is below system GND, these are also called > +differential true bipolar inputs. > + > +Device tree example of a differential bipolar channel:: > + > + adc@0 { > + ... > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0>; > + bipolar; > + diff-channels = <0 1>; > + }; > + }; > + > +In the ADC driver, ``differential = 1`` is set into ``struct iio_chan_spec`` for > +the channel. See ``include/linux/iio/iio.h`` for more information. > + > +1.2.2 Differential Unipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +For **differential unipolar** channels, the analog voltage at the positive input > +must also be higher than the voltage at the negative input. Thus, the actual > +input range allowed to a differential unipolar channel is IN- to +VREF. Because > +IN+ is allowed to swing with the measured analog signal and the input setup must > +guarantee IN+ will not go below IN- (nor IN- will raise above IN+), most > +differential unipolar channel setups have IN- fixed to a known voltage that does > +not fall within the voltage range expected for the measured signal. This leads > +to a setup that is equivalent to a pseudo-differential channel. Thus, > +differential unipolar channels are actually pseudo-differential unipolar > +channels. I don't think this is equevent to pseudo-differential unipolar. That one has a common mode voltage supply on the negative input. This one has a full range signal on the negative input. This is the diagram I was expecting here. :: -------- VREF ------- ´ ` ´ ` +-------------------+ / \ / \ / / | `-´ `-´ --- < IN+ | ------ GND (0V) ----- | | | ADC | -------- VREF ------- | | ´ ` ´ ` --- < IN- | \ / \ / \ \ VREF | `-´ `-´ +-------------------+ ------ GND (0V) ----- ^ | External VREF > + > +1.3 Pseudo-differential Channels > +-------------------------------- > + > +There is a third ADC input type which is called pseudo-differential or > +single-ended to differential configuration. A pseudo-differential channel is > +similar to a differential channel in that it also measures IN+ relative to IN-. > +However, unlike differential channels, the negative input is limited to a narrow > +(taken as constant) voltage range while only IN+ is allowed to swing. A > +pseudo-differential channel can be made out from a differential pair of inputs > +by restricting the negative input to a known voltage while allowing only the > +positive input to swing. Aside from that, some parts have a COM pin that allows > +single-ended inputs to be referenced to a common-mode voltage, making them > +pseudo-differential channels. Often, the common mode input voltage > +can be nicely described in the device tree as a voltage regulator (e.g. > +``com-supply``) since it is basically a constant voltage source. > + > +1.3.1 Pseudo-differential Unipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +:: > + > + -------- +VREF ------ +-------------------+ > + ´ ` ´ ` / | > + / \ / \ / --- < IN+ | > + `-´ `-´ | | > + --------- IN- ------- | ADC | The bottom rail should be GND (or -VREF), not IN-. Making it GND in the diagram would be consistent with the other unipolar diagrams and reflect most typical cases. I think the counterexample you gave of AD4170-4 is the unusual case rather than the typical case. FWIW, when I was first learning this stuff, I didn't really understand bipolar vs. unipolar until I saw diagrams that showed 0V as the bottom rail for unipolar and negative VREF as the bottom rail for bipolar. Even if it isn't strictly true in all cases, seeing the pattern was more helpful. Hard to say if most other people think like me though. :-) > + | | > + Common-mode voltage --> --- < IN- | > + \ +VREF | > + +-------------------+ > + ^ > + | > + External +VREF > + > +A **pseudo-differential unipolar** input has the limitations a differential > +unipolar channel would have, meaning the analog voltage to the positive input > +IN+ must stay within IN- to +VREF. As above, this is not strictyly true. On the chips I have worked with (at least two out of three different families), IN- is VREF/2 and IN+ can go from GND to VREF. > The fixed voltage to IN- is sometimes called > +common-mode voltage and it must be within -VREF to +VREF as would be expected > +from the signal to any differential channel negative input. > + > +In pseudo-differential configuration, the voltage measured from IN+ is not > +relative to GND (as it would be for a single-ended channel) but to IN-, which This is a bit confusing because you say not relative to GND, but then below, you say that GND can be connected to IN-, so it would be relative to GND. So might want to qualify this GND is different than external common mode voltage GND. > +causes the measurement to always be offset by IN- volts. To allow applications > +to calculate IN+ voltage with respect to system ground, the IIO channel should > +provide an ``_offset`` sysfs attribute to report the channel offset to user > +space. In cases where IN- (or the common mode voltage applied to IN-) is GND, > +the ``_offset`` attribute may be omitted. > + > +Device tree example for pseudo-differential unipolar channel:: > + > + adc@0 { > + ... > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0>; > + single-channel = <0>; > + common-mode-channel = <1>; > + }; > + }; > + > +Do not set ``differential`` in the channel ``iio_chan_spec`` struct of > +pseudo-differential channels. Some might appritate an explanatin why here since it is odd to have differental in the name "pseudo-differential". I.e. because we consider the negative pin a voltage supply rather than a second anolog input. > + > +1.3.2 Pseudo-differential Bipolar Channels > +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > + > +:: > + > + -------- +VREF ------ +-------------------+ > + ´ ` ´ ` / | > + / \ / \ / --- < IN+ | > + `-´ `-´ | | > + -------- -VREF ------ | ADC | > + | | > + Common-mode voltage --> --- < IN- | > + \ +VREF -VREF | > + +-------------------+ > + ^ ^ > + | +---- External -VREF > + External +VREF > + > +A **pseudo-differential bipolar** input is not limited by the level at IN- but > +it will be limited to -VREF or to GND on the lower end of the input range If it was limited to GND, then it would be uniploar. It is only bipolar if -VREF is less than 0V. > +depending on the particular ADC. Similar to their unipolar counter parts, > +pseudo-differential bipolar channels may define an ``_offset`` attribute to > +provide the read offset relative to GND. > + > +Device tree example for pseudo-differential bipolar channel:: > + > + adc@0 { > + ... > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0>; > + bipolar; > + single-channel = <0>; > + common-mode-channel = <1>; > + }; > + }; > + > +Again, the ``differential`` field of ``struct iio_chan_spec`` is not set for > +pseudo-differential channels. > diff --git a/Documentation/iio/index.rst b/Documentation/iio/index.rst > index 5710f5b9e958..48b88044d7cb 100644 > --- a/Documentation/iio/index.rst > +++ b/Documentation/iio/index.rst > @@ -7,6 +7,7 @@ Industrial I/O > .. toctree:: > :maxdepth: 1 > > + iio_adc > iio_configfs > iio_devbuf > iio_dmabuf_api > > base-commit: 9b75dd1b7d6b98699a104c6b1eec0c8817e5fd4b