Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43817 )
Change subject: mb/emulation/qemu-i440fx: Remove TRACE=y from test build ......................................................................
Patch Set 2:
Patch Set 2: Code-Review-1
(1 comment)
The `TRACE=y` is there so the option gets build tested. If removed for this board, it should at least be set for another. But generally, we build test so things can get fixed and not worked around. Maybe somebody will want to try it with QEMU? and then it doesn't build... that's what we actually want to avoid.
Ever since RELOCATABLE_RAMSTAGE, TRACE=y has not made much sense, the pointers returned to console are not normalized. Looks like 8 years of nobody really caring about this.
Let's see how many boards actually build with it enabled.