build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35763 )
Change subject: soc/intel/skylake: rework and fix platform detection ......................................................................
Patch Set 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35763/6/src/soc/intel/skylake/chip_... File src/soc/intel/skylake/chip_fsp20.c:
https://review.coreboot.org/c/coreboot/+/35763/6/src/soc/intel/skylake/chip_... PS6, Line 99: .name = (const char*) "SKL/KBL Sunrise/Union Point PCH-LP SKU", "(foo*)" should be "(foo *)"
https://review.coreboot.org/c/coreboot/+/35763/6/src/soc/intel/skylake/chip_... PS6, Line 105: .name = (const char*) "SKL/KBL Sunrise Point PCH-H SKU", "(foo*)" should be "(foo *)"
https://review.coreboot.org/c/coreboot/+/35763/6/src/soc/intel/skylake/chip_... PS6, Line 111: .name = (const char*) "SKL/KBL Union Point PCH-H SKU", "(foo*)" should be "(foo *)"