From 6d643cf722872dfe25a3fb6f8b149642b93a1adc Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Wed, 21 Jun 2017 15:38:51 -0700 Subject: [PATCH] rockchip: Correct UART reference clock value The Rockchip UARTs are tied directly to the 24MHz oscillator and are thus clocked with exactly 24MHz. The reasons why our code instead uses some 23.xxMHz value have long been lost in time. For the current shared 8250 implementation, the baud rate divisor for 115200 would be the same. Correcting this does make the information in the coreboot table more accurate and help payloads chose a better divisor, though. Change-Id: Ieceb07760178f8ddbb5936f8742b78f8def4072d Signed-off-by: Julius Werner Reviewed-on: https://review.coreboot.org/23556 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Patrick Georgi Reviewed-by: Paul Menzel --- src/soc/rockchip/common/uart.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/soc/rockchip/common/uart.c b/src/soc/rockchip/common/uart.c index d2115da5e8..9ae6e1e356 100644 --- a/src/soc/rockchip/common/uart.c +++ b/src/soc/rockchip/common/uart.c @@ -14,11 +14,12 @@ */ #include +#include #include unsigned int uart_platform_refclk(void) { - return 23040000; + return OSC_HZ; } uintptr_t uart_platform_base(int idx)