build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32527 )
Change subject: soc/intel/cannonlake: Dump ME f/w version and status information. ......................................................................
Patch Set 2:
(6 comments)
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c File src/soc/intel/cannonlake/me.c:
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@259 PS2, Line 259: printk(BIOS_DEBUG, "ME: HFSTS1 : 0x%08X\n", hfsts1.raw); line over 80 characters
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@260 PS2, Line 260: printk(BIOS_DEBUG, "ME: HFSTS2 : 0x%08X\n", hfsts2.raw); line over 80 characters
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@261 PS2, Line 261: printk(BIOS_DEBUG, "ME: HFSTS3 : 0x%08X\n", hfsts3.raw); line over 80 characters
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@262 PS2, Line 262: printk(BIOS_DEBUG, "ME: HFSTS4 : 0x%08X\n", hfsts4.raw); line over 80 characters
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@263 PS2, Line 263: printk(BIOS_DEBUG, "ME: HFSTS5 : 0x%08X\n", hfsts5.raw); line over 80 characters
https://review.coreboot.org/#/c/32527/2/src/soc/intel/cannonlake/me.c@264 PS2, Line 264: printk(BIOS_DEBUG, "ME: HFSTS6 : 0x%08X\n", hfsts6.raw); line over 80 characters