From: Andy Shevchenko <andriy.shevchenko@intel.com>
To: Sanjay Chitroda <sanjayembeddedse@gmail.com>
Cc: Jonathan Cameron <jic23@kernel.org>,
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, 5 May 2026 12:04:40 +0300 [thread overview]
Message-ID: <afmyqE6-XcT-SRt3@ashevche-desk.local> (raw)
In-Reply-To: <AB6B7E29-83E2-4BE8-AF74-1DE8358527E6@gmail.com>
On Tue, May 05, 2026 at 08:00:23AM +0530, Sanjay Chitroda wrote:
> 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:
...
> >> -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.
But it's removing people behind their will! You really should do this
case-by-case. Some of the people who left the company might still want to
follow the driver or even maintain it. What you should do is also pay attention
on the Supported vs. Maintained which makes a huge difference on the
responsibility and financial side of the Subject.
Btw, Have you consulted with your Legal department on all this?
> >> -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.
No, you should give a good explanation why in each case. Yes, you may combine
the cases when people left the company, but you also need to
- consult with your Legal
- consult with those who left on what they wish
None of them seems done so far. So, again BIG NAK from me on this series!
Rework them and even part that drops the entries that already have the same
maintainers as current ADI level ones (assuming adding Nuno in a separate
prerequisite patch) is good to go.
> - 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
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2026-05-05 9:04 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
2026-05-05 9:04 ` Andy Shevchenko [this message]
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=afmyqE6-XcT-SRt3@ashevche-desk.local \
--to=andriy.shevchenko@intel.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 \
--cc=sanjayembeddedse@gmail.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