Attention is currently required from: Paul Menzel, Stefan Reinauer. Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54385 )
Change subject: Documentation: Add proposal to allow enabling serial console with a flag ......................................................................
Patch Set 3:
(1 comment)
File Documentation/technotes/2021-05-selectable-serial-console.md:
https://review.coreboot.org/c/coreboot/+/54385/comment/b48f7400_c124055b PS1, Line 112: and change the default to whatever is found there.
I updated the proposal to use a cbfs file (created with cbfstool add-int).
SGTM in general, but you removed the part about forgoing the bootblock now? That part is still important -- remember that non-x86 platforms can't access flash until a lot of other initialization in the bootblock has happened. (Also -- this is just something to keep in mind for the implementation -- you'll need some safeguards to avoid recursion when the CBFS code tries to print things.)