[U-Boot] [PATCH] Revert "serial: ns16550: fix debug uart putc called before init"
Kever Yang
kever.yang at rock-chips.com
Tue Dec 4 02:49:52 UTC 2018
Hi Simon,
On 12/04/2018 04:55 AM, Simon Goldschmidt wrote:
> This reverts commit 6f57c34473d37b8da5e6a3764d0d377d748aeef1 since it
> does not seem to work at least on rk3399.
>
> The Rockchip Technical Reference Manual (TRM) for the rk3399 says the baud
> rate prescaler register is readable only when USR[0] is zero. Since this
> bit is defined as "reserved" in the socfpga cylcone5 TRM, let's rather
> drop this than making the ns16550 debug uart more platform specific.
>
> Reported-by: Roosen Henri <Henri.Roosen at ginzinger.com>
> Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>
Thanks,
- Kever
> ---
>
> drivers/serial/ns16550.c | 18 ++----------------
> 1 file changed, 2 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/serial/ns16550.c b/drivers/serial/ns16550.c
> index f21c240e64..25b9d17243 100644
> --- a/drivers/serial/ns16550.c
> +++ b/drivers/serial/ns16550.c
> @@ -272,26 +272,12 @@ static inline void _debug_uart_init(void)
> serial_dout(&com_port->lcr, UART_LCRVAL);
> }
>
> -static inline int NS16550_read_baud_divisor(struct NS16550 *com_port)
> -{
> - int ret;
> -
> - serial_dout(&com_port->lcr, UART_LCR_BKSE | UART_LCRVAL);
> - ret = serial_din(&com_port->dll) & 0xff;
> - ret |= (serial_din(&com_port->dlm) & 0xff) << 8;
> - serial_dout(&com_port->lcr, UART_LCRVAL);
> -
> - return ret;
> -}
> -
> static inline void _debug_uart_putc(int ch)
> {
> struct NS16550 *com_port = (struct NS16550 *)CONFIG_DEBUG_UART_BASE;
>
> - while (!(serial_din(&com_port->lsr) & UART_LSR_THRE)) {
> - if (!NS16550_read_baud_divisor(com_port))
> - return;
> - }
> + while (!(serial_din(&com_port->lsr) & UART_LSR_THRE))
> + ;
> serial_dout(&com_port->thr, ch);
> }
>
More information about the U-Boot
mailing list