Angel Pons 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:
(3 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
It was not there in the original, so I chose to not add it.
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
It was never there
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
It was never there