diff mbox

MAINTAINERS: add myself as Renesas Ethernet drivers reviewer

Message ID 1846877.jRW8849mPU@wasted.cogentembedded.com (mailing list archive)
State Superseded
Delegated to: Simon Horman
Headers show

Commit Message

Sergei Shtylyov Dec. 5, 2015, 12:03 a.m. UTC
liAdd myself as  a reviewer for the Renesas Ethernet drivers -- hopefully I
won't miss the buggy  patches anymore. :-)

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
The patch is against DaveM's 'net.git' repo.

 MAINTAINERS |    8 ++++++++
 1 file changed, 8 insertions(+)


--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Joe Perches Dec. 5, 2015, 12:41 a.m. UTC | #1
On Sat, 2015-12-05 at 03:03 +0300, Sergei Shtylyov wrote:
> liAdd myself as  a reviewer for the Renesas Ethernet drivers --
> hopefully I
> won't miss the buggy  patches anymore. :-)
> 
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
> ---
> The patch is against DaveM's 'net.git' repo.
> 
>  MAINTAINERS |    8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> Index: net/MAINTAINERS
> ===================================================================
> --- net.orig/MAINTAINERS
> +++ net/MAINTAINERS
> @@ -8929,6 +8929,14 @@ F:	drivers/rpmsg/
>  F:	Documentation/rpmsg.txt
>  F:	include/linux/rpmsg.h
>  
> +RENESAS ETHERNET DRIVERS
> +R:	Sergei Shtylyov <serge.shtylyov@cogentembedded.com>
> +L:	netdev@vger.kernel.orgc
> +L:	linux-sh@vger.kernel.org
> +S:	Maintained
> +F:	drivers/net/ethernet/renesas/
> +F:	include/linux/sh_eth.h

Hello Sergei

Typically a reviewer isn't a maintainer.
Does anyone actually maintain this?

I suggest you mark this "S: Odd fixes" or
remove the "S:" line altogether unless you
really want to become the maintainer.

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sergei Shtylyov Dec. 5, 2015, 12:57 a.m. UTC | #2
Hello.

On 12/05/2015 03:41 AM, Joe Perches wrote:

>> liAdd myself as  a reviewer for the Renesas Ethernet drivers --
>> hopefully I
>> won't miss the buggy  patches anymore. :-)
>>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>> ---
>> The patch is against DaveM's 'net.git' repo.
>>
>>   MAINTAINERS |    8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> Index: net/MAINTAINERS
>> ===================================================================
>> --- net.orig/MAINTAINERS
>> +++ net/MAINTAINERS
>> @@ -8929,6 +8929,14 @@ F:	drivers/rpmsg/
>>   F:	Documentation/rpmsg.txt
>>   F:	include/linux/rpmsg.h
>>
>> +RENESAS ETHERNET DRIVERS
>> +R:	Sergei Shtylyov <serge.shtylyov@cogentembedded.com>
>> +L:	netdev@vger.kernel.orgc
>> +L:	linux-sh@vger.kernel.org
>> +S:	Maintained
>> +F:	drivers/net/ethernet/renesas/
>> +F:	include/linux/sh_eth.h

> Hello Sergei
>
> Typically a reviewer isn't a maintainer.

    Yeah, I didn't take courage to become a real maintainer, being always 
short of time... But I was annoyed by dubious quality patches being merged 
recently.

> Does anyone actually maintain this?

    Only DaveM.

> I suggest you mark this "S: Odd fixes" or
> remove the "S:" line altogether unless you
> really want to become the maintainer.

    OK, will remove, if that's OK.

PS: Since you seem to be get_maintainers.pl maintainer, I (and not only I)
would like to propose a switch to suppress the committers/authors/etc. It's 
generally annoying and unhelpful to have the bunch of people listed, none of 
which usually are a good addressees for the patches...

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joe Perches Dec. 5, 2015, 1:02 a.m. UTC | #3
On Sat, 2015-12-05 at 03:57 +0300, Sergei Shtylyov wrote:
> Since you seem to be get_maintainers.pl maintainer, I (and not only I)
> would like to propose a switch to suppress the committers/authors/etc. It's 
> generally annoying and unhelpful to have the bunch of people listed, none of 
> which usually are a good addressees for the patches...

Done!

Add "--nogit" and "--nogit-fallback"

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Sergei Shtylyov Dec. 5, 2015, 1:13 a.m. UTC | #4
On 12/05/2015 04:02 AM, Joe Perches wrote:

>> Since you seem to be get_maintainers.pl maintainer, I (and not only I)
>> would like to propose a switch to suppress the committers/authors/etc. It's
>> generally annoying and unhelpful to have the bunch of people listed, none of
>> which usually are a good addressees for the patches...
>
> Done!

> Add "--nogit" and "--nogit-fallback"

    Em, --nogit is the dafault, according to --help. --nogit-fallback is 
enough but it's undocumented. :-/
    Thanks anyway!

MBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joe Perches Dec. 5, 2015, 1:18 a.m. UTC | #5
On Sat, 2015-12-05 at 04:13 +0300, Sergei Shtylyov wrote:
> On 12/05/2015 04:02 AM, Joe Perches wrote:
> > > Since you seem to be get_maintainers.pl maintainer, I (and not only I)
> > > would like to propose a switch to suppress the committers/authors/etc. It's
> > > generally annoying and unhelpful to have the bunch of people listed, none of
> > > which usually are a good addressees for the patches...
> > 
> > Done!
> 
> > Add "--nogit" and "--nogit-fallback"
> 
>     Em, --nogit is the dafault, according to --help. --nogit-fallback is 
> enough but it's undocumented. :-/

umm

$ ./scripts/get_maintainer.pl --help
[...]
    --git-fallback => use git when no exact MAINTAINERS pattern (default: 1)
[...]
  Most options have both positive and negative forms.
      The negative forms for --<foo> are --no<foo> and --no-<foo>.
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

Index: net/MAINTAINERS
===================================================================
--- net.orig/MAINTAINERS
+++ net/MAINTAINERS
@@ -8929,6 +8929,14 @@  F:	drivers/rpmsg/
 F:	Documentation/rpmsg.txt
 F:	include/linux/rpmsg.h
 
+RENESAS ETHERNET DRIVERS
+R:	Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
+L:	netdev@vger.kernel.org
+L:	linux-sh@vger.kernel.org
+S:	Maintained
+F:	drivers/net/ethernet/renesas/
+F:	include/linux/sh_eth.h
+
 RESET CONTROLLER FRAMEWORK
 M:	Philipp Zabel <p.zabel@pengutronix.de>
 S:	Maintained