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

From: David Lechner

Date: Fri Jul 24 2026 - 14:35:24 EST


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@xxxxxxxxxxxx>
> ---
> 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.