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

David Lechner <[email protected]>
Newsgroups org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
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) <[email protected]>
> ---
> 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.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.