diff mbox

[net-next] MAINTAINERS: PHY: Change maintainer to reviewer

Message ID 1448407790-22336-1-git-send-email-f.fainelli@gmail.com
State Changes Requested, archived
Delegated to: David Miller
Headers show

Commit Message

Florian Fainelli Nov. 24, 2015, 11:29 p.m. UTC
Now that there is a reviewer role, add myself as reviewer since the PHY
library code is maintained via the networking tree.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Joe Perches Nov. 25, 2015, 1:04 a.m. UTC | #1
On Tue, 2015-11-24 at 15:29 -0800, Florian Fainelli wrote:
> Now that there is a reviewer role, add myself as reviewer since the PHY
> library code is maintained via the networking tree.

[]

> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -4195,7 +4195,7 @@ F:	include/linux/netfilter_bridge/
>  F:> 	> net/bridge/
>  
>  ETHERNET PHY LIBRARY
> -M:	Florian Fainelli <f.fainelli@gmail.com>
> +R:	Florian Fainelli <f.fainelli@gmail.com>
>  L:	netdev@vger.kernel.org
>  S:	Maintained
>  F:	include/linux/phy.h

Just because the upstream path is not a tree you
manage doesn't mean you aren't a maintainer.

I think the status should be something other than
"Maintained" if you are not the maintainer.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller Nov. 25, 2015, 4:36 p.m. UTC | #2
From: Joe Perches <joe@perches.com>
Date: Tue, 24 Nov 2015 17:04:11 -0800

> On Tue, 2015-11-24 at 15:29 -0800, Florian Fainelli wrote:
>> Now that there is a reviewer role, add myself as reviewer since the PHY
>> library code is maintained via the networking tree.
> 
> []
> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -4195,7 +4195,7 @@ F:	include/linux/netfilter_bridge/
>>  F:> 	> net/bridge/
>>  
>>  ETHERNET PHY LIBRARY
>> -M:	Florian Fainelli <f.fainelli@gmail.com>
>> +R:	Florian Fainelli <f.fainelli@gmail.com>
>>  L:	netdev@vger.kernel.org
>>  S:	Maintained
>>  F:	include/linux/phy.h
> 
> Just because the upstream path is not a tree you
> manage doesn't mean you aren't a maintainer.
> 
> I think the status should be something other than
> "Maintained" if you are not the maintainer.

Yeah, I kinda agree, Florian you are pretty much the de-facto
maintainer and I usually wait for you to review any PHY library
changes before applying them unless they are _really_ obvious.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Florian Fainelli Nov. 25, 2015, 6:07 p.m. UTC | #3
On 25/11/15 08:36, David Miller wrote:
> From: Joe Perches <joe@perches.com>
> Date: Tue, 24 Nov 2015 17:04:11 -0800
> 
>> On Tue, 2015-11-24 at 15:29 -0800, Florian Fainelli wrote:
>>> Now that there is a reviewer role, add myself as reviewer since the PHY
>>> library code is maintained via the networking tree.
>>
>> []
>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>> []
>>> @@ -4195,7 +4195,7 @@ F:	include/linux/netfilter_bridge/
>>>  F:> 	> net/bridge/
>>>  
>>>  ETHERNET PHY LIBRARY
>>> -M:	Florian Fainelli <f.fainelli@gmail.com>
>>> +R:	Florian Fainelli <f.fainelli@gmail.com>
>>>  L:	netdev@vger.kernel.org
>>>  S:	Maintained
>>>  F:	include/linux/phy.h
>>
>> Just because the upstream path is not a tree you
>> manage doesn't mean you aren't a maintainer.
>>
>> I think the status should be something other than
>> "Maintained" if you are not the maintainer.
> 
> Yeah, I kinda agree, Florian you are pretty much the de-facto
> maintainer and I usually wait for you to review any PHY library
> changes before applying them unless they are _really_ obvious.

Sounds good, let's drop this patch then, thanks!
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index ea1751283b49..950c321eef73 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4195,7 +4195,7 @@  F:	include/linux/netfilter_bridge/
 F:	net/bridge/
 
 ETHERNET PHY LIBRARY
-M:	Florian Fainelli <f.fainelli@gmail.com>
+R:	Florian Fainelli <f.fainelli@gmail.com>
 L:	netdev@vger.kernel.org
 S:	Maintained
 F:	include/linux/phy.h