Anton Kochkov ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/1541

-gerrit

commit 83a039fde2ec1516d0b38971b1e9db1605f38745
Author: Anton Kochkov <[email protected]>
Date:   Wed Sep 26 22:31:04 2012 +0400

    libpayload: Set 8bits per char for serial port
    
    Previously we assume that hardware using 8 bits
    per char by default, but on Asrock A53 Pro
    this is not true (7 bit per char by default).
    Forcing use 8n1 now.
    
    Change-Id: Ib701725d2ec6dacd7862016b2045270956b27029
    Signed-off-by: Anton Kochkov <[email protected]>
---
 payloads/libpayload/drivers/serial.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/payloads/libpayload/drivers/serial.c 
b/payloads/libpayload/drivers/serial.c
index c6804d2..0f79b52 100644
--- a/payloads/libpayload/drivers/serial.c
+++ b/payloads/libpayload/drivers/serial.c
@@ -40,8 +40,6 @@ static void serial_io_hardware_init(int port, int speed, int 
word_bits, int pari
 {
        unsigned char reg;
 
-       /* We will assume 8n1 for now. Does anyone use anything else these 
days? */
-
        /* Disable interrupts. */
        outb(0, port + 0x01);
 
@@ -56,8 +54,9 @@ static void serial_io_hardware_init(int port, int speed, int 
word_bits, int pari
        outb(DIVISOR(speed) & 0xFF, port);
        outb(DIVISOR(speed) >> 8 & 0xFF, port + 1);
 
-       /* Restore the previous value of the divisor. */
-       outb(reg & ~0x80, port + 0x03);
+       /* Restore the previous value of the divisor.
+        * And set 8 bits per character */
+       outb((reg & ~0x80) | 3, port + 0x03);
 }
 
 static void serial_mem_hardware_init(int port, int speed, int word_bits, int 
parity, int stop_bits)
@@ -80,8 +79,9 @@ static void serial_mem_hardware_init(int port, int speed, int 
word_bits, int par
        writeb(DIVISOR(speed) & 0xFF, MEMBASE);
        writeb(DIVISOR(speed) >> 8 & 0xFF, MEMBASE + 1);
 
-       /* Restore the previous value of the divisor. */
-       writeb(reg & ~0x80, MEMBASE + 0x03);
+       /* Restore the previous value of the divisor.
+        * And set 8 bits per character */
+       writeb((reg & ~0x80) | 3, MEMBASE + 0x03);
 }
 #endif
 

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to