Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40927 )
Change subject: soc/intel/xeon_sp/cpx: ACPI xSDT Table update ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/40927/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40927/2//COMMIT_MSG@7 PS2, Line 7: soc/intel/xeon_sp/cpx: ACPI xSDT Table update Please make it a statement by adding a verb (in imperative mood):
Update ACPI xSDT Table
https://review.coreboot.org/c/coreboot/+/40927/2/src/soc/intel/xeon_sp/cpx/a... File src/soc/intel/xeon_sp/cpx/acpi/globalnvs.asl:
https://review.coreboot.org/c/coreboot/+/40927/2/src/soc/intel/xeon_sp/cpx/a... PS2, Line 17: */ Please use SPDX headers.