Re: [PATCH 1/2] serial: a37xx: Use CONFIG_BAUDRATE for initializing early debug UART

[Stefan Roese]

On 26.07.21 14:58, Pali Rohár wrote:

CONFIG_BAUDRATE should be used for setting the baudrate for the early debug
UART. This replaces current hardcoded 115200 value.

Signed-off-by: Pali Rohár <p...@kernel.org>

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

---
  drivers/serial/serial_mvebu_a3700.c | 5 ++---
  1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/serial/serial_mvebu_a3700.c 
b/drivers/serial/serial_mvebu_a3700.c
index c7e66fef8768..52dc3fdad7b4 100644
--- a/drivers/serial/serial_mvebu_a3700.c
+++ b/drivers/serial/serial_mvebu_a3700.c
@@ -309,7 +309,7 @@ U_BOOT_DRIVER(serial_mvebu) = {
  static inline void _debug_uart_init(void)
  {
        void __iomem *base = (void __iomem *)CONFIG_DEBUG_UART_BASE;
-       u32 baudrate, parent_rate, divider;
+       u32 parent_rate, divider;

/* reset FIFOs */

        writel(UART_CTRL_RXFIFO_RESET | UART_CTRL_TXFIFO_RESET,
@@ -322,9 +322,8 @@ static inline void _debug_uart_init(void)
         * Calculate divider
         * baudrate = clock / 16 / divider
         */
-       baudrate = 115200;
        parent_rate = get_ref_clk() * 1000000;
-       divider = DIV_ROUND_CLOSEST(parent_rate, baudrate * 16);
+       divider = DIV_ROUND_CLOSEST(parent_rate, CONFIG_BAUDRATE * 16);
        writel(divider, base + UART_BAUD_REG);

/*

Viele Grüße,
Stefan

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de