Re: [PATCH v3] powerpc: Add missing SPDX license identifiers
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2022-01-21 14:35:56
Also in:
lkml
On Fri, Jan 21, 2022 at 11:03:20AM +0000, Christophe Leroy wrote:
Several files are missing SPDX license identifiers. Following files are given the following SPDX identifier based on the comments in the top of the file: boot/crtsavres.S:/* SPDX-License-Identifier: GPL-2.0+ */
Are you sure that this is the correct license for this file? Also you dropped the arch/powerpc/ prefix here...
include/asm/epapr_hcalls.h:/* SPDX-License-Identifier: GPL-2.0+ OR BSD */ include/asm/fsl_hcalls.h:/* SPDX-License-Identifier: GPL-2.0+ OR BSD */ include/asm/ibmebus.h:/* SPDX-License-Identifier: GPL-2.0 OR OpenIB BSD */ include/asm/sfp-machine.h:/* SPDX-License-Identifier: LGPL-2.0+ */ kvm/mpic.c:// SPDX-License-Identifier: GPL-2.0 lib/crtsavres.S:/* SPDX-License-Identifier: GPL-2.0+ */ mm/book3s64/hash_4k.c:// SPDX-License-Identifier: LGPL-2.0 mm/book3s64/hash_64k.c:// SPDX-License-Identifier: LGPL-2.0 mm/book3s64/hash_hugepage.c:// SPDX-License-Identifier: LGPL-2.1 platforms/pseries/ibmebus.c:// SPDX-License-Identifier: GPL-2.0 OR OpenIB BSD tools/head_check.sh:# SPDX-License-Identifier: GPL-2.0+ xmon/ppc.h:/* SPDX-License-Identifier: GPL-1.0+ */ Add to other files the default kernel license identifier, in extenso GPL-2.0. DTS files are handled in a separate commit. Signed-off-by: Christophe Leroy <redacted> Cc: Thomas Gleixner <redacted> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: linux-spdx@vger.kernel.org --- v3: Removed license text and license note in the files that have any. v2: Changed from GPL-2.0 to a licence consistant with the file's comments for the few files listed in the commit message. --- arch/powerpc/Makefile | 5 +-- arch/powerpc/boot/44x.h | 5 +-- arch/powerpc/boot/4xx.h | 5 +-- arch/powerpc/boot/crtsavres.S | 16 +--------- arch/powerpc/boot/dummy.c | 1 + arch/powerpc/boot/install.sh | 5 +-- arch/powerpc/boot/ops.h | 6 ++-- arch/powerpc/boot/serial.c | 6 ++-- arch/powerpc/boot/simple_alloc.c | 6 ++-- arch/powerpc/include/asm/8xx_immap.h | 1 + arch/powerpc/include/asm/asm-compat.h | 1 + arch/powerpc/include/asm/asm-const.h | 1 + arch/powerpc/include/asm/asm-offsets.h | 1 + arch/powerpc/include/asm/cpm.h | 1 + arch/powerpc/include/asm/dtl.h | 1 + arch/powerpc/include/asm/edac.h | 6 ++-- arch/powerpc/include/asm/ehv_pic.h | 5 +-- arch/powerpc/include/asm/emergency-restart.h | 1 + arch/powerpc/include/asm/epapr_hcalls.h | 32 +------------------ arch/powerpc/include/asm/fixmap.h | 5 +-- arch/powerpc/include/asm/floppy.h | 5 +-- arch/powerpc/include/asm/fs_pd.h | 5 +-- arch/powerpc/include/asm/fsl_hcalls.h | 32 +------------------ arch/powerpc/include/asm/hydra.h | 5 +-- arch/powerpc/include/asm/ibmebus.h | 29 +---------------- arch/powerpc/include/asm/kgdb.h | 6 ++-- arch/powerpc/include/asm/membarrier.h | 1 + arch/powerpc/include/asm/module.lds.h | 1 + arch/powerpc/include/asm/mpc52xx.h | 5 +-- arch/powerpc/include/asm/mpc52xx_psc.h | 5 +-- arch/powerpc/include/asm/pmac_feature.h | 5 +-- arch/powerpc/include/asm/ppc_asm.h | 1 + arch/powerpc/include/asm/pte-walk.h | 1 + arch/powerpc/include/asm/rheap.h | 6 ++-- arch/powerpc/include/asm/sfp-machine.h | 16 +--------- arch/powerpc/include/asm/vmalloc.h | 1 + arch/powerpc/include/asm/word-at-a-time.h | 1 + arch/powerpc/kernel/interrupt_64.S | 1 + arch/powerpc/kernel/kgdb.c | 5 +-- arch/powerpc/kernel/ptrace/ptrace.c | 5 +-- arch/powerpc/kernel/ptrace/ptrace32.c | 5 +-- arch/powerpc/kernel/signal.c | 5 +-- arch/powerpc/kernel/signal.h | 5 +-- arch/powerpc/kernel/vdso32/note.S | 1 + arch/powerpc/kernel/vdso64/note.S | 1 + arch/powerpc/kvm/mpic.c | 19 +---------- arch/powerpc/lib/crtsavres.S | 16 +--------- arch/powerpc/lib/restart_table.c | 1 + arch/powerpc/lib/rheap.c | 6 ++-- arch/powerpc/mm/book3s64/hash_4k.c | 10 +----- arch/powerpc/mm/book3s64/hash_64k.c | 10 +----- arch/powerpc/mm/book3s64/hash_hugepage.c | 10 +----- arch/powerpc/mm/hugetlbpage.c | 1 + arch/powerpc/perf/req-gen/_end.h | 1 + arch/powerpc/platforms/44x/fsp2.h | 1 + arch/powerpc/platforms/4xx/pci.c | 1 + arch/powerpc/platforms/4xx/pci.h | 1 + arch/powerpc/platforms/52xx/efika.c | 5 +-- arch/powerpc/platforms/52xx/mpc52xx_common.c | 6 +--- arch/powerpc/platforms/52xx/mpc52xx_pci.c | 5 +-- arch/powerpc/platforms/52xx/mpc52xx_pic.c | 6 +--- arch/powerpc/platforms/85xx/ksi8560.c | 6 ++-- arch/powerpc/platforms/85xx/p1022_ds.c | 5 +-- arch/powerpc/platforms/85xx/p1022_rdk.c | 5 +-- arch/powerpc/platforms/8xx/ep88xc.c | 5 +-- arch/powerpc/platforms/8xx/mpc86xads.h | 5 ++- arch/powerpc/platforms/8xx/mpc86xads_setup.c | 5 +-- arch/powerpc/platforms/8xx/mpc885ads.h | 5 ++- arch/powerpc/platforms/8xx/mpc885ads_setup.c | 5 +-- arch/powerpc/platforms/8xx/mpc8xx.h | 5 +-- arch/powerpc/platforms/8xx/pic.c | 1 + arch/powerpc/platforms/8xx/pic.h | 1 + arch/powerpc/platforms/8xx/tqm8xx_setup.c | 5 +-- .../cell/spufs/spu_restore_dump.h_shipped | 1 + .../cell/spufs/spu_save_dump.h_shipped | 1 + arch/powerpc/platforms/chrp/gg2.h | 5 +-- .../platforms/embedded6xx/linkstation.c | 5 +-- arch/powerpc/platforms/embedded6xx/ls_uart.c | 5 +-- arch/powerpc/platforms/embedded6xx/mpc10x.h | 6 ++-- .../platforms/embedded6xx/storcenter.c | 5 +-- arch/powerpc/platforms/microwatt/Makefile | 1 + arch/powerpc/platforms/microwatt/setup.c | 1 + arch/powerpc/platforms/pseries/ibmebus.c | 29 +---------------- arch/powerpc/sysdev/cpm2.c | 5 +-- arch/powerpc/sysdev/cpm2_pic.c | 5 +-- arch/powerpc/sysdev/ehv_pic.c | 5 +-- arch/powerpc/sysdev/ge/ge_pic.c | 5 +-- arch/powerpc/sysdev/mpic.c | 5 +-- arch/powerpc/sysdev/rtc_cmos_setup.c | 6 ++-- arch/powerpc/tools/ci-build.sh | 1 + arch/powerpc/tools/head_check.sh | 6 +--- arch/powerpc/xmon/ppc.h | 15 ++------- 92 files changed, 105 insertions(+), 422 deletions(-)
You might want to change less of these at once, as this is hard to review as-is.
quoted hunk ↗ jump to hunk
diff --git a/arch/powerpc/boot/crtsavres.S b/arch/powerpc/boot/crtsavres.S index 085fb2b9a8b8..25e924459dcc 100644 --- a/arch/powerpc/boot/crtsavres.S +++ b/arch/powerpc/boot/crtsavres.S@@ -1,3 +1,4 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ /* * Special support for eabi and SVR4 *@@ -7,11 +8,6 @@ * * Based on gcc/config/rs6000/crtsavres.asm from gcc * - * This file 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, or (at your option) any - * later version. - * * In addition to the permissions in the GNU General Public License, the * Free Software Foundation gives you unlimited permission to link the * compiled version of this file with other programs, and to distribute@@ -20,16 +16,6 @@ * respects; for example, they cover modification of the file, and * distribution when not linked into another program.) * - * This file 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. - * - * You should have received a copy of the GNU General Public License - * along with this program; see the file COPYING. If not, write to - * the Free Software Foundation, 51 Franklin Street, Fifth Floor, - * Boston, MA 02110-1301, USA. - * * As a special exception, if you link this library with files * compiled with GCC to produce an executable, this does not cause * the resulting executable to be covered by the GNU General Public License.
Look at that "special exception", why are you ignoring it here? You can't do that :( Please do these in smaller batches, perhaps one patch per type of license and directory to make it easier to notice mistakes like this? thanks, greg k-h