[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/apollolake/Kconfig: Add CONSOLE_UART_BASE_ADDRESS

[Nico Huber (Code Review)]

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

Change subject: soc/intel/apollolake/Kconfig: Add CONSOLE_UART_BASE_ADDRESS
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33065/1/src/soc/intel/apollolake/Kconfig 
File src/soc/intel/apollolake/Kconfig:

https://review.coreboot.org/#/c/33065/1/src/soc/intel/apollolake/Kconfig@369 
PS1, Line 369: depends on DRIVERS_UART_8250MEM_32
             : 	depends on CONSOLE_SERIAL
Are the addresses specific to the SoC internal UARTS?
Then I'd suggest `depends on INTEL_LPSS_UART_FOR_CONSOLE`
instead.