diff mbox series

MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver

Message ID 20201203100436.25630-1-mickeyr@marvell.com (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series MAINTAINERS: Add entry for Marvell Prestera Ethernet Switch driver | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Mickey Rachamim Dec. 3, 2020, 10:04 a.m. UTC
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(+)

Comments

Andrew Lunn Dec. 3, 2020, 2:35 p.m. UTC | #1
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
Mickey Rachamim Dec. 3, 2020, 3:44 p.m. UTC | #2
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.
Andrew Lunn Dec. 3, 2020, 3:48 p.m. UTC | #3
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
Jakub Kicinski Dec. 3, 2020, 5:59 p.m. UTC | #4
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.
Mickey Rachamim Dec. 3, 2020, 7:31 p.m. UTC | #5
> > 
> > > 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
Mickey Rachamim Dec. 3, 2020, 8:01 p.m. UTC | #6
> 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.
diff mbox series

Patch

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