From: Andrew Lunn <firstname.lastname@example.org>
To: Taras Chornyi <email@example.com>
Cc: Jakub Kicinski <firstname.lastname@example.org>,
Vadym Kochan <email@example.com>,
"David S. Miller" <firstname.lastname@example.org>,
Paolo Abeni <email@example.com>,
firstname.lastname@example.org, Elad Nachman <email@example.com>,
Mickey Rachamim <firstname.lastname@example.org>,
Taras Chornyi <email@example.com>
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+qxXCnMfirstname.lastname@example.org> (raw)
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
> He has made many Linux kernel contributions to several community projects,
> including the Linux kernel, DPDK (KNI Linux Kernel driver) and the DENT
> 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.
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.
next prev parent 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
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 \
* 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).