From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: "Andreas Färber" <afaerber@suse.de>,
robh+dt@kernel.org, liuwei@actions-semi.com,
96boards@ucrobotics.com, devicetree@vger.kernel.org,
amit.kucheria@linaro.org, hzhang@ucrobotics.com,
bdong@ucrobotics.com, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, manivannanece23@gmail.com,
"Jeff Chen" <jeff.chen@actions-semi.com>,
"Thomas C. Liau" <thomas.liau@actions-semi.com>
Subject: Re: [PATCH 1/4] dt-bindings: power: Add Actions Semi S900 SPS
Date: Wed, 27 Jun 2018 20:40:52 +0530 [thread overview]
Message-ID: <20180627151052.GA24330@mani> (raw)
In-Reply-To: <20180627145713.uao2gr5kknk2cypk@holly.lan>
On Wed, Jun 27, 2018 at 03:57:13PM +0100, Daniel Thompson wrote:
> On Tue, Jun 26, 2018 at 10:11:17PM +0530, Manivannan Sadhasivam wrote:
> > > > diff --git a/include/dt-bindings/power/owl-s900-powergate.h b/include/dt-bindings/power/owl-s900-powergate.h
> > > > new file mode 100644
> > > > index 000000000000..f1aaf761112b
> > > > --- /dev/null
> > > > +++ b/include/dt-bindings/power/owl-s900-powergate.h
> > > > @@ -0,0 +1,23 @@
> > > > +// SPDX-License-Identifier: GPL-2.0+
> > >
> > > checkpatch.pl warns about this line due to not using /* ... */ syntax
> > > for headers. I could fix that up on my own, but it made me realize that
> > > you are licensing this file under GPL-2.0+ only, whereas the .dts[i] is
> > > supposed to be dual-licensed.
> > >
> > > Can you please give your consent to make this
> > >
> > > /* SPDX-License-Identifier: (GPL-2.0+ OR MIT) */
> > >
> >
> > You can change the license to GPL-2.0+ OR MIT.
> >
> > > like my S500 and S700 bindings?
> > >
> > > Question: Should it still be GPL-2.0+ or the new GPL-2.0-or-later?
> > >
> >
> > I think GPL-2.0+ is fine.
>
> So... when I saw this I did wonder why one would knowly choose to use a
> deprecated license identifier (https://spdx.org/licenses/ ).
>
> However running `git diff v4.17..v4.18-rc1` through some filters does
> indicate that GPL-2.0+ is the more popular choice by 256 to 0! Doesn't
> mean you couldn't break the mould though ;-).
>
I did grep through the source and hit with same numbers ;-) But I was not
sure whether to break the mould or not, so chose the safest path :)
Since you already gave the green signal, I think we can move to new style
identifier.
Thanks,
Mani
>
> Daniel.
next prev parent reply other threads:[~2018-06-27 15:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-11 16:40 [PATCH 0/4] Add Actions Semi S900 SPS support Manivannan Sadhasivam
2018-04-11 16:40 ` [PATCH 1/4] dt-bindings: power: Add Actions Semi S900 SPS Manivannan Sadhasivam
2018-04-16 18:48 ` Rob Herring
2018-06-24 13:04 ` Andreas Färber
2018-06-26 16:41 ` Manivannan Sadhasivam
2018-06-27 14:57 ` Daniel Thompson
2018-06-27 15:05 ` Andreas Färber
2018-06-27 15:10 ` Manivannan Sadhasivam [this message]
2018-04-11 16:40 ` [PATCH 2/4] arm64: dts: actions: Add S900 SPS node Manivannan Sadhasivam
2018-06-24 13:06 ` Andreas Färber
2018-04-11 16:40 ` [PATCH 3/4] soc: actions: sps: Add S900 power domains Manivannan Sadhasivam
2018-06-24 13:07 ` Andreas Färber
2018-09-30 15:46 ` Andreas Färber
2018-04-11 16:40 ` [PATCH 4/4] soc: actions: Fix help text in Kconfig for OWL_PM_DOMAINS Manivannan Sadhasivam
2018-06-24 13:28 ` Andreas Färber
2018-06-24 14:01 ` [PATCH] soc: actions: Update SPS help text for S700 Andreas Färber
2018-09-22 20:11 ` Andreas Färber
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=20180627151052.GA24330@mani \
--to=manivannan.sadhasivam@linaro.org \
--cc=96boards@ucrobotics.com \
--cc=afaerber@suse.de \
--cc=amit.kucheria@linaro.org \
--cc=bdong@ucrobotics.com \
--cc=daniel.thompson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=hzhang@ucrobotics.com \
--cc=jeff.chen@actions-semi.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=liuwei@actions-semi.com \
--cc=manivannanece23@gmail.com \
--cc=robh+dt@kernel.org \
--cc=thomas.liau@actions-semi.com \
/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 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).