lkml.org 
[lkml]   [2019]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86/mm: Add SPDX identifier in arch/x86/mm files
On Wed, 30 Jan 2019, Liu Xiang wrote:
> --- a/arch/x86/mm/iomap_32.c
> +++ b/arch/x86/mm/iomap_32.c
> @@ -1,19 +1,7 @@
> +// SPDX-License-Identifier: GPL-2.0

This is wrong.

> /*
> * Copyright © 2008 Ingo Molnar
> *
> - * 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.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

And there are more of those wrongs.

Aside of that, you really need to differentiate between files, which do not
have any license notice or reference at all and those which have. For the
ones without adding the identifier is fine. For thos which have, you really
need to create separate patches and cc the copyright holder(s) and the
initial author if possible.

Just slapping random identifiers into source files is not helping anything,
actually it's making the problem worse because you are relicensing the
source files.

Thanks,

tglx
\
 
 \ /
  Last update: 2019-01-30 19:06    [W:0.024 / U:9.724 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site