[coreboot-gerrit] Patch merged into coreboot/master: 9f5a5c5 Add table driven way to add platform specific reg_script routines

gerrit at coreboot.org gerrit at coreboot.org
Sat Apr 4 12:40:44 CEST 2015


the following patch was just integrated into master:
commit 9f5a5c532343fe72753fc507b0f2ef1a26afabd3
Author: Lee Leahy <leroy.p.leahy at intel.com>
Date:   Fri Aug 29 13:38:59 2014 -0700

    Add table driven way to add platform specific reg_script routines
    
    Extend lib/reg_script.c to use a platform table to declare
    additional platform specific register access routine functions.
    REG_SCRIPT_TYPE_PLATFORM_BASE is the starting value for platform
    specific register types.  Additional register access types may be
    defined above this value.  The type and access routines are placed
    into reg_script_type_table.
    
    The Baytrail type value for IOSF was left the enumeration since it
    was already defined and is being used for Braswell.
    
    BRANCH=none
    BUG=None
    TEST=Use the following steps to test:
    1.  Build for a Baytrail platform
    2.  Build for the Samus platform
    3.  Add a platform_bus_table routine to a platform which returns the
        address of an array of reg_script_bus_entry structures and the
        number of entries in the array.
    
    Change-Id: Ic99d345c4b067c52b4e9c47e59ed4472a05bc1a5
    Signed-off-by: Stefan Reinauer <reinauer at chromium.org>
    Original-Commit-Id: 2d9fecf4287dff6311a81d818603212248f1a248
    Original-Signed-off-by: Lee Leahy <leroy.p.leahy at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/215645
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Change-Id: I7cd37abc5a08cadb3166d4048f65b919b86ab5db
    Original-Reviewed-on: https://chromium-review.googlesource.com/229612
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: http://review.coreboot.org/9279
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list