Stefan Reinauer wrote:
The UART divider should be calculated based on the base frequency and baudrate, not hardcoded in addition to that.
Signed-off-by: Stefan Reinauer stefan.reinauer@coreboot.org
With some changes pointed out below it's:
Acked-by: Peter Stuge peter@stuge.se
+++ src/mainboard/iwave/iWRainbowG6/Kconfig (working copy) @@ -37,9 +37,7 @@ int default 10
-# WTF, is this 19200? -config TTYS0_DIV
- int
- default 6
+# This mainboard might have a higher clocked UART or might not be able to run +# serial output at 115200 baud
I don't think this board has a special UART, I think the setting was just chosen by the original developer as a way to make coreboot configure the desired baudrate. So just drop the setting and let the regular baudrate mechanism in Kconfig do the job.
+++ src/console/uart8250_console.c (working copy) @@ -25,7 +25,13 @@ { static const unsigned char div[8] = { 1, 2, 3, 6, 12, 24, 48, 96 }; int b_index = 0;
- unsigned int divisor = CONFIG_TTYS0_DIV;
- /* TODO the divisor calculation is hard coded to standard UARTs. Some
* UARTs won't work with these values. This should be a property of the* UART used, worst case a Kconfig variable. For now live with hard* codes as the only devices that are different are the iWave* iRainbowG6 and the OXPCIe952 card (and the latter is memory mapped)
Also change this wording since it's not really certain that iRainbowG6 actually is different. (Btw it would be nice to make the mainboard directory name all lowercase, like with every other board.)
+++ src/lib/uart8250.c (working copy) @@ -88,6 +88,12 @@ #if defined(__PRE_RAM__) || defined(__SMM__) void uart_init(void) {
- /* TODO the divisor calculation is hard coded to standard UARTs. Some
* UARTs won't work with these values. This should be a property of the* UART used, worst case a Kconfig variable. For now live with hard* codes as the only devices that are different are the iWave* iRainbowG6 and the OXPCIe952 card (and the latter is memory mapped)
Same with this comment.
//Peter