[M] Change in coreboot[main]: soc/intel/mtl: Display Sign-of-Life message using FSP-M

[Paul Menzel (Code Review)]

Attention is currently required from: Anil Kumar K, Dinesh Gehlot, Eran Mitrani, Jérémy Compostella, Kapil Porwal, Martin L Roth, Martin Roth, Shelley Chen, Subrata Banik, Tarun.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78244?usp=email )

Change subject: soc/intel/mtl: Display Sign-of-Life message using FSP-M
......................................................................

Patch Set 23:

(6 comments)

File src/soc/intel/meteorlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/78244/comment/54714ab9_bec00689 :
PS23, Line 463: 	depends on MAINBOARD_HAS_CHROMEOS
Why this dependency?

https://review.coreboot.org/c/coreboot/+/78244/comment/4bced432_e343a582 :
PS23, Line 464: 	select VBT_CBFS_COMPRESSION_DEFAULT_LZ4
Why?

https://review.coreboot.org/c/coreboot/+/78244/comment/7fd59793_b3223bad :
PS23, Line 466: which display
1.  which display*s*
2.  … to display …

https://review.coreboot.org/c/coreboot/+/78244/comment/ab8eb02b_1ca10f03 :
PS23, Line 467: Memory
memory

https://review.coreboot.org/c/coreboot/+/78244/comment/cfa935da_60c29a07 :
PS23, Line 467: configurable text message
Where can this be configured?

https://review.coreboot.org/c/coreboot/+/78244/comment/8641fbd7_8f52f49d :
PS23, Line 468: 	  and CSME update.
It’d be nice, if you added as much information how the FSP-M graphics init works either here in the help text or the commit message.