* [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier @ 2018-07-18 20:02 ` Krzysztof Kozlowski 2018-07-24 16:46 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2018-07-18 20:02 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz, Zhang Rui, Eduardo Valentin, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, Krzysztof Kozlowski, devicetree, linux-arm-kernel Replace GPL license statement with SPDX license identifier (GPL-2.0+). Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- 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 <l.majewski@samsung.com> - * - * 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 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier 2018-07-18 20:02 ` [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier Krzysztof Kozlowski @ 2018-07-24 16:46 ` Bartlomiej Zolnierkiewicz 2018-07-24 20:03 ` Krzysztof Kozlowski 0 siblings, 1 reply; 6+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2018-07-24 16:46 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Zhang Rui, Eduardo Valentin, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, devicetree, linux-arm-kernel 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@kernel.org> > --- > 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 <l.majewski@samsung.com> > - * > - * 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier 2018-07-24 16:46 ` Bartlomiej Zolnierkiewicz @ 2018-07-24 20:03 ` Krzysztof Kozlowski 2018-07-25 14:43 ` Bartlomiej Zolnierkiewicz 0 siblings, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2018-07-24 20:03 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz Cc: Zhang Rui, Eduardo Valentin, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, devicetree, linux-arm-kernel 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@kernel.org> > > --- > > 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier 2018-07-24 20:03 ` Krzysztof Kozlowski @ 2018-07-25 14:43 ` Bartlomiej Zolnierkiewicz 2018-07-25 16:27 ` Krzysztof Kozlowski 0 siblings, 1 reply; 6+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2018-07-25 14:43 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Zhang Rui, Eduardo Valentin, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, devicetree, linux-arm-kernel 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@kernel.org> > > > --- > > > 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier 2018-07-25 14:43 ` Bartlomiej Zolnierkiewicz @ 2018-07-25 16:27 ` Krzysztof Kozlowski 2018-07-27 22:53 ` Eduardo Valentin 0 siblings, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2018-07-25 16:27 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz Cc: Zhang Rui, Eduardo Valentin, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, devicetree, linux-arm-kernel On 25 July 2018 at 16:43, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> 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 <krzk@kernel.org> >> > > --- >> > > 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier 2018-07-25 16:27 ` Krzysztof Kozlowski @ 2018-07-27 22:53 ` Eduardo Valentin 0 siblings, 0 replies; 6+ messages in thread From: Eduardo Valentin @ 2018-07-27 22:53 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Bartlomiej Zolnierkiewicz, Zhang Rui, linux-pm, linux-samsung-soc, linux-kernel, Rob Herring, Mark Rutland, Kukjin Kim, devicetree, linux-arm-kernel On Wed, Jul 25, 2018 at 06:27:59PM +0200, Krzysztof Kozlowski wrote: > On 25 July 2018 at 16:43, Bartlomiej Zolnierkiewicz > <b.zolnierkie@samsung.com> 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 <krzk@kernel.org> > >> > > --- > >> > > 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-07-27 22:53 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20180718200318epcas3p16aed6c1725d6c1e63ea6b1ad209c5756@epcas3p1.samsung.com>
2018-07-18 20:02 ` [PATCH] dt-bindings: thermal: samsung: Add SPDX license identifier Krzysztof Kozlowski
2018-07-24 16:46 ` Bartlomiej Zolnierkiewicz
2018-07-24 20:03 ` Krzysztof Kozlowski
2018-07-25 14:43 ` Bartlomiej Zolnierkiewicz
2018-07-25 16:27 ` Krzysztof Kozlowski
2018-07-27 22:53 ` Eduardo Valentin
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).