Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40130 )
Change subject: soc/cavium: Use SPDX for GPL-2.0-only files ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/bootb... File src/soc/cavium/cn81xx/bootblock_custom.S:
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/bootb... PS2, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */ misses the this is part of coreboot line
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/cpu_s... File src/soc/cavium/cn81xx/cpu_secondary.S:
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/cpu_s... PS2, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */ same line also missing here
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/inclu... File src/soc/cavium/cn81xx/include/atf/plat_params.h:
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/inclu... PS2, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */ same line missing
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/soc.c File src/soc/cavium/cn81xx/soc.c:
https://review.coreboot.org/c/coreboot/+/40130/2/src/soc/cavium/cn81xx/soc.c... PS2, Line 3: * SPDX-License-Identifier: GPL-2.0-only * wrong order and also one instead of separate comments