Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31491 )
Change subject: [WIP] console: Refactor printk() varargs prototypes ......................................................................
Patch Set 1:
Patch Set 1: Code-Review+2
LGTM conceptually. Unfortunately, it seems that GCC is (as usual) stupid and cannot inline vararg functions no matter what (I even tried with always_inline and it just flat out refuses), so this leads no slightly suboptimal code generation. No big deal I think, but you'll need to fix up those memlayouts again (for Veyron maybe it's time to just drop the pre-RAM console completely...).
I did a fresh install of arm crossgcc from commit da7ffb4 and abuild passed. I did not feel like digging deeper into this and did not have older arm toolchain around to compare.
Being lazy, I think I just wait until buildbots have had their toolchains updated and rebase then.