Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier

[Bartlomiej Zolnierkiewicz]

On Tuesday, July 24, 2018 10:03:12 PM Krzysztof Kozlowski wrote:
> On Tue, Jul 24, 2018 at 06:46:35PM +0200, Bartlomiej Zolnierkiewicz wrote:
> > 
> > Hi,
> > 
> > On Wednesday, July 18, 2018 10:02:50 PM Krzysztof Kozlowski wrote:
> > > Replace GPL license statement with SPDX license identifier (GPL-2.0+).
> > > 
> > > Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> > > ---
> > >  include/dt-bindings/thermal/thermal_exynos.h | 12 +-----------
> > >  1 file changed, 1 insertion(+), 11 deletions(-)
> > > 
> > > diff --git a/include/dt-bindings/thermal/thermal_exynos.h b/include/dt-bindings/thermal/thermal_exynos.h
> > > index 0646500bca69..642e4e7f4084 100644
> > > --- a/include/dt-bindings/thermal/thermal_exynos.h
> > > +++ b/include/dt-bindings/thermal/thermal_exynos.h
> > > @@ -1,19 +1,9 @@
> > > +/* SPDX-License-Identifier: GPL-2.0+ */
> > 
> > This code should be GPL-2.0 only, original license statement is most
> > likely result of using wrong file as the source of copy-paste. So I
> > would like to have the license fixed first before doing conversion to
> > SPDX.
> 
> If you think that's the case, then it makes sense and please go ahead.
> As fair as I remember this required signed-off from all contributors...
> so probably just Lukasz and Eduardo?

git blame shows just Lukasz so his Ack should be enough.

I'm also fine with doing it in one patch like i.e. in commit a80ea4cb944e
("pata_bk3710: clarify license version and use SPDX header"), if you are
also fine with it could you please re-spin the patch (+Cc: Lukasz)?

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics