Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: Salvatore Bonaccorso <salvatore.bonaccorso@gmail.com>
Date:   Sun, 16 May 2021 18:58:00 +0200
From:   Salvatore Bonaccorso <carnil@debian.org>
To:     Jernej =?utf-8?Q?=C5=A0krabec?= <jernej.skrabec@siol.net>
Cc:     Maxime Ripard <maxime@cerno.tech>,
        "B.R. Oake" <broake@mailfence.com>, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com,
        Rob Herring <robh+dt@kernel.org>,
        Chen-Yu Tsai <wens@kernel.org>,
        linux-arm-kernel@lists.infradead.org,
        Vagrant Cascadian <vagrant@reproducible-builds.org>
Subject: Re: [PATCH] ARM: dts: sun8i: h3: orangepi-plus: Fix Ethernet PHY mode
Message-ID: <YKFPGC2qBMipQPbd@eldamar.lan>
References: <1243888060.510560.1612783497400@ichabod.co-bxl>
 <20210210150118.ly252i37eykayrcb@gilmour>
 <1719200956.433094.1613199092092@ichabod.co-bxl>
 <6612268.HtAl026vyE@jernej-laptop>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <6612268.HtAl026vyE@jernej-laptop>
Precedence: bulk

Hi,

On Sat, Feb 13, 2021 at 09:51:17AM +0100, Jernej Škrabec wrote:
> Hi!
> 
> Let me first explain that it was oversight on my side not noticing initials in 
> your SoB tag. But since the issue was raised by Maxime, I didn't follow up.
> 
> Dne sobota, 13. februar 2021 ob 07:51:32 CET je B.R. Oake napisal(a):
> > On Wed Feb 10 at 16:01:18 CET 2021, Maxime Ripard wrote:
> > > Unfortunately we can't take this patch as is, this needs to be your real
> > > name, see:
> > > https://www.kernel.org/doc/html/latest/process/submitting-patches.html#de
> > > veloper-s-certificate-of-origin-1-1
> > Dear Maxime,
> > 
> > Thank you very much for considering my contribution and for all your
> > work on supporting sunxi-based hardware; I appreciate it.
> > 
> > Thank you for referring me to the Developer's Certificate of Origin, but
> > I had already read it before submitting (I had to do so in order to know
> > what I was saying by "Signed-off-by:") and I do certify what it says.
> > 
> > Looking through recent entries in the commit log of the mainline kernel,
> > I see several patches from authors such as:
> > 
> >   H.J. Lu <hjl.tools@gmail.com>
> >   B K Karthik <karthik.bk2000@live.com>
> >   JC Kuo <jckuo@nvidia.com>
> >   EJ Hsu <ejh@nvidia.com>
> >   LH Lin <lh.lin@mediatek.com>
> >   KP Singh <kpsingh@kernel.org>
> >   Karthik B S <karthik.b.s@intel.com>
> >   Shreyas NC <shreyas.nc@intel.com>
> >   Vandana BN <bnvandana@gmail.com>
> > 
> > so I believe names of this form are in fact acceptable, even if the
> > style might seem a little old-fashioned to some.
> 
> Speaking generally, not only for this case, prior art arguments rarely hold, 
> because:
> - it might be oversight,
> - it might be a bad practice, which should not be followed in new 
> contributions,
> - different maintainers have different point of view on same thing,
> - maintainer wants to adapt new practice or steer subsystem in new direction
> 
> > 
> > I would like to add that I have met many people with names such as C.J.,
> > A A, TC, MG, etc. That is what everybody calls them and it would be
> > natural for them to sign themselves that way. Some of them might want to
> > contribute to Linux some day, and I think it would be a great shame and
> > a loss to all of us if they were discouraged from doing so by reading
> > our conversation in the archives and concluding that any contribution
> > from them, however small, would be summarily refused simply because of
> > their name. Please could you ensure that does not happen?
> 
> The link you posted says following:
> "using your real name (sorry, no pseudonyms or anonymous contributions.)"
> 
> I believe that real name means no initials, no matter what people are 
> accustomed to. From my point of view, CJ is pseudonym derived from real name.
> 
> This is not the first time that fix of SoB tag was requested, you can find such 
> requests in ML archives.

Any chance this could be resolved? In downstream (here Debian) we
would be interested to have the patch applied due to
https://bugs.debian.org/988574 reported by Vagrant Cascadian.

Regards,
Salvatore