[coreboot-gerrit] Patch merged into coreboot/master: FMAP: Clean up debug output

gerrit at coreboot.org gerrit at coreboot.org
Sat Feb 13 20:03:21 CET 2016


the following patch was just integrated into master:
commit bc2c0a325abca26a7b99e98dfd17a5c7bed2a04f
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Tue Feb 9 09:17:56 2016 -0800

    FMAP: Clean up debug output
    
    Reduce the debug output from FMAP lookups.  When we had one or
    two FMAP lookups in a boot this was not a big deal, but now that
    we do many lookups it is a lot of unnecessary output duplication.
    
    This change reduces these 3 lines:
    
    FMAP: area VBLOCK_A found
    FMAP:   offset: 200000
    FMAP:   size:   65536 bytes
    
    To just one line:
    
    FMAP: area VBLOCK_A found @ 200000 (65536 bytes)
    
    And makes the header output only print once:
    
    FMAP: Found "FMAP" version 1.0 at c10000.
    FMAP: base = 0 size = 1000000 #areas = 29
    
    BUG=chrome-os-partner:40635
    BRANCH=glados
    TEST=boot on chell and enjoy non-truncated memconsole
    
    Change-Id: Ib5862b8bfad113a700faae89089557094aa6d499
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 6890f36536d4ae6fc4988fc8191b0cff4e33e2e6
    Original-Change-Id: Ifefee1ab26e6ee406de552880fbbd5b7916fcadd
    Original-Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/326887
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: https://review.coreboot.org/13695
    Tested-by: build bot (Jenkins)
    Reviewed-by: Martin Roth <martinroth at google.com>


See https://review.coreboot.org/13695 for details.

-gerrit



More information about the coreboot-gerrit mailing list