* [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer
@ 2015-11-24 23:29 Florian Fainelli
2015-11-25 1:04 ` Joe Perches
0 siblings, 1 reply; 4+ messages in thread
From: Florian Fainelli @ 2015-11-24 23:29 UTC (permalink / raw)
To: netdev; +Cc: davem, Florian Fainelli
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(-)
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
--
2.1.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer
2015-11-24 23:29 [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer Florian Fainelli
@ 2015-11-25 1:04 ` Joe Perches
2015-11-25 16:36 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Joe Perches @ 2015-11-25 1:04 UTC (permalink / raw)
To: Florian Fainelli, netdev; +Cc: davem
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.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer
2015-11-25 1:04 ` Joe Perches
@ 2015-11-25 16:36 ` David Miller
2015-11-25 18:07 ` Florian Fainelli
0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2015-11-25 16:36 UTC (permalink / raw)
To: joe; +Cc: f.fainelli, netdev
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.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer
2015-11-25 16:36 ` David Miller
@ 2015-11-25 18:07 ` Florian Fainelli
0 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2015-11-25 18:07 UTC (permalink / raw)
To: David Miller, joe; +Cc: netdev
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!
--
Florian
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-11-25 18:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-24 23:29 [PATCH net-next] MAINTAINERS: PHY: Change maintainer to reviewer Florian Fainelli
2015-11-25 1:04 ` Joe Perches
2015-11-25 16:36 ` David Miller
2015-11-25 18:07 ` Florian Fainelli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).