Re: [PATCH 2/2] mfd: intel_soc_pmic: Differentiate between Bay and Cherry Trail CRC variants
From: Andy Shevchenko
Date: Fri Jul 21 2017 - 15:23:14 EST
On Fri, 2017-07-21 at 20:41 +0200, Hans de Goede wrote:
> On 18-07-17 09:17, Lee Jones wrote:
> > On Mon, 17 Jul 2017, Andy Shevchenko wrote:
> >
> > > On Mon, 2017-07-17 at 11:59 +0100, Lee Jones wrote:
> > > > On Sat, 01 Jul 2017, Hans de Goede wrote:
> > > >
> > > > > Reported-and-tested-by: russianneuromancer
> > > > > <russianneuromancer@xxxxx
> > > > > >
> > > >
> > > > Real names only please.
> > > >
> > > > What is the name of this reporter/tester?
> > >
> > > No one knows, I think.
> >
> > Then I don't think we can credit him/her for their efforts.
> > SubmittingPatches clearly states "no pseudonyms or anonymous
> > contributions".ÂÂEither write to them and ask for their real name so
> > that they can be credited, or remove the line.
>
> russianneuromancer has been a great help with reporting and testing
> many Bay Trail / Cherry Trail issues. He does not want to use his
> real name.
>
> AFAIK the Real Name rule only applies tot he Signed-off-by tag, no
> other subsys-maintainers have had problems with listing him as
> reporter / tester using his alias.
>
> The Real Name only rule is part of "11) Sign your work â the
> Developerâs Certificate of Origin" of :
>
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html
> #sign-your-work-the-developer-s-certificate-of-origin
>
> Which is all about the S-o-b tag. If you despite this still want
> a new version with russianneuromancer's Reported-and-tested-by
> dropped let me know and I will send a new version.
I saw some patches applied just with email in tags like Reported-by.
So, I think everyone will be okay if you remove nick and leave email
only.
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy