diff mbox series

[v3,1/2] MAINTAINERS: Consolidate Analog Devices IIO entries and remove Beniamin Bia

Message ID 20200903181926.5606-1-krzk@kernel.org (mailing list archive)
State New, archived
Headers show
Series [v3,1/2] MAINTAINERS: Consolidate Analog Devices IIO entries and remove Beniamin Bia | expand

Commit Message

Krzysztof Kozlowski Sept. 3, 2020, 6:19 p.m. UTC 
Emails to Beniamin Bia bounce with no such address so remove him from
maintainers.  After this removal, many entries for Analog Devices Inc
IIO drivers look exactly the same so consolidate them.

Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

---

Changes since v2:
1. Use wildcard for dt-bindings,
2. Add Andy's review.

Changes since v1:
1. Consolidate IIO entries
---
 MAINTAINERS | 55 +++--------------------------------------------------
 1 file changed, 3 insertions(+), 52 deletions(-)

Comments

Andy Shevchenko Sept. 3, 2020, 7:41 p.m. UTC | #1 
On Thu, Sep 3, 2020 at 9:19 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> Emails to Beniamin Bia bounce with no such address so remove him from
> maintainers.  After this removal, many entries for Analog Devices Inc
> IIO drivers look exactly the same so consolidate them.
>
> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

...

>  ANALOG DEVICES INC IIO DRIVERS
>  M:     Lars-Peter Clausen <lars@metafoo.de>
>  M:     Michael Hennerich <Michael.Hennerich@analog.com>
> @@ -1152,8 +1101,11 @@ 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:     Documentation/devicetree/bindings/iio/*/adi,*
> +F:     Documentation/devicetree/bindings/iio/dac/ad5758.txt
>  F:     drivers/iio/*/ad*
>  F:     drivers/iio/adc/ltc249*
> +F:     drivers/iio/amplifiers/hmc425a.c
>  F:     drivers/staging/iio/*/ad*
>  X:     drivers/iio/*/adjd*

>  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

It's not related to this series, but I'm wondering why the above
binding is attached to a staging driver...
Alexandru Ardelean Sept. 4, 2020, 8:19 a.m. UTC | #2 
On Thu, Sep 3, 2020 at 10:42 PM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Thu, Sep 3, 2020 at 9:19 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >
> > Emails to Beniamin Bia bounce with no such address so remove him from
> > maintainers.  After this removal, many entries for Analog Devices Inc
> > IIO drivers look exactly the same so consolidate them.
> >
> > Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> > Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
> > Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>
> ...
>
> >  ANALOG DEVICES INC IIO DRIVERS
> >  M:     Lars-Peter Clausen <lars@metafoo.de>
> >  M:     Michael Hennerich <Michael.Hennerich@analog.com>
> > @@ -1152,8 +1101,11 @@ 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:     Documentation/devicetree/bindings/iio/*/adi,*
> > +F:     Documentation/devicetree/bindings/iio/dac/ad5758.txt
> >  F:     drivers/iio/*/ad*
> >  F:     drivers/iio/adc/ltc249*
> > +F:     drivers/iio/amplifiers/hmc425a.c
> >  F:     drivers/staging/iio/*/ad*
> >  X:     drivers/iio/*/adjd*
>
> >  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
>
> It's not related to this series, but I'm wondering why the above
> binding is attached to a staging driver...

Looks like a copy+paste error.
Good catch.

>
> --
> With Best Regards,
> Andy Shevchenko
Jonathan Cameron Sept. 6, 2020, 2:06 p.m. UTC | #3 
On Thu,  3 Sep 2020 20:19:25 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:

> Emails to Beniamin Bia bounce with no such address so remove him from
> maintainers.  After this removal, many entries for Analog Devices Inc
> IIO drivers look exactly the same so consolidate them.
> 
> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

As I'd assume a more specific binding always overrides a catch all,
this has the effect of giving Lars and Michael responsibility
for a few things they didn't previously cover.  If the two
of them are fine with it, than that's good, but I'd ideally
like an Ack from Lars.

(we have an indirect one from Michael via Alexandru).

Jonathan


> 
> ---
> 
> Changes since v2:
> 1. Use wildcard for dt-bindings,
> 2. Add Andy's review.
> 
> Changes since v1:
> 1. Consolidate IIO entries
> ---
>  MAINTAINERS | 55 +++--------------------------------------------------
>  1 file changed, 3 insertions(+), 52 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 8107b3d5d6df..4a120aec90a2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -950,38 +950,6 @@ S:	Supported
>  F:	arch/arm64/boot/dts/amd/amd-seattle-xgbe*.dtsi
>  F:	drivers/net/ethernet/amd/xgbe/
>  
> -ANALOG DEVICES INC AD5686 DRIVER
> -M:	Michael Hennerich <Michael.Hennerich@analog.com>
> -L:	linux-pm@vger.kernel.org
> -S:	Supported
> -W:	http://ez.analog.com/community/linux-device-drivers
> -F:	drivers/iio/dac/ad5686*
> -F:	drivers/iio/dac/ad5696*
> -
> -ANALOG DEVICES INC AD5758 DRIVER
> -M:	Michael Hennerich <Michael.Hennerich@analog.com>
> -L:	linux-iio@vger.kernel.org
> -S:	Supported
> -W:	http://ez.analog.com/community/linux-device-drivers
> -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
> -W:	http://ez.analog.com/community/linux-device-drivers
> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
> -F:	drivers/iio/adc/ad7091r5.c
> -
> -ANALOG DEVICES INC AD7124 DRIVER
> -M:	Michael Hennerich <Michael.Hennerich@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,ad7124.yaml
> -F:	drivers/iio/adc/ad7124.c
> -
>  ANALOG DEVICES INC AD7192 DRIVER
>  M:	Alexandru Tachici <alexandru.tachici@analog.com>
>  L:	linux-iio@vger.kernel.org
> @@ -998,15 +966,6 @@ W:	http://ez.analog.com/community/linux-device-drivers
>  F:	Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
>  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
> -F:	Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> -F:	drivers/iio/adc/ad7606.c
> -
>  ANALOG DEVICES INC AD7768-1 DRIVER
>  M:	Michael Hennerich <Michael.Hennerich@analog.com>
>  L:	linux-iio@vger.kernel.org
> @@ -1068,7 +1027,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
> @@ -1135,15 +1093,6 @@ S:	Supported
>  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
> -W:	http://ez.analog.com/community/linux-device-drivers
> -F:	Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
> -F:	drivers/iio/amplifiers/hmc425a.c
> -
>  ANALOG DEVICES INC IIO DRIVERS
>  M:	Lars-Peter Clausen <lars@metafoo.de>
>  M:	Michael Hennerich <Michael.Hennerich@analog.com>
> @@ -1152,8 +1101,11 @@ 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:	Documentation/devicetree/bindings/iio/*/adi,*
> +F:	Documentation/devicetree/bindings/iio/dac/ad5758.txt
>  F:	drivers/iio/*/ad*
>  F:	drivers/iio/adc/ltc249*
> +F:	drivers/iio/amplifiers/hmc425a.c
>  F:	drivers/staging/iio/*/ad*
>  X:	drivers/iio/*/adjd*
>  
> @@ -16537,7 +16489,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
Lars-Peter Clausen Sept. 6, 2020, 3:57 p.m. UTC | #4 
On 9/6/20 4:06 PM, Jonathan Cameron wrote:
> On Thu,  3 Sep 2020 20:19:25 +0200
> Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
>> Emails to Beniamin Bia bounce with no such address so remove him from
>> maintainers.  After this removal, many entries for Analog Devices Inc
>> IIO drivers look exactly the same so consolidate them.
>>
>> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>> Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
>> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> As I'd assume a more specific binding always overrides a catch all,
> this has the effect of giving Lars and Michael responsibility
> for a few things they didn't previously cover.  If the two
> of them are fine with it, than that's good, but I'd ideally
> like an Ack from Lars.

Acked-by: Lars-Peter Clausen <lars@metafoo.de>

I think I left a ticket with ADI when I left to update the maintainers 
entries and replace me with somebody else, must have gotten lost :)
Jonathan Cameron Sept. 6, 2020, 4:03 p.m. UTC | #5 
On Sun, 6 Sep 2020 17:57:54 +0200
Lars-Peter Clausen <lars@metafoo.de> wrote:

> On 9/6/20 4:06 PM, Jonathan Cameron wrote:
> > On Thu,  3 Sep 2020 20:19:25 +0200
> > Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >  
> >> Emails to Beniamin Bia bounce with no such address so remove him from
> >> maintainers.  After this removal, many entries for Analog Devices Inc
> >> IIO drivers look exactly the same so consolidate them.
> >>
> >> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> >> Suggested-by: Jonathan Cameron <Jonathan.Cameron@huawei.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>
> >> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>  
> > As I'd assume a more specific binding always overrides a catch all,
> > this has the effect of giving Lars and Michael responsibility
> > for a few things they didn't previously cover.  If the two
> > of them are fine with it, than that's good, but I'd ideally
> > like an Ack from Lars.  
> 
> Acked-by: Lars-Peter Clausen <lars@metafoo.de>
> 
> I think I left a ticket with ADI when I left to update the maintainers 
> entries and replace me with somebody else, must have gotten lost :)
> 
*laughs*

You are probably stuck with it forever if you don't remove yourself :)

Applied series to the togreg branch of iio.git and pushed out as testing
for the autobuilders to probably completely ignore.

thanks,

Jonathan
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 8107b3d5d6df..4a120aec90a2 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -950,38 +950,6 @@  S:	Supported
 F:	arch/arm64/boot/dts/amd/amd-seattle-xgbe*.dtsi
 F:	drivers/net/ethernet/amd/xgbe/
 
-ANALOG DEVICES INC AD5686 DRIVER
-M:	Michael Hennerich <Michael.Hennerich@analog.com>
-L:	linux-pm@vger.kernel.org
-S:	Supported
-W:	http://ez.analog.com/community/linux-device-drivers
-F:	drivers/iio/dac/ad5686*
-F:	drivers/iio/dac/ad5696*
-
-ANALOG DEVICES INC AD5758 DRIVER
-M:	Michael Hennerich <Michael.Hennerich@analog.com>
-L:	linux-iio@vger.kernel.org
-S:	Supported
-W:	http://ez.analog.com/community/linux-device-drivers
-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
-W:	http://ez.analog.com/community/linux-device-drivers
-F:	Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml
-F:	drivers/iio/adc/ad7091r5.c
-
-ANALOG DEVICES INC AD7124 DRIVER
-M:	Michael Hennerich <Michael.Hennerich@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,ad7124.yaml
-F:	drivers/iio/adc/ad7124.c
-
 ANALOG DEVICES INC AD7192 DRIVER
 M:	Alexandru Tachici <alexandru.tachici@analog.com>
 L:	linux-iio@vger.kernel.org
@@ -998,15 +966,6 @@  W:	http://ez.analog.com/community/linux-device-drivers
 F:	Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
 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
-F:	Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
-F:	drivers/iio/adc/ad7606.c
-
 ANALOG DEVICES INC AD7768-1 DRIVER
 M:	Michael Hennerich <Michael.Hennerich@analog.com>
 L:	linux-iio@vger.kernel.org
@@ -1068,7 +1027,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
@@ -1135,15 +1093,6 @@  S:	Supported
 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
-W:	http://ez.analog.com/community/linux-device-drivers
-F:	Documentation/devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml
-F:	drivers/iio/amplifiers/hmc425a.c
-
 ANALOG DEVICES INC IIO DRIVERS
 M:	Lars-Peter Clausen <lars@metafoo.de>
 M:	Michael Hennerich <Michael.Hennerich@analog.com>
@@ -1152,8 +1101,11 @@  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:	Documentation/devicetree/bindings/iio/*/adi,*
+F:	Documentation/devicetree/bindings/iio/dac/ad5758.txt
 F:	drivers/iio/*/ad*
 F:	drivers/iio/adc/ltc249*
+F:	drivers/iio/amplifiers/hmc425a.c
 F:	drivers/staging/iio/*/ad*
 X:	drivers/iio/*/adjd*
 
@@ -16537,7 +16489,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