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

[Nico Huber (Code Review)]

Nico Huber 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
the `select` you remove that option. So overriding the default
would be much preferred.

NB. I wonder why it's disabled by default. Maybe some
platforms don't want it for performance reasons. IMHO, we
should override the default for these and have a global
default `y` instead.