[coreboot-gerrit] Change in ...coreboot[master]: soc/intel: Replace UART_BASE() and friends with a Kconfig

[Nico Huber (Code Review)]

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

Change subject: soc/intel: Replace UART_BASE() and friends with a Kconfig
......................................................................


Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33098/1/src/soc/intel/cannonlake/Kconfig 
File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/#/c/33098/1/src/soc/intel/cannonlake/Kconfig@228 
PS1, Line 228: 	default 0xfe032000
> increment with CONFIG_UART_FOR_CONSOLE? otherwise only UART0 would be supported.
> AIUI, our whole infrastructure only supports a single console UART.
> Any UART_FOR_CONSOLE can use this address. I don't see why we'd need
> different addresses for different UARTs.