[coreboot-gerrit] Patch merged into coreboot/master: elog: Fix duplicate event type

gerrit at coreboot.org gerrit at coreboot.org
Tue Mar 7 00:12:48 CET 2017


the following patch was just integrated into master:
commit 6cbd3980aba050352b09c818ffabe822411c1901
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Sun Mar 5 14:35:57 2017 -0800

    elog: Fix duplicate event type
    
    The current elog implementation has two event types defined for 0xa7,
    apparently the result of divergent coreboot trees on chromium where
    some events were added to ARM systems but not upstreamed until later.
    
    Fix this by moving ELOG_TYPE_THERM_TRIP to be 0xab, since the current
    elog parsing code in chromium is using ELOG_TYPE_SLEEP for 0xa7.
    
    BUG=b:35977516
    TEST=check for proper "CPU Thermal Trip" event when investigating a
    device that is unexpectedly powering down.
    
    Change-Id: Idfa9b2322527803097f4f19f7930ccbdf2eccf35
    Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: https://review.coreboot.org/18579
    Tested-by: build bot (Jenkins)
    Reviewed-by: Furquan Shaikh <furquan at google.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list