Re: [PATCH 2/6] iio: backend: Add support for CRC

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The newly added `crc_enable` and `crc_disable` function pointers in `struct iio_backend_ops` are missing their corresponding descriptions in the kernel-doc block.
--

--- Patch [2]: [PATCH 2/6] iio: backend: Add support for CRC ---
commit 98a340052f70f8ab707e22920844ec1a36237e6a
Author: Janani Sunil <janani.sunil@analog.com>

iio: backend: Add support for CRC

This commit adds backend operations to enable or disable Cyclic Redundancy
Check (CRC) processing. This allows the backend to handle data integrity
verification for samples transmitted over the interface.

[ ... ]

> diff --git a/include/linux/iio/backend.h b/include/linux/iio/backend.h
> index 3f95ed1fdf9eb..197cf1ad5e68e 100644
> --- a/include/linux/iio/backend.h
> +++ b/include/linux/iio/backend.h
> @@ -189,6 +189,8 @@ struct iio_backend_ops {
>  			       enum iio_backend_filter_type type);
>  	int (*interface_data_align)(struct iio_backend *back, u32 timeout_us);
>  	int (*num_lanes_set)(struct iio_backend *back, unsigned int num_lanes);
> +	int (*crc_enable)(struct iio_backend *back);
> +	int (*crc_disable)(struct iio_backend *back);

[Severity: Low]
Are the kernel-doc descriptions for crc_enable and crc_disable missing from
the documentation block above struct iio_backend_ops?

>  	int (*ddr_enable)(struct iio_backend *back);
>  	int (*ddr_disable)(struct iio_backend *back);
>  	int (*data_stream_enable)(struct iio_backend *back);

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260709-ad7768-driver-v1-0-44e1194fd96a@analog.com?part=2