Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40133 )
Change subject: soc/nvidia: Use SPDX for GPL-2.0-only files ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra124/spi... File src/soc/nvidia/tegra124/spi.c:
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra124/spi... PS2, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */
add a newline between those two?
I placed it as if it were the "this file is part of coreboot yadda yadda" line in the vast majority of files
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra210/jdi... File src/soc/nvidia/tegra210/jdi_25x18_display/panel-jdi-lpm102a188a.h:
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra210/jdi... PS2, Line 2: /* This file is part of the coreboot project. */
add a newline after this?
Done
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra210/spi... File src/soc/nvidia/tegra210/spi.c:
https://review.coreboot.org/c/coreboot/+/40133/2/src/soc/nvidia/tegra210/spi... PS2, Line 1: /* SPDX-License-Identifier: GPL-2.0-only */
I placed it as if it were the "this file is part of coreboot yadda yadda" line in the vast majority […]
Ack