Re: [PATCH v10 07/17] iio: test: add kunit tests for channel prefix naming generation

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The Kconfig help text incorrectly claims that the tests are compiled into the IIO core module.
--

--- Patch [7]: [PATCH v10 07/17] iio: test: add kunit tests for channel prefix naming generation ---
commit 1dfc71650388b9628c03d727921e09ce430c3fe7
Author: Rodrigo Alencar <rodrigo.alencar@analog.com>

iio: test: add kunit tests for channel prefix naming generation

This commit adds a KUnit test suite for the __iio_chan_prefix_emit() helper.
It exercises various channel prefix generation combinations including shared
modes, indexing, and error paths.

> diff --git a/drivers/iio/test/Kconfig b/drivers/iio/test/Kconfig
> index 4fc17dd0dcd77..7d2c29cc583b9 100644
> --- a/drivers/iio/test/Kconfig
> +++ b/drivers/iio/test/Kconfig
[ ... ]
> +config IIO_CHANNEL_PREFIX_KUNIT_TEST
> +	tristate "Test IIO channel prefix" if !KUNIT_ALL_TESTS
> +	depends on KUNIT && IIO
> +	default KUNIT_ALL_TESTS
> +	help
> +	  Build unit tests for __iio_chan_prefix_emit(), the helper that
> +	  builds IIO sysfs attribute name prefixes from an iio_chan_spec.
> +	  The tests are compiled into the IIO core module.

[Severity: Low]
This isn't a bug, but is this help text accurate regarding how the tests
are compiled? 

Looking at drivers/iio/test/Makefile, the test is built as a separate
object:

obj-$(CONFIG_IIO_CHANNEL_PREFIX_KUNIT_TEST) += iio-test-channel-prefix.o

rather than being compiled directly into the IIO core module itself.
Additionally, the iio-test-channel-prefix.c file explicitly imports the
EXPORTED_FOR_KUNIT_TESTING namespace, which further confirms it acts as
a standalone module testing the exported core symbols.

Could this description be updated to avoid confusion about where the test
resides?

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260729-ad9910-iio-driver-v10-0-aa1f6167729e@analog.com?part=7