All of lore.kernel.org
 help / color / mirror / Atom feed
From: Willy Tarreau <w@1wt.eu>
To: Stephen Boyd <sboyd@kernel.org>
Cc: Daniel Palmer <daniel@0x0f.com>,
	Michael Turquette <mturquette@baylibre.com>,
	Romain Perier <romain.perier@gmail.com>,
	linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND v5 1/1] clk: mstar: msc313 cpupll clk driver
Date: Fri, 30 Sep 2022 21:35:35 +0200	[thread overview]
Message-ID: <20220930193535.GB9469@1wt.eu> (raw)
In-Reply-To: <20220930191742.9A9FEC433C1@smtp.kernel.org>

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@0x0f.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@1wt.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@0x0f.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

WARNING: multiple messages have this Message-ID (diff)
From: Willy Tarreau <w@1wt.eu>
To: Stephen Boyd <sboyd@kernel.org>
Cc: Daniel Palmer <daniel@0x0f.com>,
	Michael Turquette <mturquette@baylibre.com>,
	Romain Perier <romain.perier@gmail.com>,
	linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH RESEND v5 1/1] clk: mstar: msc313 cpupll clk driver
Date: Fri, 30 Sep 2022 21:35:35 +0200	[thread overview]
Message-ID: <20220930193535.GB9469@1wt.eu> (raw)
In-Reply-To: <20220930191742.9A9FEC433C1@smtp.kernel.org>

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@0x0f.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@1wt.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@0x0f.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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-09-30 19:37 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-03 19:05 [PATCH RESEND v5 0/1] ARM: mstar: cpupll Romain Perier
2022-06-03 19:05 ` Romain Perier
2022-06-03 19:05 ` [PATCH RESEND v5 1/1] clk: mstar: msc313 cpupll clk driver Romain Perier
2022-06-03 19:05   ` Romain Perier
2022-09-30 19:17   ` Stephen Boyd
2022-09-30 19:17     ` Stephen Boyd
2022-09-30 19:35     ` Willy Tarreau [this message]
2022-09-30 19:35       ` Willy Tarreau
2022-10-05  7:33       ` Romain Perier
2022-10-05  7:33         ` Romain Perier
2022-10-05  7:34         ` Willy Tarreau
2022-10-05  7:34           ` Willy Tarreau
2022-10-05  7:35     ` Romain Perier
2022-10-05  7:35       ` Romain Perier

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:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

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

  git send-email \
    --in-reply-to=20220930193535.GB9469@1wt.eu \
    --to=w@1wt.eu \
    --cc=daniel@0x0f.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=romain.perier@gmail.com \
    --cc=sboyd@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.