Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31419 )
Change subject: src/mainboard/pcengines/apu2/Kconfig: increase pre-CBMEM console size ......................................................................
Patch Set 1: Code-Review-1
(2 comments)
https://review.coreboot.org/#/c/31419/1/src/mainboard/pcengines/apu2/Kconfig File src/mainboard/pcengines/apu2/Kconfig:
https://review.coreboot.org/#/c/31419/1/src/mainboard/pcengines/apu2/Kconfig... PS1, Line 120: Increase this value if preram cbmem console is getting truncated We don't have good upper limit for this value. Therefore we/I are generally refusing to increase the default even for individual boards. Reasoning is this; if you have problems in pre-ram, it's not so likely that you even reach payload or OS to actually read CBMEM console at all. So the debugging would be done on serial console.
https://review.coreboot.org/#/c/31419/1/src/mainboard/pcengines/apu2/Kconfig... PS1, Line 126: When selected only the BSP CPU will output to early console. Probably all other console types than CBMEM console and serial will break with SQUELCH_EARLY_SMP=n. And with AGESA/binaryPI it actually (currently) makes log almos uncompherensible because lines are not tagged with APIC number.
I have some work coming up on review on this part...