Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30933 )
Change subject: siemens/mc_apl4: Change UART_FOR_CONSOLE index ......................................................................
Patch Set 2: Code-Review+2
Patch Set 2:
Patch Set 2: Code-Review+2
Well, I said "in theory" ;) did you test it?
Yes, I've checked the .config file for mc_apl4 and there's an 1 set for CONFIG_UART_FOR_CONSOLE. So that's how it's gonna work.
Yes, it wokrs because the Kconfig files of the variants are sourced first and if a Kconfig symbol has been declared there, it cannot be re-declared here again. Just in the case where this symbol is missing in the variants Kconfig the one in the "top-level" Kconfig will be used. So that should pretty well serve our demand here.