[coreboot-gerrit] Patch merged into coreboot/master: 90f6cd3 Add console wrapper for UART driver

gerrit at coreboot.org gerrit at coreboot.org
Tue Apr 14 21:25:35 CEST 2015


the following patch was just integrated into master:
commit 90f6cd35673b10535227db0b2b5fa8379804933f
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Wed Apr 23 13:11:40 2014 -0700

    Add console wrapper for UART driver
    
    Coreboot is designed to have a single serial console at most, on top
    of that it may have a CBMEM (virtual) console. Matters are complicated
    by the fact that console interface is different between bootblock and
    later stages.
    
    A linker list of console driver descriptors is used to allow to
    determine the set and type of console drivers at compile time. Even
    though the upstream seems to have done away with this approach, which
    does not seem the best idea.
    
    As an alternative this patch introduces a common wrapper which
    different UART drivers can plug in into. The driver exports a single
    API which can be used both directly (in bootblock) and through the
    wrapper (in later stages).
    
    The existing drivers can be adjusted to fit this scheme one by one.
    The common UART driver API also aligns fine with the upstream
    approach.
    
    BUG=chrome-os-partner:27784
    TEST=none yet
    
    Original-Change-Id: Id1fe73d29f2a3c722bd77180beebaedb9bf7d6a1
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/196660
    Original-Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    (cherry picked from commit 94a36ad79a96f83d283c0fd073b05f98ae48820c)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: Id1fe73d29f2a3c722bd77180beebaedb9bf7d6a1
    Reviewed-on: http://review.coreboot.org/7872
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list