diff mbox series

MAINTAINERS: remove Stefan Popa's email

Message ID 20200317143336.6098-1-alexandru.ardelean@analog.com (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: remove Stefan Popa's email | expand

Commit Message

Alexandru Ardelean March 17, 2020, 2:33 p.m. UTC
The email is no longer active. This change removes Stefan's email from the
MAINTAINERS list and replaces it with Michael Hennerich's.

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
---
 MAINTAINERS | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

Comments

Andy Shevchenko March 21, 2020, 7:48 p.m. UTC | #1
On Tue, Mar 17, 2020 at 04:33:36PM +0200, Alexandru Ardelean wrote:
> The email is no longer active. This change removes Stefan's email from the
> MAINTAINERS list and replaces it with Michael Hennerich's.
> 

Are you planning to fix entries as parse-maintainers.pl does?
(Maybe in a separate change, but also why not here)
Joe Perches March 21, 2020, 8:09 p.m. UTC | #2
On Sat, 2020-03-21 at 21:48 +0200, Andy Shevchenko wrote:
> On Tue, Mar 17, 2020 at 04:33:36PM +0200, Alexandru Ardelean wrote:
> > The email is no longer active. This change removes Stefan's email from the
> > MAINTAINERS list and replaces it with Michael Hennerich's.
> > 
> 
> Are you planning to fix entries as parse-maintainers.pl does?
> (Maybe in a separate change, but also why not here)

I think perhaps you should not ask people to use parse-maintainers
on specific sections anymore as it's rather confusing to use and
it's best done as a one-shot by Linus one day.  I hope soon.

Linus, are you going to apply this patch to parse-maintainers
and one day run and commit the large change to MAINTAINERS?

https://lore.kernel.org/lkml/4d5291fa3fb4962b1fa55e8fd9ef421ef0c1b1e5.camel@perches.com/

$ ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS

Current diffstat for -next would be:

$ git diff --shortstat MAINTAINERS
 1 file changed, 2749 insertions(+), 2749 deletions(-)

btw: your current tree is still missing this patch from
-next to fix a broken pattern in 'TI VPE/CAL DRIVERS'

commit d44535cb14c981dea66e565edf1df1df4b9e823b
Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date:   Tue Jan 28 11:58:28 2020 -0300

    media: MAINTAINERS: Sort entries in database for TI VPE/CAL
    
    Run parse-maintainers.pl and choose TI VPE/CAL record. Fix it accordingly.
    
    Note, this is urgent fix, without which parse-maintainers.pl throws
    an exception:
    
    Odd non-pattern line '  Documentation/devicetree/bindings/media/ti,cal.yaml
    ' for 'TI VPE/CAL DRIVERS' at scripts/parse-maintainers.pl line 147, <$file> line 16770.
    
    Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
    Acked-by: Benoit Parrot <bparrot@ti.com>
    Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
    Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Andy Shevchenko March 21, 2020, 8:18 p.m. UTC | #3
On Sat, Mar 21, 2020 at 10:11 PM Joe Perches <joe@perches.com> wrote:
>
> On Sat, 2020-03-21 at 21:48 +0200, Andy Shevchenko wrote:
> > On Tue, Mar 17, 2020 at 04:33:36PM +0200, Alexandru Ardelean wrote:
> > > The email is no longer active. This change removes Stefan's email from the
> > > MAINTAINERS list and replaces it with Michael Hennerich's.
> > >
> >
> > Are you planning to fix entries as parse-maintainers.pl does?
> > (Maybe in a separate change, but also why not here)
>
> I think perhaps you should not ask people to use parse-maintainers
> on specific sections anymore as it's rather confusing to use and
> it's best done as a one-shot by Linus one day.  I hope soon.

Why not? The benefits are
 - people will know about the tool
 - the particular sections will be fixed since they are touching the data base

And it doesn't prevent Linus or anybody to run the script in the future.

Actually it will teach people ahead to use the tool then in the future
we won't see such drastic deviation in ordering.
Alexandru Ardelean March 22, 2020, 9:09 a.m. UTC | #4
On Sat, 2020-03-21 at 22:18 +0200, Andy Shevchenko wrote:
> [External]
> 
> On Sat, Mar 21, 2020 at 10:11 PM Joe Perches <joe@perches.com> wrote:
> > On Sat, 2020-03-21 at 21:48 +0200, Andy Shevchenko wrote:
> > > On Tue, Mar 17, 2020 at 04:33:36PM +0200, Alexandru Ardelean wrote:
> > > > The email is no longer active. This change removes Stefan's email from
> > > > the
> > > > MAINTAINERS list and replaces it with Michael Hennerich's.
> > > > 
> > > 
> > > Are you planning to fix entries as parse-maintainers.pl does?
> > > (Maybe in a separate change, but also why not here)
> > 
> > I think perhaps you should not ask people to use parse-maintainers
> > on specific sections anymore as it's rather confusing to use and
> > it's best done as a one-shot by Linus one day.  I hope soon.
> 
> Why not? The benefits are
>  - people will know about the tool
>  - the particular sections will be fixed since they are touching the data base
> 
> And it doesn't prevent Linus or anybody to run the script in the future.
> 
> Actually it will teach people ahead to use the tool then in the future
> we won't see such drastic deviation in ordering.
> 

i guess this means it's a noop for me now
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 8fa40c3eb72a..72e8960c4f4f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -552,7 +552,7 @@  F:	drivers/input/misc/adxl34x.c
 F:	Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
 
 ADXL372 THREE-AXIS DIGITAL ACCELEROMETER DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <michael.hennerich@analog.com>
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
 F:	drivers/iio/accel/adxl372.c
@@ -900,7 +900,7 @@  F:	drivers/net/ethernet/amd/xgbe/
 F:	arch/arm64/boot/dts/amd/amd-seattle-xgbe*.dtsi
 
 ANALOG DEVICES INC AD5686 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-pm@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
@@ -908,7 +908,7 @@  F:	drivers/iio/dac/ad5686*
 F:	drivers/iio/dac/ad5696*
 
 ANALOG DEVICES INC AD5758 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-iio@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
@@ -924,7 +924,7 @@  F:	drivers/iio/adc/ad7091r5.c
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
 
 ANALOG DEVICES INC AD7124 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-iio@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
@@ -948,7 +948,7 @@  F:	drivers/iio/adc/ad7292.c
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
 
 ANALOG DEVICES INC AD7606 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 M:	Beniamin Bia <beniamin.bia@analog.com>
 L:	linux-iio@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
@@ -957,7 +957,7 @@  F:	drivers/iio/adc/ad7606.c
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
 
 ANALOG DEVICES INC AD7768-1 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-iio@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
@@ -1018,7 +1018,7 @@  F:	drivers/hwmon/adm1177.c
 F:	Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml
 
 ANALOG DEVICES INC ADP5061 DRIVER
-M:	Stefan Popa <stefan.popa@analog.com>
+M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-pm@vger.kernel.org
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported
@@ -1078,7 +1078,6 @@  F:	drivers/dma/dma-axi-dmac.c
 ANALOG DEVICES INC IIO DRIVERS
 M:	Lars-Peter Clausen <lars@metafoo.de>
 M:	Michael Hennerich <Michael.Hennerich@analog.com>
-M:	Stefan Popa <stefan.popa@analog.com>
 W:	http://wiki.analog.com/
 W:	http://ez.analog.com/community/linux-device-drivers
 S:	Supported