[coreboot-gerrit] Patch merged into coreboot/master: f61809a storm: handle dual purpose recovery button

gerrit at coreboot.org gerrit at coreboot.org
Fri Apr 17 10:04:03 CEST 2015


the following patch was just integrated into master:
commit f61809ab38ddde631586da7c1345c8dc787174f4
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Fri Feb 20 12:49:47 2015 -0800

    storm: handle dual purpose recovery button
    
    Storm devices' recovery button is overloaded. Pressing it when the
    system is running is supposed to reset the device. To trigger recovery
    mode the button must be held pressed for at least 5 seconds after
    reset.
    
    Currently interpreting the recovery button state is the responsibility
    of the board (vboot gets a consolidated state, which is a combination
    of several conditions), so the simplest way to implement this feature
    is to make the board follow the recovery button state.
    
    In case the button is not pressed when it is first sampled, its state
    is saved immediately and no recovery request is reported. In case the
    button is pressed when it is first sampled, the board code keeps
    polling it up to 5 seconds and acts accordingly.
    
    BRANCH=storm
    BUG=chrome-os-partner:36059
    TEST=tried starting a whirlwind with recovery button pressed for
         various durations, it entered recovery mode when the button was
         pressed longer than 5 seconds.
    
    Change-Id: Icb3250be7c2a76089c070acd68cb521d1399e245
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 45e7265bc760944f93dd98903d39d2b30aa96365
    Original-Change-Id: Iab3609ebce3a74e3d0270775b83f3cf03a8837ca
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/251711
    Original-Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    Reviewed-on: http://review.coreboot.org/9761
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list