[coreboot-gerrit] Patch merged into coreboot/master: 864ec8c libpayload: Consolidate coreboot table parsing

gerrit at coreboot.org gerrit at coreboot.org
Fri Mar 20 15:32:04 CET 2015


the following patch was just integrated into master:
commit 864ec8c488eb195b73c876be8fe2b879efa470e0
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu Jul 31 21:02:57 2014 -0700

    libpayload: Consolidate coreboot table parsing
    
    There are three instances of coreboot.c in libpayload. for x86, arm
    and arm64 architectures. The arm and arm64 instances are exactly the
    same. The differences with the x86 instance are as follows:
    
     - a very slightly different set of coreboot table tags is parsed (one
       tag added and two removed)
    
     - instead of checking a fixed address if it contains the coreboot
       table, the x86 version iterates over two address ranges.
    
    This patch refactors the module, leaving architecture specific
    processing in arch subdirectories and moving the common code into
    libc.
    
    BUG=none
    TEST=none yet
    
    Change-Id: I1c7ad6f74e3498e93df78086ba0ff708c08e0a5c
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 3df209d58ebd5c5b1cf0168f6466e065d1ef3598
    Original-Change-Id: I6dfed73f6ba5939f692d0f98d2774c0e0312a25f
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/210770
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/8750
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list