[PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections

Geert Uytterhoeven posted 1 patch 1 month ago
There is a newer version of this series
MAINTAINERS | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
[PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Geert Uytterhoeven 1 month ago
Removing full driver sections also removed mailing list entries, causing
submitters of future patches to forget CCing these mailing lists.

Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Anyone who wants to take over maintenance for these drivers?
Thanks in advance!
---
 MAINTAINERS | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index f04cba42a59301fa..97a23cea2729942e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19521,6 +19521,14 @@ S:	Supported
 F:	Documentation/devicetree/bindings/i2c/renesas,iic-emev2.yaml
 F:	drivers/i2c/busses/i2c-emev2.c
 
+RENESAS ETHERNET AVB DRIVER
+L:	netdev@vger.kernel.org
+L:	linux-renesas-soc@vger.kernel.org
+F:	Documentation/devicetree/bindings/net/renesas,etheravb.yaml
+F:	drivers/net/ethernet/renesas/Kconfig
+F:	drivers/net/ethernet/renesas/Makefile
+F:	drivers/net/ethernet/renesas/ravb*
+
 RENESAS ETHERNET SWITCH DRIVER
 R:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
 L:	netdev@vger.kernel.org
@@ -19570,6 +19578,13 @@ F:	Documentation/devicetree/bindings/i2c/renesas,rmobile-iic.yaml
 F:	drivers/i2c/busses/i2c-rcar.c
 F:	drivers/i2c/busses/i2c-sh_mobile.c
 
+RENESAS R-CAR SATA DRIVER
+L:	linux-ide@vger.kernel.org
+L:	linux-renesas-soc@vger.kernel.org
+S:	Supported
+F:	Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
+F:	drivers/ata/sata_rcar.c
+
 RENESAS R-CAR THERMAL DRIVERS
 M:	Niklas Söderlund <niklas.soderlund@ragnatech.se>
 L:	linux-renesas-soc@vger.kernel.org
@@ -19645,6 +19660,15 @@ S:	Supported
 F:	Documentation/devicetree/bindings/i2c/renesas,rzv2m.yaml
 F:	drivers/i2c/busses/i2c-rzv2m.c
 
+RENESAS SUPERH ETHERNET DRIVER
+L:	netdev@vger.kernel.org
+L:	linux-renesas-soc@vger.kernel.org
+F:	Documentation/devicetree/bindings/net/renesas,ether.yaml
+F:	drivers/net/ethernet/renesas/Kconfig
+F:	drivers/net/ethernet/renesas/Makefile
+F:	drivers/net/ethernet/renesas/sh_eth*
+F:	include/linux/sh_eth.h
+
 RENESAS USB PHY DRIVER
 M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
 L:	linux-renesas-soc@vger.kernel.org
-- 
2.34.1

Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Greg Kroah-Hartman 3 weeks, 1 day ago
On Mon, Oct 21, 2024 at 01:56:51PM +0200, Geert Uytterhoeven wrote:
> +RENESAS R-CAR SATA DRIVER
> +L:	linux-ide@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +S:	Supported
> +F:	Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F:	drivers/ata/sata_rcar.c

You can't have a "Supported" entry with no person assigned to it :(
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Niklas Cassel 1 month ago
On Mon, Oct 21, 2024 at 01:56:51PM +0200, Geert Uytterhoeven wrote:
> Removing full driver sections also removed mailing list entries, causing
> submitters of future patches to forget CCing these mailing lists.
> 
> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Anyone who wants to take over maintenance for these drivers?
> Thanks in advance!
> ---
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f04cba42a59301fa..97a23cea2729942e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19521,6 +19521,14 @@ S:	Supported
>  F:	Documentation/devicetree/bindings/i2c/renesas,iic-emev2.yaml
>  F:	drivers/i2c/busses/i2c-emev2.c
>  
> +RENESAS ETHERNET AVB DRIVER
> +L:	netdev@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +F:	Documentation/devicetree/bindings/net/renesas,etheravb.yaml
> +F:	drivers/net/ethernet/renesas/Kconfig
> +F:	drivers/net/ethernet/renesas/Makefile
> +F:	drivers/net/ethernet/renesas/ravb*
> +
>  RENESAS ETHERNET SWITCH DRIVER
>  R:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	netdev@vger.kernel.org
> @@ -19570,6 +19578,13 @@ F:	Documentation/devicetree/bindings/i2c/renesas,rmobile-iic.yaml
>  F:	drivers/i2c/busses/i2c-rcar.c
>  F:	drivers/i2c/busses/i2c-sh_mobile.c
>  
> +RENESAS R-CAR SATA DRIVER
> +L:	linux-ide@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +S:	Supported

Seems a bit weird to keep the "S: Supported" line, if there is no
"M: " or "R: " entry. I suggest you kill it.

Regardless:
Acked-by: Niklas Cassel <cassel@kernel.org>


> +F:	Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F:	drivers/ata/sata_rcar.c
> +
>  RENESAS R-CAR THERMAL DRIVERS
>  M:	Niklas Söderlund <niklas.soderlund@ragnatech.se>
>  L:	linux-renesas-soc@vger.kernel.org
> @@ -19645,6 +19660,15 @@ S:	Supported
>  F:	Documentation/devicetree/bindings/i2c/renesas,rzv2m.yaml
>  F:	drivers/i2c/busses/i2c-rzv2m.c
>  
> +RENESAS SUPERH ETHERNET DRIVER
> +L:	netdev@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +F:	Documentation/devicetree/bindings/net/renesas,ether.yaml
> +F:	drivers/net/ethernet/renesas/Kconfig
> +F:	drivers/net/ethernet/renesas/Makefile
> +F:	drivers/net/ethernet/renesas/sh_eth*
> +F:	include/linux/sh_eth.h
> +
>  RENESAS USB PHY DRIVER
>  M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	linux-renesas-soc@vger.kernel.org
> -- 
> 2.34.1
> 
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Jakub Kicinski 4 weeks ago
On Mon, 21 Oct 2024 17:11:43 +0200 Niklas Cassel wrote:
> > +RENESAS R-CAR SATA DRIVER
> > +L:	linux-ide@vger.kernel.org
> > +L:	linux-renesas-soc@vger.kernel.org
> > +S:	Supported  
> 
> Seems a bit weird to keep the "S: Supported" line, if there is no
> "M: " or "R: " entry. I suggest you kill it.

Agreed, I think they should all be (temporarily?) designated as Orphans.
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Niklas Söderlund 1 month ago
Hello Geert,

On 2024-10-21 13:56:51 +0200, Geert Uytterhoeven wrote:
> Removing full driver sections also removed mailing list entries, causing
> submitters of future patches to forget CCing these mailing lists.
> 
> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Anyone who wants to take over maintenance for these drivers?

In case Sergei is not interested to keep looking after the RAVB and/or 
SUPERH Ethernet drivers I would be happy to do so.

In either case should not the maintainer entry in the bindings documents
also be updated?

> Thanks in advance!
> ---
>  MAINTAINERS | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f04cba42a59301fa..97a23cea2729942e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -19521,6 +19521,14 @@ S:	Supported
>  F:	Documentation/devicetree/bindings/i2c/renesas,iic-emev2.yaml
>  F:	drivers/i2c/busses/i2c-emev2.c
>  
> +RENESAS ETHERNET AVB DRIVER
> +L:	netdev@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +F:	Documentation/devicetree/bindings/net/renesas,etheravb.yaml
> +F:	drivers/net/ethernet/renesas/Kconfig
> +F:	drivers/net/ethernet/renesas/Makefile
> +F:	drivers/net/ethernet/renesas/ravb*
> +
>  RENESAS ETHERNET SWITCH DRIVER
>  R:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	netdev@vger.kernel.org
> @@ -19570,6 +19578,13 @@ F:	Documentation/devicetree/bindings/i2c/renesas,rmobile-iic.yaml
>  F:	drivers/i2c/busses/i2c-rcar.c
>  F:	drivers/i2c/busses/i2c-sh_mobile.c
>  
> +RENESAS R-CAR SATA DRIVER
> +L:	linux-ide@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +S:	Supported
> +F:	Documentation/devicetree/bindings/ata/renesas,rcar-sata.yaml
> +F:	drivers/ata/sata_rcar.c
> +
>  RENESAS R-CAR THERMAL DRIVERS
>  M:	Niklas Söderlund <niklas.soderlund@ragnatech.se>
>  L:	linux-renesas-soc@vger.kernel.org
> @@ -19645,6 +19660,15 @@ S:	Supported
>  F:	Documentation/devicetree/bindings/i2c/renesas,rzv2m.yaml
>  F:	drivers/i2c/busses/i2c-rzv2m.c
>  
> +RENESAS SUPERH ETHERNET DRIVER
> +L:	netdev@vger.kernel.org
> +L:	linux-renesas-soc@vger.kernel.org
> +F:	Documentation/devicetree/bindings/net/renesas,ether.yaml
> +F:	drivers/net/ethernet/renesas/Kconfig
> +F:	drivers/net/ethernet/renesas/Makefile
> +F:	drivers/net/ethernet/renesas/sh_eth*
> +F:	include/linux/sh_eth.h
> +
>  RENESAS USB PHY DRIVER
>  M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	linux-renesas-soc@vger.kernel.org
> -- 
> 2.34.1
> 

-- 
Kind Regards,
Niklas Söderlund
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Sergei Shtylyov 1 month ago
On 10/21/24 6:04 PM, Niklas Söderlund wrote:
[...]

>> Removing full driver sections also removed mailing list entries, causing
>> submitters of future patches to forget CCing these mailing lists.
>>
>> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>> Anyone who wants to take over maintenance for these drivers?
> 
> In case Sergei is not interested to keep looking after the RAVB and/or 
> SUPERH Ethernet drivers I would be happy to do so.

   I am still interested, of course... but looks like I'm not allowed to anymore. :-/

> In either case should not the maintainer entry in the bindings documents
> also be updated?

   These still have my Gmail address... I'm not sure yet what to do with it...

[...]

MBR, Sergey

Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Sergei Shtylyov 1 month ago
On 10/21/24 2:56 PM, Geert Uytterhoeven wrote:

> Removing full driver sections also removed mailing list entries, causing
> submitters of future patches to forget CCing these mailing lists.
> 
> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")

   It looks like nobody had seen this one before merging... :-/

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

   Thank you for informing me of the situation and fixing up Greg's patch!

> ---
> Anyone who wants to take over maintenance for these drivers?

   I could use my Gmail address again (I did use it back in 2020 for some time)
but looking at the removed Baikal entries, this probably is not going to fly...
   BTW, OMP is not interested in the Renesas drivers or the PATA drivers
themselves; they were interested in hiring an official maintainer/reviewer
like me... :-)

[...]
MBR, Sergey
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Simon Horman 1 month ago
On Mon, Oct 21, 2024 at 01:56:51PM +0200, Geert Uytterhoeven wrote:
> Removing full driver sections also removed mailing list entries, causing
> submitters of future patches to forget CCing these mailing lists.
> 
> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Simon Horman <horms@kernel.org>
Re: [PATCH/RFC] MAINTAINERS: Re-add cancelled Renesas driver sections
Posted by Greg Kroah-Hartman 1 month ago
On Mon, Oct 21, 2024 at 01:56:51PM +0200, Geert Uytterhoeven wrote:
> Removing full driver sections also removed mailing list entries, causing
> submitters of future patches to forget CCing these mailing lists.
> 
> Fixes: 6e90b675cf942e50 ("MAINTAINERS: Remove some entries due to various compliance requirements.")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>