linux-kernel - Re: [PATCH] iio: adc: ad7476: Support ROHM BU79100G

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] [day] [month] [year] [list]

Message-ID: <20250531164258.47f0d1a2@jic23-huawei>
Date: Sat, 31 May 2025 16:42:58 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Matti Vaittinen <mazziesaccount@...il.com>
Cc: Matti Vaittinen <matti.vaittinen@...rohmeurope.com>, Lars-Peter Clausen
 <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com>,
 linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] iio: adc: ad7476: Support ROHM BU79100G

On Fri, 30 May 2025 07:40:08 +0300
Matti Vaittinen <mazziesaccount@...il.com> wrote:

> ROHM BU79100G is a 12-bit, single channel ADC. From the software point
> of view it is identical to the TI's ADS7866. Support reading ADC
> measurements using the ad7476.c
> 
> Signed-off-by: Matti Vaittinen <mazziesaccount@...il.com>
Applied.
> ---
> 
> Please, see the relevant discussion (as to why an ID for a fallback is
> added) in:
> https://lore.kernel.org/all/f8ec547a-5924-4563-aa1d-dde8227844fa@gmail.com/
> 
> This patch was based on v6.15-rc1. I can rebase if needed.
> 
>  drivers/iio/adc/ad7476.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/iio/adc/ad7476.c b/drivers/iio/adc/ad7476.c
> index 37b0515cf4fc..d48ee8f1d547 100644
> --- a/drivers/iio/adc/ad7476.c
> +++ b/drivers/iio/adc/ad7476.c
> @@ -435,6 +435,13 @@ static const struct spi_device_id ad7476_id[] = {
>  	{ "ads7866", ID_ADS7866 },
>  	{ "ads7867", ID_ADS7867 },
>  	{ "ads7868", ID_ADS7868 },
> +	/*
> +	 * The ROHM BU79100G is identical to the TI's ADS7866 from the software
> +	 * point of view. The binding document mandates the ADS7866 to be
> +	 * marked as a fallback for the BU79100G, but we still need the SPI ID
> +	 * here to make the module loading work.
> +	 */
> +	{ "bu79100g", ID_ADS7866 },
>  	{ "ltc2314-14", ID_LTC2314_14 },
>  	{ }
>  };