Re: [PATCH v1 01/13] iio: Fix typo in vendor name
From: Marcelo Schmitt
Date: Fri Sep 04 2026 - 15:42:59 EST
On 09/03, Andy Shevchenko wrote:
> On Wed, Sep 02, 2026 at 02:21:34PM -0300, Marcelo Schmitt wrote:
>
> > ---
> > Now
> > find drivers/iio/ -type f | xargs grep -i "Analog Device "
> > finds no leftover occurrence of such typo.
> >
> > Also,
> > grep -nrIi drivers/iio/ -e "Analog Device" | wc -l
> > and
> > grep -nrIi drivers/iio/ -e "Analog Devices" | wc -l
> > both count the same number of lines.
>
> Why not `git grep ...`?
>
Could be. I'm just more used to non-git grep.
> ...
>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
>
> But you also want to fix possessive suffix usage
>
> $ git grep -n 'Analog Device[^s]'
>
> Documentation/iio/ade9000.rst:7:This driver supports Analog Device's ADE9000 and ADE9078 energy measurement
> Documentation/iio/adis16475.rst:7:This driver supports Analog Device's IMUs on SPI bus.
> Documentation/iio/adis16480.rst:7:This driver supports Analog Device's IMUs on SPI bus.
> Documentation/iio/adis16550.rst:7:This driver supports Analog Device's IMUs on SPI bus.
> Documentation/iio/adxl313.rst:7:This driver supports Analog Device's ADXL313 on SPI/I2C bus.
> Documentation/iio/adxl345.rst:7:This driver supports Analog Device's ADXL345/375 on SPI/I2C bus.
> Documentation/iio/adxl380.rst:7:This driver supports Analog Device's ADXL380/382 on SPI/I2C bus.
> drivers/iio/adc/Kconfig:108: tristate "Analog Device AD4130 ADC Driver"
> drivers/iio/adc/Kconfig:124: tristate "Analog Device AD4134 ADC Driver"
> drivers/iio/adc/Kconfig:136: tristate "Analog Device AD4170-4 ADC Driver"
> drivers/iio/adc/Kconfig:167: tristate "Analog Device AD4695 ADC Driver"
> drivers/iio/adc/Kconfig:182: tristate "Analog Device AD4851 DAS Driver"
> drivers/iio/adc/Kconfig:350: tristate "Analog Device AD7405 ADC Driver"
> drivers/iio/dac/ad3552r.c:699:MODULE_DESCRIPTION("Analog Device AD3552R DAC");
Ah, thanks for bringing this suggestion. Will adjust the possessive use as well.
Thanks for suggestions to other patches too. Will apply them for v2.
Thanks,
Marcelo