2018-07-18 20:05:41

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier

Replace GPL license statement with SPDX license identifier (GPL-2.0+).

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
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+ */
/*
* thermal_exynos.h - Samsung EXYNOS TMU device tree definitions
*
* Copyright (C) 2014 Samsung Electronics
* Lukasz Majewski <[email protected]>
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
*/

#ifndef _EXYNOS_THERMAL_TMU_DT_H
--
2.14.1



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


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 <[email protected]>
> ---
> 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.

> /*
> * thermal_exynos.h - Samsung EXYNOS TMU device tree definitions
> *
> * Copyright (C) 2014 Samsung Electronics
> * Lukasz Majewski <[email protected]>
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> - * GNU General Public License for more details.
> - *
> */
>
> #ifndef _EXYNOS_THERMAL_TMU_DT_H

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


2018-07-24 20:05:08

by Krzysztof Kozlowski

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

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 <[email protected]>
> > ---
> > 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?

Best regards,
Krzysztof


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

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 <[email protected]>
> > > ---
> > > 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


2018-07-25 16:29:42

by Krzysztof Kozlowski

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

On 25 July 2018 at 16:43, Bartlomiej Zolnierkiewicz
<[email protected]> wrote:
> 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 <[email protected]>
>> > > ---
>> > > 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)?

I do not have interest in changing the license, I do not want to deal
with possible legal issues so I'll skip it.

I am fine however with abandoning this patch if you or someone else
sends license change + SPDX.

Best regards,
Krzysztof

2018-07-27 22:54:28

by Eduardo Valentin

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

On Wed, Jul 25, 2018 at 06:27:59PM +0200, Krzysztof Kozlowski wrote:
> On 25 July 2018 at 16:43, Bartlomiej Zolnierkiewicz
> <[email protected]> wrote:
> > 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 <[email protected]>
> >> > > ---
> >> > > 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.

You probably want to have his ack yes.

> >
> > 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)?
>
> I do not have interest in changing the license, I do not want to deal
> with possible legal issues so I'll skip it.
>

:-)

> I am fine however with abandoning this patch if you or someone else
> sends license change + SPDX.
>

If that is the case, I am skipping this patch. Ping me / RESEND in case
you still want this. :-)

> Best regards,
> Krzysztof