]> git.sur5r.net Git - u-boot/commitdiff
serial/serial_arc: switch from {read|write}l to {read|write}b accessors
authorAlexey Brodkin <Alexey.Brodkin@synopsys.com>
Sat, 8 Feb 2014 06:10:02 +0000 (10:10 +0400)
committerTom Rini <trini@ti.com>
Fri, 21 Feb 2014 12:54:05 +0000 (07:54 -0500)
This is required for proper functionality on big-endian targets.
Memory-mapped registres of ARC UART are not 32-bit words but 8-bit bytes
so on little-endian target either acessor (_l or _b) works fine.
On big-endian only _b accessors works as expected.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Vineet Gupta <vgupta@synopsys.com>
Cc: Noam Camus <noamc@ezchip.com>
Cc: Tom Rini <trini@ti.com>
drivers/serial/serial_arc.c

index 55d07691fcf227a65eebfee94f9710b096c57c6b..b21b12ba44238cc23a519063c32e51b2198c173d 100644 (file)
@@ -38,7 +38,7 @@ static void arc_serial_setbrg(void)
                gd->baudrate = CONFIG_BAUDRATE;
 
        arc_console_baud = gd->cpu_clk / (gd->baudrate * 4) - 1;
-       writel(arc_console_baud & 0xff, &regs->baudl);
+       writeb(arc_console_baud & 0xff, &regs->baudl);
 
 #ifdef CONFIG_ARC
        /*
@@ -50,11 +50,11 @@ static void arc_serial_setbrg(void)
         * Until that is fixed, when running on ISS, we will set baudh to !0
         */
        if (gd->arch.running_on_hw)
-               writel((arc_console_baud & 0xff00) >> 8, &regs->baudh);
+               writeb((arc_console_baud & 0xff00) >> 8, &regs->baudh);
        else
-               writel(1, &regs->baudh);
+               writeb(1, &regs->baudh);
 #else
-       writel((arc_console_baud & 0xff00) >> 8, &regs->baudh);
+       writeb((arc_console_baud & 0xff00) >> 8, &regs->baudh);
 #endif
 }
 
@@ -70,15 +70,15 @@ static void arc_serial_putc(const char c)
        if (c == '\n')
                arc_serial_putc('\r');
 
-       while (!(readl(&regs->status) & UART_TXEMPTY))
+       while (!(readb(&regs->status) & UART_TXEMPTY))
                ;
 
-       writel(c, &regs->data);
+       writeb(c, &regs->data);
 }
 
 static int arc_serial_tstc(void)
 {
-       return !(readl(&regs->status) & UART_RXEMPTY);
+       return !(readb(&regs->status) & UART_RXEMPTY);
 }
 
 static int arc_serial_getc(void)
@@ -87,10 +87,10 @@ static int arc_serial_getc(void)
                ;
 
        /* Check for overflow errors */
-       if (readl(&regs->status) & UART_OVERFLOW_ERR)
+       if (readb(&regs->status) & UART_OVERFLOW_ERR)
                return 0;
 
-       return readl(&regs->data) & 0xFF;
+       return readb(&regs->data) & 0xFF;
 }
 
 static void arc_serial_puts(const char *s)