[v3 0/2] Add SPDX license

[v3 0/2] Add SPDX license

[Jules Irenge]

 This patch series adds SPDX license identifier to fix missing and
 malformed warning reported  by checkpatch.pl tool 

 WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 
 
 The patch then removes unnecessary license boilerplates.
 Now that we have the spdx specified, the license text is not necessary.

v3:
 [PATCH 1/2]:Commit log corrected, commit message clearer
 
Jules Irenge (2):
  staging: mt7621-eth: Add SPDX license identifier
  staging: mt7621-eth: Remove license boilerplate text

 drivers/staging/mt7621-eth/ethtool.h | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

-- 
2.20.1

[v3 1/2] staging: mt7621-eth: Add SPDX license identifier

[Jules Irenge]

Add the SPDX GPL-2.0 license identifier to fix checkpatch.pl warning
Issue reported by checkpatch.pl warning:
"WARNING: Missing or malformed SPDX-License-Identifier tag in line 1"

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
---
 drivers/staging/mt7621-eth/ethtool.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/mt7621-eth/ethtool.h b/drivers/staging/mt7621-eth/ethtool.h
index 40b4cf011660..d0e97ae0d82e 100644
--- a/drivers/staging/mt7621-eth/ethtool.h
+++ b/drivers/staging/mt7621-eth/ethtool.h
@@ -1,3 +1,4 @@
+/* SPDX-License-Identifier: GPL-2.0 */
 /*   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; version 2 of the License
-- 
2.20.1

Re: [v3 1/2] staging: mt7621-eth: Add SPDX license identifier

[Greg KH]

On Fri, Mar 01, 2019 at 12:14:16AM +0000, Jules Irenge wrote:
> Add the SPDX GPL-2.0 license identifier to fix checkpatch.pl warning
> Issue reported by checkpatch.pl warning:
> "WARNING: Missing or malformed SPDX-License-Identifier tag in line 1"
> 
> Signed-off-by: Jules Irenge <jbi.octave@gmail.com>

This is fine, but where did the PATCH go in the subject line?

thanks,

greg k-h

[v3 2/2] staging: mt7621-eth: Remove license boilerplate text

[Jules Irenge]

Removes license boilerplate text

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
---
 drivers/staging/mt7621-eth/ethtool.h | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/staging/mt7621-eth/ethtool.h b/drivers/staging/mt7621-eth/ethtool.h
index d0e97ae0d82e..0071469aea6c 100644
--- a/drivers/staging/mt7621-eth/ethtool.h
+++ b/drivers/staging/mt7621-eth/ethtool.h
@@ -1,13 +1,5 @@
 /* SPDX-License-Identifier: GPL-2.0 */
-/*   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; version 2 of the License
- *
- *   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.
- *
+/*
  *   Copyright (C) 2009-2016 John Crispin <blogic@openwrt.org>
  *   Copyright (C) 2009-2016 Felix Fietkau <nbd@openwrt.org>
  *   Copyright (C) 2013-2016 Michael Lee <igvtee@gmail.com>
-- 
2.20.1

Re: [v3 0/2] Add SPDX license

[Greg KH]

On Fri, Mar 01, 2019 at 12:14:15AM +0000, Jules Irenge wrote:
>  This patch series adds SPDX license identifier to fix missing and
>  malformed warning reported  by checkpatch.pl tool 
> 
>  WARNING: Missing or malformed SPDX-License-Identifier tag in line 1 
>  
>  The patch then removes unnecessary license boilerplates.
>  Now that we have the spdx specified, the license text is not necessary.
> 
> v3:
>  [PATCH 1/2]:Commit log corrected, commit message clearer

Next time, the subject for this email should match the patch emails.  It
should be something like:
	Subject: [PATCH 0/2 v3] staging: mt7621-eth: SPDX license cleanups

thanks,

greg k-h