linux-kernel - Re: [PATCH RESEND v5 1/1] clk: mstar: msc313 cpupll clk driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20220930193535.GB9469@1wt.eu>
Date:   Fri, 30 Sep 2022 21:35:35 +0200
From:   Willy Tarreau <w@....eu>
To:     Stephen Boyd <sboyd@...nel.org>
Cc:     Daniel Palmer <daniel@...f.com>,
        Michael Turquette <mturquette@...libre.com>,
        Romain Perier <romain.perier@...il.com>,
        linux-clk@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH RESEND v5 1/1] clk: mstar: msc313 cpupll clk driver

Hi Stephen,

On Fri, Sep 30, 2022 at 12:17:40PM -0700, Stephen Boyd wrote:
> Quoting Romain Perier (2022-06-03 12:05:09)
> > From: Daniel Palmer <daniel@...f.com>
> > 
> > Add a driver for the CPU pll/ARM pll/MIPS pll that is present
> > in MStar SoCs.
> > 
> > Currently there is no documentation for this block so it's possible
> > this driver isn't entirely correct.
> > 
> > Only tested on the version of this IP in the MStar/SigmaStar
> > ARMv7 SoCs.
> > 
> > Co-authored-by: Willy Tarreau <w@....eu>
> 
> This is not a standard tag, maybe Co-developed-by is what you want?

Yeah it's the same. We're seeing 122 co-authored-by tags in the
kernel's history vs 3122 co-developed-by, so that's only 3% but
at least not a fantasist one..

> A Signed-off-by tag should be here from Willy Tarreau then.

Not all commits follow this, but indeed some do. Regardless I
really don't care at all about my name being listed there.

> > Signed-off-by: Daniel Palmer <daniel@...f.com>
> 
> Your Signed-off-by needs to be here. I can't apply this otherwise.

Quite frankly, we're talking about an old patch that was first
submitted 9 months ago (!), let's not postpone it even further just
for headers reason, please just drop this line mentioning my
little contribution, I don't care a single second at all, really.

Thanks,
Willy