linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] selftests/ftrace: Fix invalid SPDX identifiers
@ 2018-11-12 19:11 Thomas Gleixner
  2018-11-29 21:20 ` Thomas Gleixner
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Gleixner @ 2018-11-12 19:11 UTC (permalink / raw)
  To: LKML; +Cc: Shuah Khan, Masami Hiramatsu

While GPL2.0 looks about right, the correct and valid identifiers for GPL v2
only code are 'GPL-2.0' or 'GPL-2.0-only'.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Shuah Khan (Samsung OSG) <shuah@kernel.org>

---

P.S.: checkpatch.pl emits a warning when crappy license ids are used

 tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc |    2 +-
 tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc           |    2 +-
 tools/testing/selftests/ftrace/test.d/template                         |    2 +-
 tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc                 |    2 +-
 tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc              |    2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

--- a/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc
+++ b/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc
@@ -1,5 +1,5 @@
 #!/bin/sh
-# SPDX-License-Identifier: GPL2.0
+# SPDX-License-Identifier: GPL-2.0
 # description: ftrace - stacktrace filter command
 # flags: instance
 
--- a/tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc
+++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc
@@ -1,5 +1,5 @@
 #!/bin/sh
-# SPDX-License-Identifier: GPL2.0
+# SPDX-License-Identifier: GPL-2.0
 # description: ftrace - function trace with cpumask
 
 if ! which nproc ; then
--- a/tools/testing/selftests/ftrace/test.d/template
+++ b/tools/testing/selftests/ftrace/test.d/template
@@ -1,5 +1,5 @@
 #!/bin/sh
-# SPDX-License-Identifier: GPL2.0
+# SPDX-License-Identifier: GPL-2.0
 # description: %HERE DESCRIBE WHAT THIS DOES%
 # you have to add ".tc" extention for your testcase file
 # Note that all tests are run with "errexit" option.
--- a/tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc
+++ b/tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc
@@ -1,5 +1,5 @@
 #!/bin/sh
-# SPDX-License-Identifier: GPL2.0
+# SPDX-License-Identifier: GPL-2.0
 # description: Test wakeup tracer
 
 if ! which chrt ; then
--- a/tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc
+++ b/tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc
@@ -1,5 +1,5 @@
 #!/bin/sh
-# SPDX-License-Identifier: GPL2.0
+# SPDX-License-Identifier: GPL-2.0
 # description: Test wakeup RT tracer
 
 if ! which chrt ; then

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] selftests/ftrace: Fix invalid SPDX identifiers
  2018-11-12 19:11 [PATCH] selftests/ftrace: Fix invalid SPDX identifiers Thomas Gleixner
@ 2018-11-29 21:20 ` Thomas Gleixner
  2018-11-29 21:54   ` shuah
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Gleixner @ 2018-11-29 21:20 UTC (permalink / raw)
  To: LKML; +Cc: Shuah Khan, Masami Hiramatsu

On Mon, 12 Nov 2018, Thomas Gleixner wrote:

Polite reminder....

> While GPL2.0 looks about right, the correct and valid identifiers for GPL v2
> only code are 'GPL-2.0' or 'GPL-2.0-only'.
> 
> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> Cc: Masami Hiramatsu <mhiramat@kernel.org>
> Cc: Shuah Khan (Samsung OSG) <shuah@kernel.org>
> 
> ---
> 
> P.S.: checkpatch.pl emits a warning when crappy license ids are used
> 
>  tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc |    2 +-
>  tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc           |    2 +-
>  tools/testing/selftests/ftrace/test.d/template                         |    2 +-
>  tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc                 |    2 +-
>  tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc              |    2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)
> 
> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc
> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func-filter-stacktrace.tc
> @@ -1,5 +1,5 @@
>  #!/bin/sh
> -# SPDX-License-Identifier: GPL2.0
> +# SPDX-License-Identifier: GPL-2.0
>  # description: ftrace - stacktrace filter command
>  # flags: instance
>  
> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc
> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_cpumask.tc
> @@ -1,5 +1,5 @@
>  #!/bin/sh
> -# SPDX-License-Identifier: GPL2.0
> +# SPDX-License-Identifier: GPL-2.0
>  # description: ftrace - function trace with cpumask
>  
>  if ! which nproc ; then
> --- a/tools/testing/selftests/ftrace/test.d/template
> +++ b/tools/testing/selftests/ftrace/test.d/template
> @@ -1,5 +1,5 @@
>  #!/bin/sh
> -# SPDX-License-Identifier: GPL2.0
> +# SPDX-License-Identifier: GPL-2.0
>  # description: %HERE DESCRIBE WHAT THIS DOES%
>  # you have to add ".tc" extention for your testcase file
>  # Note that all tests are run with "errexit" option.
> --- a/tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc
> +++ b/tools/testing/selftests/ftrace/test.d/tracer/wakeup.tc
> @@ -1,5 +1,5 @@
>  #!/bin/sh
> -# SPDX-License-Identifier: GPL2.0
> +# SPDX-License-Identifier: GPL-2.0
>  # description: Test wakeup tracer
>  
>  if ! which chrt ; then
> --- a/tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc
> +++ b/tools/testing/selftests/ftrace/test.d/tracer/wakeup_rt.tc
> @@ -1,5 +1,5 @@
>  #!/bin/sh
> -# SPDX-License-Identifier: GPL2.0
> +# SPDX-License-Identifier: GPL-2.0
>  # description: Test wakeup RT tracer
>  
>  if ! which chrt ; then
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] selftests/ftrace: Fix invalid SPDX identifiers
  2018-11-29 21:20 ` Thomas Gleixner
@ 2018-11-29 21:54   ` shuah
  2018-11-29 22:10     ` Masami Hiramatsu
  0 siblings, 1 reply; 4+ messages in thread
From: shuah @ 2018-11-29 21:54 UTC (permalink / raw)
  To: Thomas Gleixner, LKML; +Cc: Masami Hiramatsu, shuah

On 11/29/18 2:20 PM, Thomas Gleixner wrote:
> On Mon, 12 Nov 2018, Thomas Gleixner wrote:
> 
> Polite reminder....

Thanks for the reminder.

> 
>> While GPL2.0 looks about right, the correct and valid identifiers for GPL v2
>> only code are 'GPL-2.0' or 'GPL-2.0-only'.
>>
>> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
>> Cc: Masami Hiramatsu <mhiramat@kernel.org>
>> Cc: Shuah Khan (Samsung OSG) <shuah@kernel.org>
>>

Hi Masami,

I am pulling this in for 4.21-rc1.

thanks,
-- Shuah


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] selftests/ftrace: Fix invalid SPDX identifiers
  2018-11-29 21:54   ` shuah
@ 2018-11-29 22:10     ` Masami Hiramatsu
  0 siblings, 0 replies; 4+ messages in thread
From: Masami Hiramatsu @ 2018-11-29 22:10 UTC (permalink / raw)
  To: shuah; +Cc: Thomas Gleixner, LKML, Masami Hiramatsu

On Thu, 29 Nov 2018 14:54:16 -0700
shuah <shuah@kernel.org> wrote:

> On 11/29/18 2:20 PM, Thomas Gleixner wrote:
> > On Mon, 12 Nov 2018, Thomas Gleixner wrote:
> > 
> > Polite reminder....
> 
> Thanks for the reminder.
> 
> > 
> >> While GPL2.0 looks about right, the correct and valid identifiers for GPL v2
> >> only code are 'GPL-2.0' or 'GPL-2.0-only'.
> >>
> >> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> >> Cc: Masami Hiramatsu <mhiramat@kernel.org>
> >> Cc: Shuah Khan (Samsung OSG) <shuah@kernel.org>
> >>
> 
> Hi Masami,
> 
> I am pulling this in for 4.21-rc1.

Oops, I missed it. 

Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thank you!

-- 
Masami Hiramatsu <mhiramat@kernel.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-11-29 22:10 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-12 19:11 [PATCH] selftests/ftrace: Fix invalid SPDX identifiers Thomas Gleixner
2018-11-29 21:20 ` Thomas Gleixner
2018-11-29 21:54   ` shuah
2018-11-29 22:10     ` Masami Hiramatsu

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).