[PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Krzysztof Kozlowski]

Emails to Beniamin Bia bounce with no such address so remove him from
maintainers.

Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Michael Hennerich <Michael.Hennerich@analog.com>
Cc: Jonathan Cameron <jic23@kernel.org>
Cc: linux-iio <linux-iio@vger.kernel.org>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 MAINTAINERS | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8107b3d5d6df..523ac1602b62 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -967,9 +967,8 @@ F:	Documentation/devicetree/bindings/iio/dac/ad5758.txt
 F:	drivers/iio/dac/ad5758.c
 
 ANALOG DEVICES INC AD7091R5 DRIVER
-M:	Beniamin Bia <beniamin.bia@analog.com>
 L:	linux-iio@vger.kernel.org
-S:	Supported
+S:	Orphan
 W:	http://ez.analog.com/community/linux-device-drivers
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
 F:	drivers/iio/adc/ad7091r5.c
@@ -1000,7 +999,6 @@ F:	drivers/iio/adc/ad7292.c
 
 ANALOG DEVICES INC AD7606 DRIVER
 M:	Michael Hennerich <Michael.Hennerich@analog.com>
-M:	Beniamin Bia <beniamin.bia@analog.com>
 L:	linux-iio@vger.kernel.org
 S:	Supported
 W:	http://ez.analog.com/community/linux-device-drivers
@@ -1068,7 +1066,6 @@ F:	drivers/iio/imu/adis16475.c
 F:	Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
 
 ANALOG DEVICES INC ADM1177 DRIVER
-M:	Beniamin Bia <beniamin.bia@analog.com>
 M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-hwmon@vger.kernel.org
 S:	Supported
@@ -1136,7 +1133,6 @@ W:	http://ez.analog.com/community/linux-device-drivers
 F:	drivers/dma/dma-axi-dmac.c
 
 ANALOG DEVICES INC HMC425A DRIVER
-M:	Beniamin Bia <beniamin.bia@analog.com>
 M:	Michael Hennerich <michael.hennerich@analog.com>
 L:	linux-iio@vger.kernel.org
 S:	Supported
@@ -16537,7 +16533,6 @@ F:	drivers/staging/rtl8712/
 
 STAGING - SEPS525 LCD CONTROLLER DRIVERS
 M:	Michael Hennerich <michael.hennerich@analog.com>
-M:	Beniamin Bia <beniamin.bia@analog.com>
 L:	linux-fbdev@vger.kernel.org
 S:	Supported
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
-- 
2.17.1

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Andy Shevchenko]

On Thu, Aug 27, 2020 at 11:17 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> Emails to Beniamin Bia bounce with no such address so remove him from
> maintainers.
>
> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>

I guess Reported-by suits better... But okay.

> Cc: Michael Hennerich <Michael.Hennerich@analog.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: linux-iio <linux-iio@vger.kernel.org>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  MAINTAINERS | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8107b3d5d6df..523ac1602b62 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -967,9 +967,8 @@ F:  Documentation/devicetree/bindings/iio/dac/ad5758.txt
>  F:     drivers/iio/dac/ad5758.c
>
>  ANALOG DEVICES INC AD7091R5 DRIVER
> -M:     Beniamin Bia <beniamin.bia@analog.com>
>  L:     linux-iio@vger.kernel.org
> -S:     Supported
> +S:     Orphan
>  W:     http://ez.analog.com/community/linux-device-drivers
>  F:     Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
>  F:     drivers/iio/adc/ad7091r5.c
> @@ -1000,7 +999,6 @@ F: drivers/iio/adc/ad7292.c
>
>  ANALOG DEVICES INC AD7606 DRIVER
>  M:     Michael Hennerich <Michael.Hennerich@analog.com>
> -M:     Beniamin Bia <beniamin.bia@analog.com>
>  L:     linux-iio@vger.kernel.org
>  S:     Supported
>  W:     http://ez.analog.com/community/linux-device-drivers
> @@ -1068,7 +1066,6 @@ F:        drivers/iio/imu/adis16475.c
>  F:     Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
>
>  ANALOG DEVICES INC ADM1177 DRIVER
> -M:     Beniamin Bia <beniamin.bia@analog.com>
>  M:     Michael Hennerich <Michael.Hennerich@analog.com>
>  L:     linux-hwmon@vger.kernel.org
>  S:     Supported
> @@ -1136,7 +1133,6 @@ W:        http://ez.analog.com/community/linux-device-drivers
>  F:     drivers/dma/dma-axi-dmac.c
>
>  ANALOG DEVICES INC HMC425A DRIVER
> -M:     Beniamin Bia <beniamin.bia@analog.com>
>  M:     Michael Hennerich <michael.hennerich@analog.com>
>  L:     linux-iio@vger.kernel.org
>  S:     Supported
> @@ -16537,7 +16533,6 @@ F:      drivers/staging/rtl8712/
>
>  STAGING - SEPS525 LCD CONTROLLER DRIVERS
>  M:     Michael Hennerich <michael.hennerich@analog.com>
> -M:     Beniamin Bia <beniamin.bia@analog.com>
>  L:     linux-fbdev@vger.kernel.org
>  S:     Supported
>  F:     Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> --
> 2.17.1
>


-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Jonathan Cameron]

On Fri, 28 Aug 2020 11:19:21 +0300
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:

> On Thu, Aug 27, 2020 at 11:17 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >
> > Emails to Beniamin Bia bounce with no such address so remove him from
> > maintainers.
> >
> > Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>  
> 
> I guess Reported-by suits better... But okay.

As mentioned below, I'm not keen on marking a driver as orphan
when it is covered by a catch all.

drivers/iio/*/ad*

Let's find out what the Analog team want to do about these.

Jonathan


> 
> > Cc: Michael Hennerich <Michael.Hennerich@analog.com>
> > Cc: Jonathan Cameron <jic23@kernel.org>
> > Cc: linux-iio <linux-iio@vger.kernel.org>
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >  MAINTAINERS | 7 +------
> >  1 file changed, 1 insertion(+), 6 deletions(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 8107b3d5d6df..523ac1602b62 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -967,9 +967,8 @@ F:  Documentation/devicetree/bindings/iio/dac/ad5758.txt
> >  F:     drivers/iio/dac/ad5758.c
> >
> >  ANALOG DEVICES INC AD7091R5 DRIVER
> > -M:     Beniamin Bia <beniamin.bia@analog.com>
> >  L:     linux-iio@vger.kernel.org
> > -S:     Supported
> > +S:     Orphan

Given it should be covered by the catch all for Analog devices IIO drivers,
either we should confirm if it should move to someone else at Analog, or
if we should just drop specifically listing this one.
Listing it as Orphan when they are good at supporting their drivers
may give the wrong impression.

+CC Alex to make sure people at Analog notice :)

Jonathan

> >  W:     http://ez.analog.com/community/linux-device-drivers
> >  F:     Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
> >  F:     drivers/iio/adc/ad7091r5.c
> > @@ -1000,7 +999,6 @@ F: drivers/iio/adc/ad7292.c
> >
> >  ANALOG DEVICES INC AD7606 DRIVER
> >  M:     Michael Hennerich <Michael.Hennerich@analog.com>
> > -M:     Beniamin Bia <beniamin.bia@analog.com>
> >  L:     linux-iio@vger.kernel.org
> >  S:     Supported
> >  W:     http://ez.analog.com/community/linux-device-drivers
> > @@ -1068,7 +1066,6 @@ F:        drivers/iio/imu/adis16475.c
> >  F:     Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> >
> >  ANALOG DEVICES INC ADM1177 DRIVER
> > -M:     Beniamin Bia <beniamin.bia@analog.com>
> >  M:     Michael Hennerich <Michael.Hennerich@analog.com>
> >  L:     linux-hwmon@vger.kernel.org
> >  S:     Supported
> > @@ -1136,7 +1133,6 @@ W:        http://ez.analog.com/community/linux-device-drivers
> >  F:     drivers/dma/dma-axi-dmac.c
> >
> >  ANALOG DEVICES INC HMC425A DRIVER
> > -M:     Beniamin Bia <beniamin.bia@analog.com>
> >  M:     Michael Hennerich <michael.hennerich@analog.com>
> >  L:     linux-iio@vger.kernel.org
> >  S:     Supported
> > @@ -16537,7 +16533,6 @@ F:      drivers/staging/rtl8712/
> >
> >  STAGING - SEPS525 LCD CONTROLLER DRIVERS
> >  M:     Michael Hennerich <michael.hennerich@analog.com>
> > -M:     Beniamin Bia <beniamin.bia@analog.com>
> >  L:     linux-fbdev@vger.kernel.org
> >  S:     Supported
> >  F:     Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> > --
> > 2.17.1
> >  
> 
> 

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Krzysztof Kozlowski]

On Sat, 29 Aug 2020 at 16:54, Jonathan Cameron <jic23@kernel.org> wrote:

(...)

> > >  ANALOG DEVICES INC AD7091R5 DRIVER
> > > -M:     Beniamin Bia <beniamin.bia@analog.com>
> > >  L:     linux-iio@vger.kernel.org
> > > -S:     Supported
> > > +S:     Orphan
>
> Given it should be covered by the catch all for Analog devices IIO drivers,
> either we should confirm if it should move to someone else at Analog, or
> if we should just drop specifically listing this one.
> Listing it as Orphan when they are good at supporting their drivers
> may give the wrong impression.
>
> +CC Alex to make sure people at Analog notice :)

Sure, good point. I wanted to start the discussion so the interested
people might appear.

Best regards,
Krzysztof

RE: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Ardelean, Alexandru]

[yes, I know, bad-email format, but I wanted this to come from my work email]

Apologies also for the delay here. Things pile-up on my side and I defer things a bit.

Talked to Michael Hennerich about this [since he's the more senior contact at Analog].
We can replace the email from Beniamin Bia with Michael's.
Or, we can remove the "Orphan" blocks and just have the catch-all "drivers/iio/*/ad*" cover this driver and others that were upstreamed by Beniamin.

Either option is fine from us.

-----Original Message-----
From: Krzysztof Kozlowski <krzk@kernel.org> 
Sent: Saturday, August 29, 2020 5:58 PM
To: Jonathan Cameron <jic23@kernel.org>
Cc: Andy Shevchenko <andy.shevchenko@gmail.com>; Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Hennerich, Michael <Michael.Hennerich@analog.com>; linux-iio <linux-iio@vger.kernel.org>; Ardelean, Alexandru <alexandru.Ardelean@analog.com>
Subject: Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

On Sat, 29 Aug 2020 at 16:54, Jonathan Cameron <jic23@kernel.org> wrote:

(...)

> > >  ANALOG DEVICES INC AD7091R5 DRIVER
> > > -M:     Beniamin Bia <beniamin.bia@analog.com>
> > >  L:     linux-iio@vger.kernel.org
> > > -S:     Supported
> > > +S:     Orphan
>
> Given it should be covered by the catch all for Analog devices IIO 
> drivers, either we should confirm if it should move to someone else at 
> Analog, or if we should just drop specifically listing this one.
> Listing it as Orphan when they are good at supporting their drivers 
> may give the wrong impression.
>
> +CC Alex to make sure people at Analog notice :)

Sure, good point. I wanted to start the discussion so the interested people might appear.

Best regards,
Krzysztof

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Andy Shevchenko]

On Wed, Sep 2, 2020 at 8:56 AM Ardelean, Alexandru
<alexandru.Ardelean@analog.com> wrote:
>
> [yes, I know, bad-email format, but I wanted this to come from my work email]
>
> Apologies also for the delay here. Things pile-up on my side and I defer things a bit.
>
> Talked to Michael Hennerich about this [since he's the more senior contact at Analog].
> We can replace the email from Beniamin Bia with Michael's.
> Or, we can remove the "Orphan" blocks and just have the catch-all "drivers/iio/*/ad*" cover this driver and others that were upstreamed by Beniamin.
>
> Either option is fine from us.

As a participant of activity to put MAINTAINERS in some order, I
prefer to have similar blocks be squashed.
If there will be a dedicated maintainer for a certain driver in the
future, we can always split into separate sections.

So, something like
ANALOG DEVICES ADC (I guess it makes sense to have some grouping still)
...
F: .../ad1234
F: .../ad2345

In one section (no need to have wildcards right now).

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Jonathan Cameron]

On Wed, 2 Sep 2020 10:49:04 +0300
Andy Shevchenko <andy.shevchenko@gmail.com> wrote:

> On Wed, Sep 2, 2020 at 8:56 AM Ardelean, Alexandru
> <alexandru.Ardelean@analog.com> wrote:
> >
> > [yes, I know, bad-email format, but I wanted this to come from my work email]
> >
> > Apologies also for the delay here. Things pile-up on my side and I defer things a bit.
> >
> > Talked to Michael Hennerich about this [since he's the more senior contact at Analog].
> > We can replace the email from Beniamin Bia with Michael's.
> > Or, we can remove the "Orphan" blocks and just have the catch-all "drivers/iio/*/ad*" cover this driver and others that were upstreamed by Beniamin.
> >
> > Either option is fine from us.  
> 
> As a participant of activity to put MAINTAINERS in some order, I
> prefer to have similar blocks be squashed.
> If there will be a dedicated maintainer for a certain driver in the
> future, we can always split into separate sections.
> 
> So, something like
> ANALOG DEVICES ADC (I guess it makes sense to have some grouping still)
> ...
> F: .../ad1234
> F: .../ad2345
> 
> In one section (no need to have wildcards right now).
> 

ANALOG DEVICES INC IIO DRIVERS
M:	Lars-Peter Clausen <lars@metafoo.de>
M:	Michael Hennerich <Michael.Hennerich@analog.com>
S:	Supported
W:	http://wiki.analog.com/
W:	http://ez.analog.com/community/linux-device-drivers
F:	Documentation/ABI/testing/sysfs-bus-iio-frequency-ad9523
F:	Documentation/ABI/testing/sysfs-bus-iio-frequency-adf4350
F:	drivers/iio/*/ad*
F:	drivers/iio/adc/ltc249*
F:	drivers/staging/iio/*/ad*
X:	drivers/iio/*/adjd*

Already exists.  We could potentially clean that up.

For the block that started this thread, we can just drop it.

Jonathan

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Krzysztof Kozlowski]

On Wed, 2 Sep 2020 at 15:28, Jonathan Cameron
<Jonathan.Cameron@huawei.com> wrote:
>
> On Wed, 2 Sep 2020 10:49:04 +0300
> Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
>
> > On Wed, Sep 2, 2020 at 8:56 AM Ardelean, Alexandru
> > <alexandru.Ardelean@analog.com> wrote:
> > >
> > > [yes, I know, bad-email format, but I wanted this to come from my work email]
> > >
> > > Apologies also for the delay here. Things pile-up on my side and I defer things a bit.
> > >
> > > Talked to Michael Hennerich about this [since he's the more senior contact at Analog].
> > > We can replace the email from Beniamin Bia with Michael's.
> > > Or, we can remove the "Orphan" blocks and just have the catch-all "drivers/iio/*/ad*" cover this driver and others that were upstreamed by Beniamin.
> > >
> > > Either option is fine from us.
> >
> > As a participant of activity to put MAINTAINERS in some order, I
> > prefer to have similar blocks be squashed.
> > If there will be a dedicated maintainer for a certain driver in the
> > future, we can always split into separate sections.
> >
> > So, something like
> > ANALOG DEVICES ADC (I guess it makes sense to have some grouping still)
> > ...
> > F: .../ad1234
> > F: .../ad2345
> >
> > In one section (no need to have wildcards right now).
> >
>
> ANALOG DEVICES INC IIO DRIVERS
> M:      Lars-Peter Clausen <lars@metafoo.de>
> M:      Michael Hennerich <Michael.Hennerich@analog.com>
> S:      Supported
> W:      http://wiki.analog.com/
> W:      http://ez.analog.com/community/linux-device-drivers
> F:      Documentation/ABI/testing/sysfs-bus-iio-frequency-ad9523
> F:      Documentation/ABI/testing/sysfs-bus-iio-frequency-adf4350
> F:      drivers/iio/*/ad*
> F:      drivers/iio/adc/ltc249*
> F:      drivers/staging/iio/*/ad*
> X:      drivers/iio/*/adjd*
>
> Already exists.  We could potentially clean that up.
>
> For the block that started this thread, we can just drop it.

I can send a follow up to merge/remove the entries.

Best regards,
Krzysztof

Re: [PATCH] MAINTAINERS: Remove bouncing email of Beniamin Bia

[Jonathan Cameron]

On Wed, 2 Sep 2020 15:35:45 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> On Wed, 2 Sep 2020 at 15:28, Jonathan Cameron
> <Jonathan.Cameron@huawei.com> wrote:
> >
> > On Wed, 2 Sep 2020 10:49:04 +0300
> > Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> >  
> > > On Wed, Sep 2, 2020 at 8:56 AM Ardelean, Alexandru
> > > <alexandru.Ardelean@analog.com> wrote:  
> > > >
> > > > [yes, I know, bad-email format, but I wanted this to come from my work email]
> > > >
> > > > Apologies also for the delay here. Things pile-up on my side and I defer things a bit.
> > > >
> > > > Talked to Michael Hennerich about this [since he's the more senior contact at Analog].
> > > > We can replace the email from Beniamin Bia with Michael's.
> > > > Or, we can remove the "Orphan" blocks and just have the catch-all "drivers/iio/*/ad*" cover this driver and others that were upstreamed by Beniamin.
> > > >
> > > > Either option is fine from us.  
> > >
> > > As a participant of activity to put MAINTAINERS in some order, I
> > > prefer to have similar blocks be squashed.
> > > If there will be a dedicated maintainer for a certain driver in the
> > > future, we can always split into separate sections.
> > >
> > > So, something like
> > > ANALOG DEVICES ADC (I guess it makes sense to have some grouping still)
> > > ...
> > > F: .../ad1234
> > > F: .../ad2345
> > >
> > > In one section (no need to have wildcards right now).
> > >  
> >
> > ANALOG DEVICES INC IIO DRIVERS
> > M:      Lars-Peter Clausen <lars@metafoo.de>
> > M:      Michael Hennerich <Michael.Hennerich@analog.com>
> > S:      Supported
> > W:      http://wiki.analog.com/
> > W:      http://ez.analog.com/community/linux-device-drivers
> > F:      Documentation/ABI/testing/sysfs-bus-iio-frequency-ad9523
> > F:      Documentation/ABI/testing/sysfs-bus-iio-frequency-adf4350
> > F:      drivers/iio/*/ad*
> > F:      drivers/iio/adc/ltc249*
> > F:      drivers/staging/iio/*/ad*
> > X:      drivers/iio/*/adjd*
> >
> > Already exists.  We could potentially clean that up.
> >
> > For the block that started this thread, we can just drop it.  
> 
> I can send a follow up to merge/remove the entries.
> 
> Best regards,
> Krzysztof

Great thanks

Jonathan