DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag
@ 2019-09-03  7:24 Hemant Agrawal
  2019-09-03 12:15 ` Neil Horman
  2019-12-31  1:05 ` Stephen Hemminger
  0 siblings, 2 replies; 3+ messages in thread
From: Hemant Agrawal @ 2019-09-03  7:24 UTC (permalink / raw)
  To: nhorman, dev, techboard; +Cc: Hemant Agrawal

This patch adds SPDX license tag to pmdinfogen files.

These files are originally drived from kernel.
They are being used as binary tool to support internal
build.

This patch requires license exception approval from
DPDK Technical Board and Governing Board.

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 buildtools/pmdinfogen/pmdinfogen.c | 7 ++-----
 buildtools/pmdinfogen/pmdinfogen.h | 8 ++------
 license/exceptions.txt             | 4 +++-
 3 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/buildtools/pmdinfogen/pmdinfogen.c b/buildtools/pmdinfogen/pmdinfogen.c
index dc0b6d5ff..a68d1ea99 100644
--- a/buildtools/pmdinfogen/pmdinfogen.c
+++ b/buildtools/pmdinfogen/pmdinfogen.c
@@ -1,11 +1,8 @@
-/* Postprocess pmd object files to export hw support
+/* SPDX-License-Identifier: GPL-2.0
+ * Postprocess pmd object files to export hw support
  *
  * Copyright 2016 Neil Horman <nhorman@tuxdriver.com>
  * Based in part on modpost.c from the linux kernel
- *
- * This software may be used and distributed according to the terms
- * of the GNU General Public License V2, incorporated herein by reference.
- *
  */
 
 #include <stdio.h>
diff --git a/buildtools/pmdinfogen/pmdinfogen.h b/buildtools/pmdinfogen/pmdinfogen.h
index c8a9e2136..e6ba54535 100644
--- a/buildtools/pmdinfogen/pmdinfogen.h
+++ b/buildtools/pmdinfogen/pmdinfogen.h
@@ -1,12 +1,8 @@
-
-/* Postprocess pmd object files to export hw support
+/* SPDX-License-Identifier: GPL-2.0
+ * Postprocess pmd object files to export hw support
  *
  * Copyright 2016 Neil Horman <nhorman@tuxdriver.com>
  * Based in part on modpost.c from the linux kernel
- *
- * This software may be used and distributed according to the terms
- * of the GNU General Public License V2, incorporated herein by reference.
- *
  */
 
 #include <stdio.h>
diff --git a/license/exceptions.txt b/license/exceptions.txt
index f1cfa462f..9443dfc63 100644
--- a/license/exceptions.txt
+++ b/license/exceptions.txt
@@ -12,7 +12,9 @@ Note that following licenses are not exceptions:-
 -----------------------------------------------------------------
 SPDX Identifier     TB Approval Date  GB Approval Date  File name
 -----------------------------------------------------------------
-1.
+1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.c
+1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.h
+
 
 -----------------------------------------------------------------
 
-- 
2.17.1


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

* Re: [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag
  2019-09-03  7:24 [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag Hemant Agrawal
@ 2019-09-03 12:15 ` Neil Horman
  2019-12-31  1:05 ` Stephen Hemminger
  1 sibling, 0 replies; 3+ messages in thread
From: Neil Horman @ 2019-09-03 12:15 UTC (permalink / raw)
  To: Hemant Agrawal; +Cc: dev, techboard

On Tue, Sep 03, 2019 at 12:54:35PM +0530, Hemant Agrawal wrote:
> This patch adds SPDX license tag to pmdinfogen files.
> 
> These files are originally drived from kernel.
> They are being used as binary tool to support internal
> build.
> 
> This patch requires license exception approval from
> DPDK Technical Board and Governing Board.
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
>  buildtools/pmdinfogen/pmdinfogen.c | 7 ++-----
>  buildtools/pmdinfogen/pmdinfogen.h | 8 ++------
>  license/exceptions.txt             | 4 +++-
>  3 files changed, 7 insertions(+), 12 deletions(-)
> 
> diff --git a/buildtools/pmdinfogen/pmdinfogen.c b/buildtools/pmdinfogen/pmdinfogen.c
> index dc0b6d5ff..a68d1ea99 100644
> --- a/buildtools/pmdinfogen/pmdinfogen.c
> +++ b/buildtools/pmdinfogen/pmdinfogen.c
> @@ -1,11 +1,8 @@
> -/* Postprocess pmd object files to export hw support
> +/* SPDX-License-Identifier: GPL-2.0
> + * Postprocess pmd object files to export hw support
>   *
>   * Copyright 2016 Neil Horman <nhorman@tuxdriver.com>
>   * Based in part on modpost.c from the linux kernel
> - *
> - * This software may be used and distributed according to the terms
> - * of the GNU General Public License V2, incorporated herein by reference.
> - *
>   */
>  
>  #include <stdio.h>
> diff --git a/buildtools/pmdinfogen/pmdinfogen.h b/buildtools/pmdinfogen/pmdinfogen.h
> index c8a9e2136..e6ba54535 100644
> --- a/buildtools/pmdinfogen/pmdinfogen.h
> +++ b/buildtools/pmdinfogen/pmdinfogen.h
> @@ -1,12 +1,8 @@
> -
> -/* Postprocess pmd object files to export hw support
> +/* SPDX-License-Identifier: GPL-2.0
> + * Postprocess pmd object files to export hw support
>   *
>   * Copyright 2016 Neil Horman <nhorman@tuxdriver.com>
>   * Based in part on modpost.c from the linux kernel
> - *
> - * This software may be used and distributed according to the terms
> - * of the GNU General Public License V2, incorporated herein by reference.
> - *
>   */
>  
>  #include <stdio.h>
> diff --git a/license/exceptions.txt b/license/exceptions.txt
> index f1cfa462f..9443dfc63 100644
> --- a/license/exceptions.txt
> +++ b/license/exceptions.txt
> @@ -12,7 +12,9 @@ Note that following licenses are not exceptions:-
>  -----------------------------------------------------------------
>  SPDX Identifier     TB Approval Date  GB Approval Date  File name
>  -----------------------------------------------------------------
> -1.
> +1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.c
> +1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.h
> +
>  
>  -----------------------------------------------------------------
>  
> -- 
> 2.17.1
> 
> 
Acked-by: Neil Horman <nhorman@tuxdriver.com>

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

* Re: [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag
  2019-09-03  7:24 [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag Hemant Agrawal
  2019-09-03 12:15 ` Neil Horman
@ 2019-12-31  1:05 ` Stephen Hemminger
  1 sibling, 0 replies; 3+ messages in thread
From: Stephen Hemminger @ 2019-12-31  1:05 UTC (permalink / raw)
  To: Hemant Agrawal; +Cc: nhorman, dev, techboard

On Tue,  3 Sep 2019 12:54:35 +0530
Hemant Agrawal <hemant.agrawal@nxp.com> wrote:

> diff --git a/license/exceptions.txt b/license/exceptions.txt
> index f1cfa462f..9443dfc63 100644
> --- a/license/exceptions.txt
> +++ b/license/exceptions.txt
> @@ -12,7 +12,9 @@ Note that following licenses are not exceptions:-
>  -----------------------------------------------------------------
>  SPDX Identifier     TB Approval Date  GB Approval Date  File name
>  -----------------------------------------------------------------
> -1.
> +1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.c
> +1.GPL-2.0	TBD	 TBD		buildtools/pmdinfogen/pmdinfogen.h

This should not be part of this patch, it needs to be a separate patch with
real approval dates.

Also, the same license must apply to the whole directory. It makes no sense
to have a BSD licensed make setup with a GPL licensed source. Better
to just list whole directory.

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

end of thread, other threads:[~2019-12-31  1:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-03  7:24 [dpdk-dev] [PATCH] pmdinfogen: add SPDX license tag Hemant Agrawal
2019-09-03 12:15 ` Neil Horman
2019-12-31  1:05 ` Stephen Hemminger

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