Re: [PATCH v2 1/2] iio: adc: ti-ads112c14: add support for I2C CRC8

[David Lechner <dlechner@baylibre.com>]

On 7/24/26 10:55 AM, David Lechner (TI) wrote:
> Add support for I2C CRC8 to the TI ADS112C14 ADC driver. This verifies
> data integrity of all I2C transactions with the device.
> 
> For now, it is always enabled, but it could be made optional in the
> future if needed (e.g. for higher-speed data acquisition).
> 
> Signed-off-by: David Lechner (TI) <dlechner@baylibre.com>
> ---
> v2 changes:
> * Rebased on iio/testing.
> ---
>  drivers/iio/adc/Kconfig        |   1 -
>  drivers/iio/adc/ti-ads112c14.c | 149 ++++++++++++++++++++++++++++++++++++++---
>  2 files changed, 140 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig
> index 642c42b1fae7..4ed6f01386bf 100644
> --- a/drivers/iio/adc/Kconfig
> +++ b/drivers/iio/adc/Kconfig
> @@ -1802,7 +1802,6 @@ config TI_ADS112C14
>  	tristate "Texas Instruments ADS112C14/ADS122C14"
>  	depends on I2C

Sashiko correctly points out we should select CRC8 here since
it is always enabled to avoid random config build failures.

>  	select REGMAP
> -	select REGMAP_I2C
>  	help
>  	  If you say yes here you get support for Texas Instruments ADS112C14,
>  	  ADS122C14 ADC chips.