Linux IIO development
 help / color / mirror / Atom feed
From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: dlechner@baylibre.com, nuno.sa@analog.com, lars@metafoo.de,
	andy@kernel.org, linux-iio@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 2/5] MAINTAINERS: iio: adi: consolidate entries
Date: Tue, 05 May 2026 08:00:23 +0530	[thread overview]
Message-ID: <AB6B7E29-83E2-4BE8-AF74-1DE8358527E6@gmail.com> (raw)
In-Reply-To: <20260504172457.4d7c7390@jic23-huawei>



On 4 May 2026 9:54:57 pm IST, Jonathan Cameron <jic23@kernel.org> wrote:
>On Fri,  1 May 2026 00:36:39 +0530
>Sanjay Chitroda <sanjayembeddedse@gmail.com> wrote:
>
>> From: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>> 
>> Remove redundant per-driver entries already covered by wildcard paths
>> under ANALOG DEVICES INC IIO DRIVERS for drivers/iio/adc/ad* drivers.
>> 
>> Keep entries where specific maintainers exist.
>
>This seems to be where it's gone wrong - if this were followed
>I'd be fine with it.  Actually also applies where there is
>a specific reviewer listed - not sure we have any where there
>isn't also a maintainer though.
>
>> 
>> Suggested-by: Jonathan Cameron <jonathan.cameron@huawei.com>
>> Acked-by: Nuno Sá <nuno.sa@analog.com>
>
>Substantial enough change from anything that I think Nuno would
>have tagged that you should have dropped his Acked-by:
>
Sure, I will drop this tag from all patches in this series.

Although tag was added in all patches of series as he mentioned Acked-by on your proposal of catch all common entry.

>> Signed-off-by: Sanjay Chitroda <sanjayembeddedse@gmail.com>
>> ---
>>  MAINTAINERS | 158 +---------------------------------------------------
>>  1 file changed, 3 insertions(+), 155 deletions(-)
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 34b67d010e87..bd14e15f04c9 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -451,17 +451,6 @@ W:	http://wiki.analog.com/AD7142
>>  W:	https://ez.analog.com/linux-software-drivers
>>  F:	drivers/input/misc/ad714x.c
>>  
>> -AD738X ADC DRIVER (AD7380/1/2/4)
>> -M:	Michael Hennerich <michael.hennerich@analog.com>
>> -M:	Nuno Sá <nuno.sa@analog.com>
>Specific maintainer. Why is it going?

I'm assuming that to avoid de-bounce issues from *@analog.com domain, we are adding linux@analog.com mailing list as primary contact. with that in mind, this series drops all entries which match the wildcard and have analog domain maintainer or reviewer while keeping the rest as unchanged.
>
>> -R:	David Lechner <dlechner@baylibre.com>

Additionally, When David is list as reviewer in individual driver entries, I consider as redundant part; All IIO subsystem drivers/iio* already covered by top level entry, which includes Andy and David as reviewer.

Summary of changes in series:
- analog domain specific entries are dropped and catch with linux@analog.com under ADI IIO section.
- Non analog devices reviewers and maintainer is unchanged for each driver entry.

Further, I also agree with adding Nuno Sá as a maintainer in the top-level ADI IIO entry, so that all drivers are maintained via the mailing list with his oversight.

>> -S:	Supported
>> -W:	https://wiki.analog.com/resources/tools-software/linux-drivers/iio-adc/ad738x
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7380.yaml
>> -F:	Documentation/iio/ad7380.rst
>> -F:	drivers/iio/adc/ad7380.c
>> -
>>  AD7476 ADC DRIVER FOR VARIOUS SIMPLE 1-CHANNEL SPI ADCs
>>  M:	Matti Vaittinen <mazziesaccount@gmail.com>
>>  S:	Maintained
>> @@ -481,16 +470,6 @@ W:	http://wiki.analog.com/AD7879
>>  W:	https://ez.analog.com/linux-software-drivers
>>  F:	drivers/input/touchscreen/ad7879.c
>>  
>> -AD7944 ADC DRIVER (AD7944/AD7985/AD7986)
>> -M:	Michael Hennerich <michael.hennerich@analog.com>
>> -M:	Nuno Sá <nuno.sa@analog.com>
>> -R:	David Lechner <dlechner@baylibre.com>
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7944.yaml
>> -F:	Documentation/iio/ad7944.rst
>> -F:	drivers/iio/adc/ad7944.c
>> -
>>  ADAFRUIT MINI I2C GAMEPAD
>>  M:	Anshul Dalal <anshulusr@gmail.com>
>>  L:	linux-input@vger.kernel.org
>> @@ -1415,15 +1394,6 @@ F:	Documentation/devicetree/bindings/iio/dac/adi,ad3552r.yaml
>>  F:	Documentation/iio/ad3552r.rst
>>  F:	drivers/iio/dac/ad3552r.c
>>  
>> -ANALOG DEVICES INC AD4000 DRIVER
>> -M:	Marcelo Schmitt <marcelo.schmitt@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4000.yaml
>> -F:	Documentation/iio/ad4000.rst
>> -F:	drivers/iio/adc/ad4000.c
>> -
>>  AD4030 ADC DRIVER (AD4030-24/AD4630-16/AD4630-24/AD4632-16/AD4632-24)
>>  M:	Michael Hennerich <michael.hennerich@analog.com>
>>  M:	Nuno Sá <nuno.sa@analog.com>
>> @@ -1435,78 +1405,6 @@ F:	Documentation/devicetree/bindings/iio/adc/adi,ad4030.yaml
>>  F:	Documentation/iio/ad4030.rst
>>  F:	drivers/iio/adc/ad4030.c
>>  
>> -ANALOG DEVICES INC AD4062 DRIVER
>> -M:	Jorge Marques <jorge.marques@analog.com>
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4062.yaml
>> -F:	Documentation/iio/ad4062.rst
>> -F:	drivers/iio/adc/ad4062.c
>> -
>> -ANALOG DEVICES INC AD4080 DRIVER
>> -M:	Antoniu Miclaus <antoniu.miclaus@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4080.yaml
>> -F:	drivers/iio/adc/ad4080.c
>> -
>> -ANALOG DEVICES INC AD4130 DRIVER
>> -M:	Cosmin Tanislav <cosmin.tanislav@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4130.yaml
>> -F:	drivers/iio/adc/ad4130.c
>> -
>> -ANALOG DEVICES INC AD4134 DRIVER
>> -M:	Marcelo Schmitt <marcelo.schmitt@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4134.yaml
>> -F:	drivers/iio/adc/ad4134.c
>> -
>> -ANALOG DEVICES INC AD4170-4 DRIVER
>> -M:	Marcelo Schmitt <marcelo.schmitt@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4170-4.yaml
>> -F:	drivers/iio/adc/ad4170-4.c
>> -
>> -ANALOG DEVICES INC AD4691 DRIVER
>> -M:	Radu Sabau <radu.sabau@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4691.yaml
>> -F:	drivers/iio/adc/ad4691.c
>> -F:	drivers/iio/adc/ad4691.rst
>> -
>> -ANALOG DEVICES INC AD4695 DRIVER
>> -M:	Michael Hennerich <michael.hennerich@analog.com>
>> -M:	Nuno Sá <nuno.sa@analog.com>
>> -R:	David Lechner <dlechner@baylibre.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4695.yaml
>> -F:	Documentation/iio/ad4695.rst
>> -F:	drivers/iio/adc/ad4695.c
>> -F:	include/dt-bindings/iio/adc/adi,ad4695.h
>> -
>> -ANALOG DEVICES INC AD4851 DRIVER
>> -M:	Sergiu Cuciurean <sergiu.cuciurean@analog.com>
>> -M:	Dragos Bogdan <dragos.bogdan@analog.com>
>> -R:	Antoniu Miclaus <antoniu.miclaus@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad4851.yaml
>> -F:	drivers/iio/adc/ad4851.c
>> -
>>  ANALOG DEVICES INC AD5706R DRIVER
>>  M:	Alexis Czezar Torreno <alexisczezar.torreno@analog.com>
>>  L:	linux-iio@vger.kernel.org
>> @@ -1515,31 +1413,6 @@ W:	https://ez.analog.com/linux-software-drivers
>>  F:	Documentation/devicetree/bindings/iio/dac/adi,ad5706r.yaml
>>  F:	drivers/iio/dac/ad5706r.c
>>  
>> -ANALOG DEVICES INC AD7091R DRIVER
>> -M:	Marcelo Schmitt <marcelo.schmitt@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	http://ez.analog.com/community/linux-device-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7091r*
>> -F:	drivers/iio/adc/ad7091r*
>> -
>> -ANALOG DEVICES INC AD7191 DRIVER
>> -M:	Alisa-Dariana Roman <alisa.roman@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7191.yaml
>> -F:	Documentation/iio/ad7191.rst
>> -F:	drivers/iio/adc/ad7191.c
>> -
>> -ANALOG DEVICES INC AD7192 DRIVER
>> -M:	Alisa-Dariana Roman <alisa.roman@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
>> -F:	drivers/iio/adc/ad7192.c
>> -
>>  ANALOG DEVICES INC AD7292 DRIVER
>>  M:	Marcelo Schmitt <marcelo.schmitt1@gmail.com>
>>  L:	linux-iio@vger.kernel.org
>> @@ -1584,15 +1457,6 @@ F:	Documentation/devicetree/bindings/iio/adc/adi,ad7625.yaml
>>  F:	Documentation/iio/ad7625.rst
>>  F:	drivers/iio/adc/ad7625.c
>>  
>> -ANALOG DEVICES INC AD7768-1 DRIVER
>> -M:	Michael Hennerich <Michael.Hennerich@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7768-1.yaml
>> -F:	drivers/iio/adc/ad7768-1.c
>> -F:	include/dt-bindings/iio/adc/adi,ad7768-1.h
>> -
>>  ANALOG DEVICES INC AD7780 DRIVER
>>  M:	Michael Hennerich <Michael.Hennerich@analog.com>
>>  M:	Renato Lui Geh <renatogeh@gmail.com>
>> @@ -1611,16 +1475,6 @@ W:	https://ez.analog.com/linux-software-drivers
>>  F:	Documentation/devicetree/bindings/iio/amplifiers/adi,ad8366.yaml
>>  F:	drivers/iio/amplifiers/ad8366.c
>>  
>> -ANALOG DEVICES INC AD9467 DRIVER
>> -M:	Michael Hennerich <Michael.Hennerich@analog.com>
>> -M:	Nuno Sa <nuno.sa@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/ABI/testing/debugfs-iio-ad9467
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad9467.yaml
>> -F:	drivers/iio/adc/ad9467.c
>> -
>>  ANALOG DEVICES INC AD8460 DRIVER
>>  M:	Mariel Tinaco <Mariel.Tinaco@analog.com>
>>  L:	linux-iio@vger.kernel.org
>> @@ -1654,14 +1508,6 @@ W:	https://ez.analog.com/linux-software-drivers
>>  F:	Documentation/devicetree/bindings/iio/amplifiers/adi,ada4250.yaml
>>  F:	drivers/iio/amplifiers/ada4250.c
>>  
>> -ANALOG DEVICES INC ADE9000 DRIVER
>> -M:	Antoniu Miclaus <antoniu.miclaus@analog.com>
>> -L:	linux-iio@vger.kernel.org
>> -S:	Supported
>> -W:	https://ez.analog.com/linux-software-drivers
>> -F:	Documentation/devicetree/bindings/iio/adc/adi,ade9000.yaml
>> -F:	drivers/iio/adc/ade9000.c
>> -
>>  ANALOG DEVICES INC ADF4377 DRIVER
>>  M:	Antoniu Miclaus <antoniu.miclaus@analog.com>
>>  L:	linux-iio@vger.kernel.org
>> @@ -1866,12 +1712,14 @@ L:	linux@analog.com
>>  S:	Supported
>>  W:	http://wiki.analog.com/
>>  W:	https://ez.analog.com/linux-software-drivers
>> +F:	Documentation/ABI/testing/debugfs-iio-ad9467
>> +F:	Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130
>>  F:	Documentation/ABI/testing/sysfs-bus-iio-frequency-ad9523
>>  F:	Documentation/ABI/testing/sysfs-bus-iio-frequency-adf4350
>>  F:	Documentation/devicetree/bindings/iio/*/adi,*
>>  F:	Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
>>  F:	Documentation/devicetree/bindings/iio/adc/lltc,ltc2497.yaml
>> -F:	Documentation/iio/ad7606.rst
>> +F:	Documentation/iio/ad*.rst
>>  F:	drivers/iio/*/ad*
>>  F:	drivers/iio/adc/ltc249*
>>  F:	drivers/iio/amplifiers/hmc425a.c
>

  reply	other threads:[~2026-05-05  8:51 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-30 19:06 [PATCH v4 0/5] MAINTAINERS: update iio adi entry and drop redundant entry Sanjay Chitroda
2026-04-30 19:06 ` [PATCH v4 1/5] MAINTAINERS: Update Analog Devices IIO drivers entry Sanjay Chitroda
2026-05-01  9:05   ` Nuno Sá
2026-05-04  7:35     ` Andy Shevchenko
2026-05-04 16:22       ` Jonathan Cameron
2026-05-07  8:09         ` Nuno Sá
2026-04-30 19:06 ` [PATCH v4 2/5] MAINTAINERS: iio: adi: consolidate entries Sanjay Chitroda
2026-05-04 16:24   ` Jonathan Cameron
2026-05-05  2:30     ` Sanjay Chitroda [this message]
2026-05-05  9:04       ` Andy Shevchenko
2026-05-05 10:54         ` Jonathan Cameron
2026-05-05 11:55           ` Sanjay Chitroda
2026-05-05 12:41           ` Andy Shevchenko
2026-05-05 16:15             ` Jonathan Cameron
2026-04-30 19:06 ` [PATCH v4 3/5] " Sanjay Chitroda
2026-04-30 19:06 ` [PATCH v4 4/5] " Sanjay Chitroda
2026-04-30 19:06 ` [PATCH v4 5/5] " Sanjay Chitroda
2026-05-04  7:40 ` [PATCH v4 0/5] MAINTAINERS: update iio adi entry and drop redundant entry Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AB6B7E29-83E2-4BE8-AF74-1DE8358527E6@gmail.com \
    --to=sanjayembeddedse@gmail.com \
    --cc=andy@kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=jic23@kernel.org \
    --cc=lars@metafoo.de \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nuno.sa@analog.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox