Re: [PATCH] iio: adc: fix typos found by codespell

From: Andy Shevchenko

Date: Tue Feb 24 2026 - 09:16:11 EST


On Tue, Feb 24, 2026 at 04:22:30PM +0400, Giorgi Tchankvetadze wrote:
> Fix various spelling mistakes in comments and error messages
> across drivers/iio/adc/, found by running codespell.

Thanks, my comments below.

...

> v2: expand to fix all typos across drivers/iio/adc/ (Andy)

Do create a new email thread for each new version.
Use proper versioning: `git format-patch -v<N> ...`
<N> = 2 this should be.

...

>
> - /* dummy read: restore default CH0 settin */
> + /* dummy read: restore default CH0 setting */
> return spi_sync(st->spi, &st->msg[AD7887_CH0]);

Please, check the code and perhaps choose a plural form (depending on what is
there in the given register).

...

> #define AD7923_PM_MODE_AS (1) /* auto shutdown */
> #define AD7923_PM_MODE_FS (2) /* full shutdown */
> #define AD7923_PM_MODE_OPS (3) /* normal operation */
> -#define AD7923_SEQUENCE_OFF (0) /* no sequence fonction */
> +#define AD7923_SEQUENCE_OFF (0) /* no sequence function */
> #define AD7923_SEQUENCE_PROTECT (2) /* no interrupt write cycle */
> #define AD7923_SEQUENCE_ON (3) /* continuous sequence */
>
> @@ -39,7 +39,7 @@
> #define AD7923_CHANNEL_WRITE(channel) ((channel) << 6) /* write channel */
> #define AD7923_SEQUENCE_WRITE(sequence) ((((sequence) & 1) << 3) \
> + (((sequence) & 2) << 9))
> - /* write sequence fonction */
> + /* write sequence function */

Nothing is unveiling that the author of that code is French-speaking :-)

...

> * @chip_info: chip specific data
> * @spi: SPI device structure
> - * @vref_mv: voltage reference value in miliVolts
> + * @vref_mv: voltage reference value in milliVolts

millivolts

With units there is a good article on how to spell them in different occasions.

...

Have you run it against Kconfig in the same folder (and subfolders if exists)?

--
With Best Regards,
Andy Shevchenko