[coreboot-gerrit] Patch set updated for coreboot: 9b5a2ef VBOOT: Set virtual recovery switch based on EC Software Sync

Aaron Durbin (adurbin@google.com) gerrit at coreboot.org
Tue Jan 28 06:31:49 CET 2014


Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5061

-gerrit

commit 9b5a2ef9cd44c336a0ee8c2f32bc274744f34ffe
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Thu Jan 23 08:34:54 2014 -0800

    VBOOT: Set virtual recovery switch based on EC Software Sync
    
    The Virtual Recovery switch flag needs to be set in coreboot since
    it is passed through directly to VBOOT layer by depthcharge.
    
    Rather than add a new config option we can assume that devices with
    EC Software Sync also have a virtual recovery switch and set the
    flag appropriately.
    
    BUG=chrome-os-partner:25250
    BRANCH=all
    TEST=build and boot on rambi, successfully enter developer mode
    
    Change-Id: Id067eacbc48bc25a86887bce8395fa3a9b85e9f2
    Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/183672
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/vendorcode/google/chromeos/vboot_loader.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/vendorcode/google/chromeos/vboot_loader.c b/src/vendorcode/google/chromeos/vboot_loader.c
index 6eff99c..badff27 100644
--- a/src/vendorcode/google/chromeos/vboot_loader.c
+++ b/src/vendorcode/google/chromeos/vboot_loader.c
@@ -94,8 +94,10 @@ static void vboot_invoke_wrapper(struct vboot_handoff *vboot_handoff)
 		*iflags |= VB_INIT_FLAG_WP_ENABLED;
 	if (CONFIG_VIRTUAL_DEV_SWITCH)
 		*iflags |= VB_INIT_FLAG_VIRTUAL_DEV_SWITCH;
-	if (CONFIG_EC_SOFTWARE_SYNC)
+	if (CONFIG_EC_SOFTWARE_SYNC) {
 		*iflags |= VB_INIT_FLAG_EC_SOFTWARE_SYNC;
+		*iflags |= VB_INIT_FLAG_VIRTUAL_REC_SWITCH;
+	}
 
 	context.handoff = vboot_handoff;
 	context.cparams = &cparams;



More information about the coreboot-gerrit mailing list