[coreboot-gerrit] Patch merged into coreboot/master: f0e23d4 uart8250: Fix and unify baudrate divisor calculation

gerrit at coreboot.org gerrit at coreboot.org
Tue Mar 4 15:22:14 CET 2014


the following patch was just integrated into master:
commit f0e23d43d1a0d4c9e80f58b2efbfe84a32228e3f
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Mon Feb 17 19:37:52 2014 +0200

    uart8250: Fix and unify baudrate divisor calculation
    
    Divisor is a function of requested baudrate, platform-specific
    reference clock and amount of oversampling done on the UART reference.
    Calculate this parameter with divisor rounded to nearest integer.
    
    When building without option_table or when there is no entry for
    baud_rate, CONFIG_TTYS0_BAUD is used for default baudrate.
    
    For OxPCIe use of 4 MHz for reference was arbitrary giving correct
    divisor for 115200 but somewhat inaccurate for lower baudrates.
    Actual hardware is 62500000 with 16 times oversampling.
    
    FIXME: Field for baudrate in lb_tables is still incorrect.
    
    Change-Id: I68539738469af780fadd3392263dd9b3d5964d2d
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>


See http://review.coreboot.org/5229 for details.

-gerrit



More information about the coreboot-gerrit mailing list