Received-SPF: pass (google.com: domain of linux-kernel+bounces-208245-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <adc6cba9-2e79-475f-9c24-039fe9d3345d@baylibre.com>
Date: Mon, 10 Jun 2024 08:40:56 -0500
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v7 0/9] Add support for AD411x
To: dumitru.ceclan@analog.com
Cc: Lars-Peter Clausen <lars@metafoo.de>,
 Michael Hennerich <Michael.Hennerich@analog.com>,
 Jonathan Cameron <jic23@kernel.org>, Rob Herring <robh@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Conor Dooley <conor+dt@kernel.org>, linux-iio@vger.kernel.org,
 devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
 Dumitru Ceclan <mitrutzceclan@gmail.com>,
 Conor Dooley <conor.dooley@microchip.com>, Nuno Sa <nuno.sa@analog.com>,
 Jonathan Cameron <Jonathan.Cameron@huawei.com>
References: <20240607-ad4111-v7-0-97e3855900a0@analog.com>
Content-Language: en-US
From: David Lechner <dlechner@baylibre.com>
In-Reply-To: <20240607-ad4111-v7-0-97e3855900a0@analog.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 6/7/24 9:53 AM, Dumitru Ceclan via B4 Relay wrote:
> This patch series adds support for the Analog Devices AD4111, AD4112,
>  AD4114, AD4115, AD4116 within the existing AD7173 driver.
> 
>   The AD411X family encompasses a series of low power, low noise, 24-bit,
> sigma-delta analog-to-digital converters that offer a versatile range of
> specifications. They integrate an analog front end suitable for processing
> fully differential/single-ended and bipolar voltage inputs.
> 
> Particularities of the models:
> - All ADCs have inputs with a precision voltage divider with a division
> ratio of 10.
> - AD4116 has 5 low level inputs without a voltage divider.
> - AD4111 and AD4112 support current inputs (0 mA to 20 mA) using a 50ohm
> shunt resistor.
> 
> Discussions from this patch series have concluded with:
> -Datasheets mention single-ended and pseudo differential capabilities by
>  the means of connecting the negative input of a differential pair (IN-)
>  to a constant voltage supply and letting the positive input fluctuate.
>  This is not a special operating mode, it is a capability of the
>  differential channels to also measure such signals.
> 
> -Single-ended and pseudo differential do not need any specific
>  configuration and cannot be differentiated from differential usage by
>  the driver side =>
> 	offer adi,channel-type attribute to flag the usage of the channel
> 
> -VINCOM is described as a dedicated pin for single-ended channels but as
>  seen in AD4116, it is a normal input connected to the cross-point
>  multiplexer (VIN10, VINCOM (single-ended or differential pair)).
>  This does not mean full functionality in any configuration:
>  AD4111:"If any two voltage inputs are paired in a configuration other
>  than what is described in this data sheet, the accuracy of the device
>  cannot be guaranteed".
> 
> -ADCIN15 input pin from AD4116 is specified as the dedicated pin for
>  pseudo-differential but from the datasheet it results that this pin is
>  also able to measure single-ended and fully differential channels
>  ("ADCIN11, ADCIN15. (pseudo differential or differential pair)";
>   "An example is to connect the ADCIN15 pin externally to the AVSS
>    pin in a single-ended configuration")
> 
>  As such, detecting the type of usage of a channel is not possible and
> will be the responsibility of the user to specify.
>  If the user has connected a non 0V (in regards to AVSS) supply to
> the negative input pin of a channel in a pseudo differential
> configuration, the offset of the measurement from AVSS will not be known
> from the driver and will need to be measured by other means.
> 
> Datasheets:
> https://www.analog.com/media/en/technical-documentation/data-sheets/AD4111.pdf
> https://www.analog.com/media/en/technical-documentation/data-sheets/AD4112.pdf
> https://www.analog.com/media/en/technical-documentation/data-sheets/AD4114.pdf
> https://www.analog.com/media/en/technical-documentation/data-sheets/AD4115.pdf
> https://www.analog.com/media/en/technical-documentation/data-sheets/AD4116.pdf
> 
> This series depends on patches:
> (iio: adc: ad7173: Use device_for_each_child_node_scoped() to simplify error paths.)
> https://lore.kernel.org/all/20240330190849.1321065-6-jic23@kernel.org
> (dt-bindings: iio: adc: Add single-channel property)
> https://lore.kernel.org/linux-iio/20240514120222.56488-5-alisa.roman@analog.com/
> 
> And patch series:
> (AD7173 fixes)
> https://lore.kernel.org/all/20240521-ad7173-fixes-v1-0-8161cc7f3ad1@analog.com/
> 
> Signed-off-by: Dumitru Ceclan <mitrutzceclan@gmail.com>
> ---

Reviewed-by: David Lechner <dlechner@baylibre.com>