mbox series

[v2,0/5] iio/hwmon/mfd/leds/net/power/ASoC: dt-bindings: few stale maintainers cleanup

Message ID 20220809162752.10186-1-krzysztof.kozlowski@linaro.org
Headers show
Series iio/hwmon/mfd/leds/net/power/ASoC: dt-bindings: few stale maintainers cleanup | expand

Message

Krzysztof Kozlowski Aug. 9, 2022, 4:27 p.m. UTC
Hi,

Changes since v1
================
1. Patch #5: Drop also Ricardo Rivera-Matos and assign TI bindings to Andrew Davis
2. Add acks.

A question
==========

Several of the bindings here had only one maintainer and history does not
always point to a new one (although I did not perform extensive digging). I
added subsystem maintainer, because dtschema requires an entry with valid email address.

This is not the best choice as simply subsystem maintainer might not have the
actual device (or its datasheets or any interest in it).

Maybe we could add some "orphaned" entry in such case?

Best regards,
Krzysztof

Krzysztof Kozlowski (5):
  dt-bindings: iio: Drop Joachim Eastwood
  dt-bindings: iio: Drop Bogdan Pricop
  dt-bindings: Drop Beniamin Bia and Stefan Popa
  dt-bindings: Drop Robert Jones
  dt-bindings: Drop Dan Murphy and Ricardo Rivera-Matos

 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 1 -
 Documentation/devicetree/bindings/iio/accel/fsl,mma7455.yaml   | 1 -
 Documentation/devicetree/bindings/iio/adc/adi,ad7091r5.yaml    | 2 +-
 Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml      | 3 +--
 Documentation/devicetree/bindings/iio/adc/nxp,lpc1850-adc.yaml | 2 +-
 Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml   | 2 +-
 Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml    | 2 +-
 .../devicetree/bindings/iio/amplifiers/adi,hmc425a.yaml        | 1 -
 Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml    | 2 +-
 .../devicetree/bindings/leds/leds-class-multicolor.yaml        | 2 +-
 Documentation/devicetree/bindings/leds/leds-lp50xx.yaml        | 2 +-
 Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml       | 1 -
 Documentation/devicetree/bindings/net/ti,dp83822.yaml          | 2 +-
 Documentation/devicetree/bindings/net/ti,dp83867.yaml          | 2 +-
 Documentation/devicetree/bindings/net/ti,dp83869.yaml          | 2 +-
 Documentation/devicetree/bindings/power/supply/bq2515x.yaml    | 3 +--
 Documentation/devicetree/bindings/power/supply/bq256xx.yaml    | 2 +-
 Documentation/devicetree/bindings/power/supply/bq25980.yaml    | 3 +--
 Documentation/devicetree/bindings/sound/tas2562.yaml           | 2 +-
 Documentation/devicetree/bindings/sound/tlv320adcx140.yaml     | 2 +-
 20 files changed, 16 insertions(+), 23 deletions(-)

Comments

Jonathan Cameron Aug. 10, 2022, 10:11 a.m. UTC | #1
On Tue,  9 Aug 2022 19:27:52 +0300
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:

> Emails to Dan Murphy and Ricardo Rivera-Matos bounce ("550 Invalid
> recipient").  Andrew Davis agreed to take over the bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Changes since v1:
> 1. Add Andrew Davis instead.
> 2. Not adding accumulated ack due to change above.
> ---
>  Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml    | 2 +-
>  .../devicetree/bindings/leds/leds-class-multicolor.yaml        | 2 +-
>  Documentation/devicetree/bindings/leds/leds-lp50xx.yaml        | 2 +-
>  Documentation/devicetree/bindings/net/ti,dp83822.yaml          | 2 +-
>  Documentation/devicetree/bindings/net/ti,dp83867.yaml          | 2 +-
>  Documentation/devicetree/bindings/net/ti,dp83869.yaml          | 2 +-
>  Documentation/devicetree/bindings/power/supply/bq2515x.yaml    | 3 +--
>  Documentation/devicetree/bindings/power/supply/bq256xx.yaml    | 2 +-
>  Documentation/devicetree/bindings/power/supply/bq25980.yaml    | 3 +--
>  Documentation/devicetree/bindings/sound/tas2562.yaml           | 2 +-
>  Documentation/devicetree/bindings/sound/tlv320adcx140.yaml     | 2 +-
>  11 files changed, 11 insertions(+), 13 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml b/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml
> index 9f5e96439c01..2e6abc9d746a 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,ads124s08.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: Texas Instruments' ads124s08 and ads124s06 ADC chip
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
For this one.

Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

>  
>  properties:
>    compatible:
> diff --git a/Documentation/devicetree/bindings/leds/leds-class-multicolor.yaml b/Documentation/devicetree/bindings/leds/leds-class-multicolor.yaml
> index 12693483231f..31840e33dcf5 100644
> --- a/Documentation/devicetree/bindings/leds/leds-class-multicolor.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-class-multicolor.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: Common properties for the multicolor LED class.
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    Bindings for multi color LEDs show how to describe current outputs of
> diff --git a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
> index e0b658f07973..63da380748bf 100644
> --- a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: LED driver for LP50XX RGB LED from Texas Instruments.
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The LP50XX is multi-channel, I2C RGB LED Drivers that can group RGB LEDs into
> diff --git a/Documentation/devicetree/bindings/net/ti,dp83822.yaml b/Documentation/devicetree/bindings/net/ti,dp83822.yaml
> index 75e8712e903a..f2489a9c852f 100644
> --- a/Documentation/devicetree/bindings/net/ti,dp83822.yaml
> +++ b/Documentation/devicetree/bindings/net/ti,dp83822.yaml
> @@ -8,7 +8,7 @@ $schema: "http://devicetree.org/meta-schemas/core.yaml#"
>  title: TI DP83822 ethernet PHY
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The DP83822 is a low-power, single-port, 10/100 Mbps Ethernet PHY. It
> diff --git a/Documentation/devicetree/bindings/net/ti,dp83867.yaml b/Documentation/devicetree/bindings/net/ti,dp83867.yaml
> index 76ff08a477ba..b8c0e4b5b494 100644
> --- a/Documentation/devicetree/bindings/net/ti,dp83867.yaml
> +++ b/Documentation/devicetree/bindings/net/ti,dp83867.yaml
> @@ -11,7 +11,7 @@ allOf:
>    - $ref: "ethernet-controller.yaml#"
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The DP83867 device is a robust, low power, fully featured Physical Layer
> diff --git a/Documentation/devicetree/bindings/net/ti,dp83869.yaml b/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> index 1b780dce61ab..b04ff0014a59 100644
> --- a/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> +++ b/Documentation/devicetree/bindings/net/ti,dp83869.yaml
> @@ -11,7 +11,7 @@ allOf:
>    - $ref: "ethernet-phy.yaml#"
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The DP83869HM device is a robust, fully-featured Gigabit (PHY) transceiver
> diff --git a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> index 27db38577822..1a1b240034ef 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> @@ -8,8 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI bq2515x 500-mA Linear charger family
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The BQ2515x family is a highly integrated battery charge management IC that
> diff --git a/Documentation/devicetree/bindings/power/supply/bq256xx.yaml b/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> index 91abe5733c41..82f382a7ffb3 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> @@ -8,7 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI bq256xx Switch Mode Buck Charger
>  
>  maintainers:
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The bq256xx devices are a family of highly-integrated battery charge
> diff --git a/Documentation/devicetree/bindings/power/supply/bq25980.yaml b/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> index 4883527ab5c7..b687b8bcd705 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> @@ -8,8 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI BQ25980 Flash Charger
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The BQ25980, BQ25975, and BQ25960 are a series of flash chargers intended
> diff --git a/Documentation/devicetree/bindings/sound/tas2562.yaml b/Documentation/devicetree/bindings/sound/tas2562.yaml
> index 5f7dd5d6cbca..30f6b029ac08 100644
> --- a/Documentation/devicetree/bindings/sound/tas2562.yaml
> +++ b/Documentation/devicetree/bindings/sound/tas2562.yaml
> @@ -8,7 +8,7 @@ $schema: "http://devicetree.org/meta-schemas/core.yaml#"
>  title: Texas Instruments TAS2562 Smart PA
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The TAS2562 is a mono, digital input Class-D audio amplifier optimized for
> diff --git a/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml b/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
> index bc2fb1a80ed7..ee698614862e 100644
> --- a/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
> +++ b/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
> @@ -8,7 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: Texas Instruments TLV320ADCX140 Quad Channel Analog-to-Digital Converter
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The TLV320ADCX140 are multichannel (4-ch analog recording or 8-ch digital
Lee Jones Aug. 10, 2022, 11:01 a.m. UTC | #2
On Tue, 09 Aug 2022, Krzysztof Kozlowski wrote:

> Emails to Robert Jones bounce ("550 5.2.1 The email account that you
> tried to reach is disabled").
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> 
> ---
> 
> For maintainers entry see:
> https://lore.kernel.org/all/20220808111113.71890-1-krzysztof.kozlowski@linaro.org/
> ---
>  Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 2 +-
>  Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml    | 1 -

Any reason to submit these as one patch?

>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> index 479e7065d4eb..0203b83b8587 100644
> --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: Freescale FXOS8700 Inertial Measurement Unit
>  
>  maintainers:
> -  - Robert Jones <rjones@gateworks.com>
> +  - Jonathan Cameron <jic23@kernel.org>
>  
>  description: |
>    Accelerometer and magnetometer combo device with an i2c and SPI interface.
> diff --git a/Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml b/Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml
> index 5a1e8d21f7a0..5e0fe3ebe1d2 100644
> --- a/Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml
> @@ -19,7 +19,6 @@ description: |
>  
>  maintainers:
>    - Tim Harvey <tharvey@gateworks.com>
> -  - Robert Jones <rjones@gateworks.com>
>  
>  properties:
>    $nodename:
Krzysztof Kozlowski Aug. 10, 2022, 11:22 a.m. UTC | #3
On 10/08/2022 14:01, Lee Jones wrote:
> On Tue, 09 Aug 2022, Krzysztof Kozlowski wrote:
> 
>> Emails to Robert Jones bounce ("550 5.2.1 The email account that you
>> tried to reach is disabled").
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
>>
>> ---
>>
>> For maintainers entry see:
>> https://lore.kernel.org/all/20220808111113.71890-1-krzysztof.kozlowski@linaro.org/
>> ---
>>  Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 2 +-
>>  Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml    | 1 -
> 
> Any reason to submit these as one patch?

Less work for me, less work for maintainer applying and sending fixes. I
think this could go via Rob's tree as fixes for current cycle.

Best regards,
Krzysztof
Lee Jones Aug. 10, 2022, 11:43 a.m. UTC | #4
On Wed, 10 Aug 2022, Krzysztof Kozlowski wrote:

> On 10/08/2022 14:01, Lee Jones wrote:
> > On Tue, 09 Aug 2022, Krzysztof Kozlowski wrote:
> > 
> >> Emails to Robert Jones bounce ("550 5.2.1 The email account that you
> >> tried to reach is disabled").
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >> Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> >>
> >> ---
> >>
> >> For maintainers entry see:
> >> https://lore.kernel.org/all/20220808111113.71890-1-krzysztof.kozlowski@linaro.org/
> >> ---
> >>  Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 2 +-
> >>  Documentation/devicetree/bindings/mfd/gateworks-gsc.yaml    | 1 -
> > 
> > Any reason to submit these as one patch?
> 
> Less work for me, less work for maintainer applying and sending fixes.

Easier for Jonathan and I to apply to our respective trees.

> I think this could go via Rob's tree as fixes for current cycle.

Happy for that to happen if it's okay with Rob:

Acked-by: Lee Jones <lee@kernel.org>
Mark Brown Aug. 10, 2022, 12:38 p.m. UTC | #5
On Tue, Aug 09, 2022 at 07:27:52PM +0300, Krzysztof Kozlowski wrote:
> Emails to Dan Murphy and Ricardo Rivera-Matos bounce ("550 Invalid
> recipient").  Andrew Davis agreed to take over the bindings.

Acked-by: Mark Brown <broonie@kernel.org>
Sebastian Reichel Aug. 10, 2022, 12:57 p.m. UTC | #6
Hi,

On Tue, Aug 09, 2022 at 07:27:52PM +0300, Krzysztof Kozlowski wrote:
> Emails to Dan Murphy and Ricardo Rivera-Matos bounce ("550 Invalid
> recipient").  Andrew Davis agreed to take over the bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> Changes since v1:
> 1. Add Andrew Davis instead.
> 2. Not adding accumulated ack due to change above.
> ---
...
>  Documentation/devicetree/bindings/power/supply/bq2515x.yaml    | 3 +--
>  Documentation/devicetree/bindings/power/supply/bq256xx.yaml    | 2 +-
>  Documentation/devicetree/bindings/power/supply/bq25980.yaml    | 3 +--

Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com>

-- Sebastian

> [...]
> diff --git a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> index 27db38577822..1a1b240034ef 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq2515x.yaml
> @@ -8,8 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI bq2515x 500-mA Linear charger family
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The BQ2515x family is a highly integrated battery charge management IC that
> diff --git a/Documentation/devicetree/bindings/power/supply/bq256xx.yaml b/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> index 91abe5733c41..82f382a7ffb3 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq256xx.yaml
> @@ -8,7 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI bq256xx Switch Mode Buck Charger
>  
>  maintainers:
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The bq256xx devices are a family of highly-integrated battery charge
> diff --git a/Documentation/devicetree/bindings/power/supply/bq25980.yaml b/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> index 4883527ab5c7..b687b8bcd705 100644
> --- a/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/bq25980.yaml
> @@ -8,8 +8,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
>  title: TI BQ25980 Flash Charger
>  
>  maintainers:
> -  - Dan Murphy <dmurphy@ti.com>
> -  - Ricardo Rivera-Matos <r-rivera-matos@ti.com>
> +  - Andrew Davis <afd@ti.com>
>  
>  description: |
>    The BQ25980, BQ25975, and BQ25960 are a series of flash chargers intended
> [...]
Rob Herring Aug. 12, 2022, 4:54 p.m. UTC | #7
On Tue, Aug 09, 2022 at 07:27:47PM +0300, Krzysztof Kozlowski wrote:
> Hi,
> 
> Changes since v1
> ================
> 1. Patch #5: Drop also Ricardo Rivera-Matos and assign TI bindings to Andrew Davis
> 2. Add acks.
> 
> A question
> ==========
> 
> Several of the bindings here had only one maintainer and history does not
> always point to a new one (although I did not perform extensive digging). I
> added subsystem maintainer, because dtschema requires an entry with valid email address.
> 
> This is not the best choice as simply subsystem maintainer might not have the
> actual device (or its datasheets or any interest in it).
> 
> Maybe we could add some "orphaned" entry in such case?

It would need to be obvious to not use for a new binding.

> 
> Best regards,
> Krzysztof
> 
> Krzysztof Kozlowski (5):
>   dt-bindings: iio: Drop Joachim Eastwood
>   dt-bindings: iio: Drop Bogdan Pricop
>   dt-bindings: Drop Beniamin Bia and Stefan Popa
>   dt-bindings: Drop Robert Jones
>   dt-bindings: Drop Dan Murphy and Ricardo Rivera-Matos

Series applied for 6.0-rc1.

Rob