[coreboot-gerrit] Change in ...coreboot[master]: soc/intel: Enable console in bootblock

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30886 )

Change subject: soc/intel: Enable console in bootblock
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/30886/1/src/soc/intel/cannonlake/Kconfig 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/#/c/30886/1/src/soc/intel/cannonlake/Kconfig@29 
PS1, Line 29: 	select BOOTBLOCK_CONSOLE
> BOOTBLOCK_CONSOLE has a prompt, thus it's a user option. With […]

If only this platform needs BOOTBLOCK_CONSOLE by default, I would leave it disabled by default. Grepping through the tree shows there are other mainboards/chipsets which select BOOTBLOCK_CONSOLE (select, not override the default). Maybe those who select BOOTBLOCK_CONSOLE have something interesting to say in the bootblock?