[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Clean up legacy UART config

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40322 )

Change subject: soc/amd/picasso: Clean up legacy UART config
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/Kconfig 
File src/soc/amd/picasso/Kconfig:

https://review.coreboot.org/c/coreboot/+/40322/2/src/soc/amd/picasso/Kconfig... 
PS2, Line 169: uart
> UART_FOR_CONSOLE decides which i/o ports the console writes to (imho, […]

Sorry, I think this was all wrong. It doesn't have anything to do
with the console, does it?

I don't understand the use case. Is it: If you want one of the
Picasso UARTs in legacy mode for later software, enable serial
console output in coreboot, select the UART for the console,
set the expected i/o range here? That seems like a lot indirection.