[coreboot-gerrit] Patch merged into coreboot/master: google/gale: Turn on monotonic timer support

[gerrit at coreboot.org]

the following patch was just integrated into master:
commit f136524d002ce4b0ad46836d53c6ca880ef92118
Author: Varadarajan Narayanan <varada at codeaurora.org>
Date:   Wed Apr 20 11:32:19 2016 +0530

    google/gale: Turn on monotonic timer support
    
    Without monotonic timer support, timer related APIs like
    timer_monotonic_get etc. are optimized out by the compiler. This
    results in timed loops to become indefinite loops
    
    	stopwatch_init_msecs_expire(...);
    	do {
    		something();
    	} while (!stopwatch_expired(...));
    
    In our specific case, loops sampling the recovery/wipeout button
    in src/mainboard/google/gale/chromeos.c:get_switch_state() turned
    into infinite loops and the boot didn't proceed.
    
    BUG=chrome-os-partner:49249
    TEST=Confirmed that the loop breaks per the specified timeout
    using the minicom's console log time stamps
    	[2016-04-11 12:34:37] recovery button pressed
    	[2016-04-11 12:34:45] wipeout requested, checking recovery
    	[2016-04-11 12:34:53] recovery requested
    BRANCH=none
    
    Change-Id: I7ed2616c50ebb28b43ad769d3105f7d4e31b1114
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: e00f888570b577849cb526220ffe6f22fe9d2ece
    Original-Change-Id: Ic0b800558ebce482da6321c30dbf732080b82941
    Original-Signed-off-by: Varadarajan Narayanan <varada at codeaurora.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/339873
    Original-Commit-Ready: David Hendricks <dhendrix at chromium.org>
    Original-Tested-by: Kan Yan <kyan at google.com>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Original-Reviewed-by: Kan Yan <kyan at google.com>
    Reviewed-on: https://review.coreboot.org/14673
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Reviewed-by: Martin Roth <martinroth at google.com>


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

-gerrit