archive mirror
 help / color / mirror / Atom feed
From: Andrew Lunn <>
To: Taras Chornyi <>
Cc: Jakub Kicinski <>,
	Vadym Kochan <>,
	"David S. Miller" <>,
	Paolo Abeni <>,, Elad Nachman <>,
	Mickey Rachamim <>,,
	Taras Chornyi <>
Subject: Re: [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera Ethernet Switch driver
Date: Thu, 1 Dec 2022 21:43:38 +0100	[thread overview]
Message-ID: <Y4kR+qxXCnM/> (raw)
In-Reply-To: <>

On Thu, Dec 01, 2022 at 10:39:07AM +0200, Taras Chornyi wrote:
> On 30.11.22 07:14, Jakub Kicinski wrote:
> > On Mon, 28 Nov 2022 11:39:34 +0200 Vadym Kochan wrote:
> > > Add Elad Nachman as maintainer for Marvell Prestera Ethernet Switch driver.
> > > 
> > > Change Taras Chornyi mailbox to plvision.
> > This is a patch, so the description needs to explain why...
> > and who these people are. It would seem more natural if you,
> > Oleksandr and Yevhen were the maintainers.
> > 
> > Seriously, this is a community project please act the part.
> The Marvell Prestera Switchdev Kernel Driver's focus and maintenance are
> shifted from PLVision (Marvell Contractors) to the Marvell team in Israel.
> In the last 12 months, the driver's development efforts have been shared
> between the PLVision team and Elad Nachman from the Marvell Israel group.
> Elad Nachman is a veteran with over ten years of experience in Linux kernel
> development.
> He has made many Linux kernel contributions to several community projects,
> including the Linux kernel, DPDK (KNI Linux Kernel driver) and the DENT
> project.
> Elad has done reviews and technical code contributions on Armada 3700,
> Helping Pali Rohár, who is the maintainer of the Armada 3700 PCI sub-system,
> as well as others in the Armada 3700 cpufreq sub-system.
> In the last year and a half, Elad has internally dealt extensively with the
> Marvell Prestera sub-system and has led various upstreaming sub-projects
> related to the Prestera sub-system, Including Prestera sub-system efforts
> related to the Marvell AC5/X SOC drivers upstreaming. This included
> technical review and guidance on the technical aspects and code content of
> the patches sent for review.
> In addition, Elad is a member of the internal review group of code before it
> applies as a PR.

Hi Taras

The problem we have is that all this is totally opaque to us, in
netdev. The name Elad Nachman does not appear on a single patch in
mainline git for the Prestera. All i can find anywhere in mainline is
one patch in 2018 for stmmac.

The community bases a lot of its judgements on trust.  We look at
contributions, be it patches, review comments, or helping others
finding bugs etc, and slowly build up a level of trust. And at the
moment there is nothing to base trust on.

So lets wait until there are a number of merged patchsets, trust has
been established, before adding this entry. It should not take too
long, given his level of experience. As i've said before, you become a
Maintainer by being a Maintainer.


  reply	other threads:[~2022-12-01 20:44 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-28  9:39 [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera Ethernet Switch driver Vadym Kochan
2022-11-30  5:14 ` Jakub Kicinski
2022-12-01  8:39   ` Taras Chornyi
2022-12-01 20:43     ` Andrew Lunn [this message]
2022-12-01 21:17     ` Jakub Kicinski
2022-12-07 12:36       ` Taras Chornyi
2022-12-07 16:53         ` Jakub Kicinski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Y4kR+qxXCnM/ \ \ \ \ \ \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).