[coreboot-gerrit] Change in coreboot[master]: drivers/uart: Override uart base address

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45405 )

Change subject: drivers/uart: Override uart base address
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45405/2/src/drivers/uart/uart8250io... 
File src/drivers/uart/uart8250io.c:

https://review.coreboot.org/c/coreboot/+/45405/2/src/drivers/uart/uart8250io... 
PS2, Line 78: static const unsigned int bases[] = { 0x3f8, 0x2f8, 0x3e8, 0x2e8 };
> What address would you use that isn't listed? I think that bases[] is the only valid settings. […]

Marc, For DeltaLake, we are in the process of polishing the firmware image for it to be used by other teams. The image needs to be user (non-coreboot engineers) friendly, but also debug-able when problem arises. Therefore, we need console UART (as well as log level) to be able to be changed without having to rebuild an image.