[coreboot-gerrit] Change in coreboot[master]: soc/intel/alderlake: Enable FSP_DEBUG_EVENT_HANDLER_IN_COREBOOT Kconfig

[Arthur Heymans (Code Review)]

Attention is currently required from: Subrata Banik, Tim Wawrzynczak, Eric Lai.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63009 )

Change subject: soc/intel/alderlake: Enable FSP_DEBUG_EVENT_HANDLER_IN_COREBOOT Kconfig
......................................................................

Patch Set 2: Code-Review+2

(1 comment)

File src/soc/intel/alderlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/63009/comment/4719c5f3_3c879a94 
PS2, Line 43: 	select FSP_DEBUG_EVENT_HANDLER_IN_COREBOOT
> Can we depend CONFIG_CONSOLE_SERIAL or CONFIG_UART_DEBUG? I think the normal image, we don't need to know FSP log.

Isn't the whole point to be console agnostic and let coreboot do the console printing wherever it seems fit? Even without series I still see value in printing output to for instance the cbmem console.