Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/23713 )
Change subject: driver/uart: Reintroduce default_baudrate as weak function. ......................................................................
Patch Set 1:
Hello, you were involved in the commit I partially revert, so I'm adding you.
Also I could do it by adding an config option to not reintroduce the function by default.
Best Regards
The rationale behind removing this was that nobody seems to use it, that it is confusing to have both a cmos setting and a Kconfig to do the same thing and that when fetching the baud rate from cmos it actually tried to print stuff before baud rate was set which could result in very long delays in the boot time.
To avoid having a something override the value defined in Kconfig (to avoid confusion) I think it would be better to not have it show this option in Kconfig at all. something like MAINBOARD_SET_KCONFIG which is mutually exlusive with TTYS0_BAUD...