Attention is currently required from: Nico Huber, Julius Werner. Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56210 )
Change subject: console: Handle verstage as "first non-bootblock stage" ......................................................................
Patch Set 1:
(1 comment)
File src/console/init.c:
https://review.coreboot.org/c/coreboot/+/56210/comment/b1ba9869_fad730a3 PS1, Line 15: #endif
For one, some stages are _pretty_ packed, e.g. verstage on google/veyron. […]
Given the back and forth here I'm thinking of making all this explicit, even though I dislike options proliferation: Kconfig options "NO_(BOOTBLOCK|VERSTAGE|ROMSTAGE|RAMSTAGE)_CONSOLE(_OVERRIDE|)" to disable everything per stage or to disable the overrides based on external factors.
As an example, rk3288 would enable NO_VERSTAGE_CONSOLE_OVERRIDE by default for space reasons, arch/arm* could default to NO_BOOTBLOCK_CONSOLE_OVERRIDE because we can't trust CBFS to be flexible enough around there, and users could vary per-board or per-build as desired.