On Thu, Dec 24, 2015 at 09:49:48AM -0600, Timur Tabi wrote:
> The REG_x macros are indices into a table, not register offsets. Since
> earlycon does not have access to the vendor data, we can currently only
> support standard ARM PL011 devices.
>
> Signed-off-by: Timur Tabi <timur@xxxxxxxxxxxxxx>
> ---
> drivers/tty/serial/amba-pl011.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/tty/serial/amba-pl011.c b/drivers/tty/serial/amba-pl011.c
> index f6ad383..06f827a 100644
> --- a/drivers/tty/serial/amba-pl011.c
> +++ b/drivers/tty/serial/amba-pl011.c
> @@ -2302,10 +2302,10 @@ static struct console amba_console = {
>
> static void pl011_putc(struct uart_port *port, int c)
> {
> - while (readl(port->membase + REG_FR) & UART01x_FR_TXFF)
> + while (readl(port->membase + UART01x_FR) & UART01x_FR_TXFF)
> ;
> - writeb(c, port->membase + REG_DR);
> - while (readl(port->membase + REG_FR) & UART01x_FR_BUSY)
> + writeb(c, port->membase + UART01x_DR);
> + while (readl(port->membase + UART01x_FR) & UART011_FR_BUSY)
I changed the UART011_FR_BUSY with UART01x_FR_BUSY, the console is okay
now.
So after you change the UART011_FR_BUSY to UART01x_FR_BUSY in the next
version, please add my test-by :
Tested-by: Huang Shijie <shijie.huang@xxxxxxx>
thanks
Huang Shijie
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
