[PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Mickey Rachamim]

Add maintainers info for new Marvell Prestera Ethernet switch driver.

Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index a7bdebf955bb..04a27eb89428 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10540,6 +10540,15 @@ S:	Supported
 F:	Documentation/networking/device_drivers/ethernet/marvell/octeontx2.rst
 F:	drivers/net/ethernet/marvell/octeontx2/af/
 
+MARVELL PRESTERA ETHERNET SWITCH DRIVER
+M:	Vadym Kochan <vkochan@marvell.com>
+M:	Taras Chornyi <tchornyi@marvell.com>
+M:	Mickey Rachamim <mickeyr@marvell.com>
+L:	netdev@vger.kernel.org
+S:	Supported
+W:	http://www.marvell.com
+F:	drivers/net/ethernet/marvell/prestera/
+
 MARVELL SOC MMC/SD/SDIO CONTROLLER DRIVER
 M:	Nicolas Pitre <nico@fluxnic.net>
 S:	Odd Fixes
-- 
2.17.1

Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Andrew Lunn]

On Thu, Dec 03, 2020 at 12:04:36PM +0200, Mickey Rachamim wrote:
> Add maintainers info for new Marvell Prestera Ethernet switch driver.
> 
> Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a7bdebf955bb..04a27eb89428 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10540,6 +10540,15 @@ S:	Supported
>  F:	Documentation/networking/device_drivers/ethernet/marvell/octeontx2.rst
>  F:	drivers/net/ethernet/marvell/octeontx2/af/
>  
> +MARVELL PRESTERA ETHERNET SWITCH DRIVER
> +M:	Vadym Kochan <vkochan@marvell.com>

Hi Mickey

All the commits came from

Vadym Kochan <vadym.kochan@plvision.eu>

Has Marvell purchased PL Vision?

    Andrew

RE: [EXT] Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Mickey Rachamim]

On Thu, Dec 03, 2020 at 04:36:00PM +0200, Andrew Lunn wrote:
>> Add maintainers info for new Marvell Prestera Ethernet switch driver.
>> 
>> Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
>> ---
>>  MAINTAINERS | 9 +++++++++
>>  1 file changed, 9 insertions(+)

> Hi Mickey
> 
> All the commits came from
> 
> Vadym Kochan <vadym.kochan@plvision.eu>
> 
> Has Marvell purchased PL Vision?
> 
>     Andrew

Hi Andrew, 

No, Marvell didn't purchase PLVision and I can understand the reason for thinking it.
PLVision and Marvell teams are keep working together as partners on supporting this program.
Vadym Kochan is and will remain an important contributor on this program.

Regards, Mickey.

Re: [EXT] Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Andrew Lunn]

On Thu, Dec 03, 2020 at 03:44:22PM +0000, Mickey Rachamim wrote:
> On Thu, Dec 03, 2020 at 04:36:00PM +0200, Andrew Lunn wrote:
> >> Add maintainers info for new Marvell Prestera Ethernet switch driver.
> >> 
> >> Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
> >> ---
> >>  MAINTAINERS | 9 +++++++++
> >>  1 file changed, 9 insertions(+)
> 
> > Hi Mickey
> > 
> > All the commits came from
> > 
> > Vadym Kochan <vadym.kochan@plvision.eu>
> > 
> > Has Marvell purchased PL Vision?
> > 
> >     Andrew
> 
> Hi Andrew, 
> 
> No, Marvell didn't purchase PLVision and I can understand the reason for thinking it.
> PLVision and Marvell teams are keep working together as partners on supporting this program.
> Vadym Kochan is and will remain an important contributor on this program.

So Vadym Kochan is still at PLVision? Please use PLVision email
addresses for people at PLVision, and Marvell email addresses for
people at Marvell. We don't want Marketing in the MAINTAINER file.

      Andrew

Re: [EXT] Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Jakub Kicinski]

On Thu, 3 Dec 2020 16:48:57 +0100 Andrew Lunn wrote:
> On Thu, Dec 03, 2020 at 03:44:22PM +0000, Mickey Rachamim wrote:
> > On Thu, Dec 03, 2020 at 04:36:00PM +0200, Andrew Lunn wrote:  
> > >> Add maintainers info for new Marvell Prestera Ethernet switch driver.
> > >> 
> > >> Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
> > >> ---
> > >>  MAINTAINERS | 9 +++++++++
> > >>  1 file changed, 9 insertions(+)  
> >   
> > > Hi Mickey
> > > 
> > > All the commits came from
> > > 
> > > Vadym Kochan <vadym.kochan@plvision.eu>
> > > 
> > > Has Marvell purchased PL Vision?
> > > 
> > >     Andrew  
> > 
> > Hi Andrew, 
> > 
> > No, Marvell didn't purchase PLVision and I can understand the reason for thinking it.
> > PLVision and Marvell teams are keep working together as partners on supporting this program.
> > Vadym Kochan is and will remain an important contributor on this program.  
> 
> So Vadym Kochan is still at PLVision? Please use PLVision email
> addresses for people at PLVision, and Marvell email addresses for
> people at Marvell. We don't want Marketing in the MAINTAINER file.

100%. 

And Vadym is the only person that should be listed there as far as 
I'm concerned. I'm intending to purge MAINTAINERS from all people 
who haven't reviewed patches in the last 3 or so years, so let's 
save ourselves the back and forth.

I'd suggest you create your own mailing list and add it under 
the L: entry, that way all managers and people within company 
who want to keep an eye on upstream can subscribe.

IMO MAINTAINERS is for listing code reviewers.

RE: [EXT] Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Mickey Rachamim]

> > 
> > > Hi Mickey
> > > 
> > > All the commits came from
> > > 
> > > Vadym Kochan <vadym.kochan@plvision.eu>
> > > 
> > > Has Marvell purchased PL Vision?
> > > 
> > >     Andrew
> > 
> > Hi Andrew,
> > 
> > No, Marvell didn't purchase PLVision and I can understand the reason for thinking it.
> > PLVision and Marvell teams are keep working together as partners on supporting this program.
> > Vadym Kochan is and will remain an important contributor on this program.
> 
> So Vadym Kochan is still at PLVision? Please use PLVision email addresses for people at PLVision, and Marvell email addresses for people at Marvell. We don't want Marketing in the MAINTAINER file.
> 
>       Andrew

Hi Andrew,
PLVision team is working under signed contract with Marvell hence they will use Marvell emails.
Regards, Mickey

RE: [EXT] Re: [PATCH] MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

[Mickey Rachamim]

> On Thu, 3 Dec 2020 16:48:57 +0100 Andrew Lunn wrote:
> > On Thu, Dec 03, 2020 at 03:44:22PM +0000, Mickey Rachamim wrote:
> > > On Thu, Dec 03, 2020 at 04:36:00PM +0200, Andrew Lunn wrote:  
> > > >> Add maintainers info for new Marvell Prestera Ethernet switch driver.
> > > >> 
> > > >> Signed-off-by: Mickey Rachamim <mickeyr@marvell.com>
> > > >> ---
> > > >>  MAINTAINERS | 9 +++++++++
> > > >>  1 file changed, 9 insertions(+)
> > >   
> > > > Hi Mickey
> > > > 
> > > > All the commits came from
> > > > 
> > > > Vadym Kochan <vadym.kochan@plvision.eu>
> > > > 
> > > > Has Marvell purchased PL Vision?
> > > > 
> > > >     Andrew
> > > 
> > > Hi Andrew,
> > > 
> > > No, Marvell didn't purchase PLVision and I can understand the reason for thinking it.
> > > PLVision and Marvell teams are keep working together as partners on supporting this program.
> > > Vadym Kochan is and will remain an important contributor on this program.  
> > 
> > So Vadym Kochan is still at PLVision? Please use PLVision email 
> > addresses for people at PLVision, and Marvell email addresses for 
> > people at Marvell. We don't want Marketing in the MAINTAINER file.
> 
> 100%. 
> 
> And Vadym is the only person that should be listed there as far as I'm concerned. I'm intending to purge MAINTAINERS from all people who haven't reviewed patches in the last 3 or so years, so let's save ourselves the back and forth.
> 
> I'd suggest you create your own mailing list and add it under the L: entry, that way all managers and people within company who want to keep an eye on upstream can subscribe.
> 
> IMO MAINTAINERS is for listing code reviewers.
> 
Hi Jakub,

Vadym will remain the main contributor but the new upcoming support will include additional engineers.
The intention is that all the three M will review the code but we OK also with two.
Additional features will be uploaded soon and will require more than one M.
We'll update the patch accordingly.

Regards, Mickey.