Re: [PATCH] iio: adc: ad7476: Support ROHM BU79100G

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 30 May 2025 07:40:08 +0300
Matti Vaittinen <mazziesaccount@gmail.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@gmail.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 },
>  	{ }
>  };