[coreboot-gerrit] Patch merged into coreboot/master: 24a53dd ipq806x: clean up UART driver tx_byte function

gerrit at coreboot.org gerrit at coreboot.org
Sat Jan 3 20:02:39 CET 2015


the following patch was just integrated into master:
commit 24a53dddb9b0b545ab9208722ff7b684114b4e91
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Tue May 27 18:03:38 2014 -0700

    ipq806x: clean up UART driver tx_byte function
    
    The driver as it was copied from u-boot provided a function to
    transmit multiple characters in one invocation. This feature was not
    ported to coreboot, there is no need to maintain the complexity when
    only one character at a time is transmitted. It is also very desirable
    to get rid of a 1024 byte array allocated on the stack.
    
    The array was necessary to allow to convert multiple newline
    characters in the transmit data flow into two character sequences
    CRLF. Now just a single word is enough to keep one or two characters
    to transmit.
    
    [EDIT km: newline translation is now part of printk]
    
    BUG=chrome-os-partner:27784
    TEST=verified that coreboot with the new code prints generates console
         output.
    
    Original-Change-Id: I73869c5f4ca87210b34811b583386554bafff1e7
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/201782
    Original-Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    Original-Reviewed-by: Trevor Bourget <tbourget at codeaurora.org>
    (cherry picked from commit eab3dc9d30c7e8355a2563e18ada78e4070e6151)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I4274b8f7188bf9636906b39bcd9ec7adf0e1222e
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
    Reviewed-on: http://review.coreboot.org/8011
    Tested-by: build bot (Jenkins)
    Reviewed-by: Marc Jones <marc.jones at se-eng.com>


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

-gerrit



More information about the coreboot-gerrit mailing list