[coreboot-gerrit] Patch merged into coreboot/master: b376ea6 baytrail: dptf: Add disable trip point methods

gerrit at coreboot.org gerrit at coreboot.org
Tue May 13 21:01:42 CEST 2014


the following patch was just integrated into master:
commit b376ea632f1498174d86fa8f8b78848607492055
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Thu Jan 9 10:10:15 2014 -0800

    baytrail: dptf: Add disable trip point methods
    
    Added a method in each temp sensor to disable the aux trip points
    and then a wrapper function to call this method for each enabled
    temperature sensor.
    
    The event handler function is changed to not use a switch statement
    so it does not need to be serialized.  This was causing issues
    with nested locking between the global lock and the EC PATM mutex.
    
    Some unused code in temp sensors that was added earlier is removed
    and instead a critical threshold is specified in _CRT.
    
    The top level DPTF device _OSC method is expanded to check for the
    passive policy UUID and initialize thermal devices.  This is done
    for both enable and disable steps to ensure that the EC thermal
    thresholds are reset in both cases.
    
    Additionally the priority based _TRT is specified with TRTR=1.
    
    BUG=chrome-os-partner:17279
    BRANCH=rambi
    TEST=build and boot on rambi, load esif_lf kernel drivers and start
    esif_uf application.  Observe that temperature thresholds are set
    properly when running 'appstart Dptf' and that they are disabled
    after running 'appstop Dptf'
    
    Change-Id: Ia15824ca42164dadae2011d4e364b70905e36f85
    Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/182024
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/5037
    Tested-by: build bot (Jenkins)
    Reviewed-by: Kyösti Mälkki <kyosti.malkki at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list