linux-kernel - Re: [PATCH v2 0/2] iio: adc: ad799x: reference voltage capability

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: <aef22644-2869-4888-8fc6-f14859917dcd@baylibre.com>
Date: Thu, 7 Aug 2025 10:47:29 -0500
From: David Lechner <dlechner@...libre.com>
To: Stefano Manni <stefano.manni@...il.com>, lars@...afoo.de,
 Michael.Hennerich@...log.com, jic23@...nel.org, nuno.sa@...log.com,
 andy@...nel.org
Cc: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] iio: adc: ad799x: reference voltage capability

On 8/7/25 2:48 AM, Stefano Manni wrote:
> This patch series refactors 6b104e7895ab16b9b7f466c5f2ca282b87f661e8
> in order to add the capability of the chip to have an
> external reference voltage into the chip_info struct.
> And so avoid ugly conditional checks on the chip id.
> 
> In addition the AD7994 is marked to have the external
> reference voltage as well.
> 
> Changes in v2 compared to v1 [1]:
> * remove has_vref from the chips that do not support it,
>   rely on the default false value
> * remove useless message "Supplied reference not supported"
>   shown for all the chips with has_vref = false
> * refactor check on regulator being err or zero
> * add external reference to ad7994 as oneliner
> 
> [1] https://lore.kernel.org/linux-iio/20250806090158.117628-1-stefano.manni@gmail.com/
> 
> Stefano Manni (2):
>   iio: adc: ad799x: add reference voltage capability to chip_info
>   iio: adc: ad799x: add reference voltage to ad7994
> 
>  drivers/iio/adc/ad799x.c | 30 +++++++++++++-----------------
>  1 file changed, 13 insertions(+), 17 deletions(-)
> 

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