| Message ID | 20200827201707.27075-1-krzk@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | MAINTAINERS: Remove bouncing email of Beniamin Bia | expand |
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 >
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 > > > >
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
[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
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).
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
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
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
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
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(-)