* dt-bindings: Invalid SPDX License Identifier
@ 2018-04-22 19:14 Thomas Gleixner
2018-04-23 7:21 ` Yixun Lan
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Gleixner @ 2018-04-22 19:14 UTC (permalink / raw)
To: LKML; +Cc: Yixun Lan, Neil Armstrong, Philipp Zabel, Kate Stewart, Greg KH
Yixun,
commit c16292578ffa437c77e17d5d3da20330a10b8a86
Author: Yixun Lan <yixun.lan@amlogic.com>
Date: Fri Nov 10 16:46:49 2017 +0800
dt-bindings: reset: Add bindings for the Meson-AXG SoC Reset Controller
Add DT bindings for the Meson-AXG SoC Reset Controller include file,
and also slightly update documentation.
Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Added the following SPDX-License-Identifier: (GPL-2.0+ OR BSD) in the
middle of the top of file comment.
There are two problems with that:
1) BSD is not a valid SPDX identifier. See https://spdx.org/licenses/
You need to say which BSD variant you mean.
2) Please put the License identifier to the place which is defined in
Documentation/process/licenses.rst, i.e. the first line of the dts file
and not to some random place.
Thanks,
tglx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: dt-bindings: Invalid SPDX License Identifier
2018-04-22 19:14 dt-bindings: Invalid SPDX License Identifier Thomas Gleixner
@ 2018-04-23 7:21 ` Yixun Lan
0 siblings, 0 replies; 3+ messages in thread
From: Yixun Lan @ 2018-04-23 7:21 UTC (permalink / raw)
To: Thomas Gleixner, LKML
Cc: yixun.lan, Neil Armstrong, Philipp Zabel, Kate Stewart, Greg KH,
linux-amlogic
Hi Thomas
On 04/23/18 03:14, Thomas Gleixner wrote:
> Yixun,
>
> commit c16292578ffa437c77e17d5d3da20330a10b8a86
> Author: Yixun Lan <yixun.lan@amlogic.com>
> Date: Fri Nov 10 16:46:49 2017 +0800
>
> dt-bindings: reset: Add bindings for the Meson-AXG SoC Reset Controller
>
> Add DT bindings for the Meson-AXG SoC Reset Controller include file,
> and also slightly update documentation.
>
> Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
> Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> Added the following SPDX-License-Identifier: (GPL-2.0+ OR BSD) in the
> middle of the top of file comment.
>
> There are two problems with that:
>
> 1) BSD is not a valid SPDX identifier. See https://spdx.org/licenses/
>
> You need to say which BSD variant you mean.
>
> 2) Please put the License identifier to the place which is defined in
> Documentation/process/licenses.rst, i.e. the first line of the dts file
> and not to some random place.
>
thanks for pointing this out
I will adjust the license to "GPL-2.0+ OR BSD-3-Clause" since this file
was copied from include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
and it's a good idea to preserve the old license.
Yixun
^ permalink raw reply [flat|nested] 3+ messages in thread
* dt-bindings: Invalid SPDX License Identifier
@ 2018-04-23 7:21 ` Yixun Lan
0 siblings, 0 replies; 3+ messages in thread
From: Yixun Lan @ 2018-04-23 7:21 UTC (permalink / raw)
To: linus-amlogic
Hi Thomas
On 04/23/18 03:14, Thomas Gleixner wrote:
> Yixun,
>
> commit c16292578ffa437c77e17d5d3da20330a10b8a86
> Author: Yixun Lan <yixun.lan@amlogic.com>
> Date: Fri Nov 10 16:46:49 2017 +0800
>
> dt-bindings: reset: Add bindings for the Meson-AXG SoC Reset Controller
>
> Add DT bindings for the Meson-AXG SoC Reset Controller include file,
> and also slightly update documentation.
>
> Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
> Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> Added the following SPDX-License-Identifier: (GPL-2.0+ OR BSD) in the
> middle of the top of file comment.
>
> There are two problems with that:
>
> 1) BSD is not a valid SPDX identifier. See https://spdx.org/licenses/
>
> You need to say which BSD variant you mean.
>
> 2) Please put the License identifier to the place which is defined in
> Documentation/process/licenses.rst, i.e. the first line of the dts file
> and not to some random place.
>
thanks for pointing this out
I will adjust the license to "GPL-2.0+ OR BSD-3-Clause" since this file
was copied from include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
and it's a good idea to preserve the old license.
Yixun
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-04-23 7:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-22 19:14 dt-bindings: Invalid SPDX License Identifier Thomas Gleixner
2018-04-23 7:21 ` Yixun Lan
2018-04-23 7:21 ` Yixun Lan
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.